]>
Commit | Line | Data |
---|---|---|
de1fc6ce | 1 | READ THIS BEFORE CHANGEING SOMETHING IN RC-SCRIPTS PACKAGE |
ec8b15cb | 2 | Arkadiusz Miskiewicz <misiek@pld-linux.org> |
de1fc6ce JR |
3 | |
4 | 1) | |
ec8b15cb | 5 | consult all major changes with people on pld-rc-scripts@pld-linux.org |
de1fc6ce JR |
6 | mailing list. This is very important. |
7 | ||
8 | 2) | |
944d6d80 | 9 | use 'local' for local variables in functions for example: |
de1fc6ce JR |
10 | |
11 | function_ble () | |
12 | { | |
944d6d80 | 13 | local qw er=0 |
de1fc6ce JR |
14 | qw=$(tty) |
15 | er=/dev/console | |
16 | echo "$qw $er" | |
17 | } | |
18 | ||
19 | 3) | |
20 | use $() instead `` for command execution for example | |
21 | ||
22 | something=$(whoami) | |
23 | instead of | |
24 | something="`whoami`" | |
25 | ||
fcf7cc19 JR |
26 | Warning! The following snippet will not work, use `` in such cases: |
27 | There is one ')' too many there. | |
28 | ||
29 | interfaces_boot=$( | |
30 | case $i in | |
31 | *ifcfg-lo) continue ;; | |
32 | esac | |
33 | ) | |
34 | ||
944d6d80 | 35 | 4) don't use sed ! It only causes one more Requires. |
de1fc6ce | 36 | Use awk ' { gsub(/pattern/,"replaced"); print $0; } ' for that. |
52c53926 | 37 | Also try to replace grep calls with awk calls. |
de1fc6ce | 38 | |
944d6d80 | 39 | 5) |
de1fc6ce JR |
40 | (PL) Tlumaczac komunikaty na PL nalezy uzywac formy bezosobowej np: |
41 | ||
42 | uruchamianie uslugi | |
43 | zamiast | |
44 | uruchamiam usluge | |
6f90b667 ER |
45 | |
46 | ||
47 | ||
48 | HOW TO MAKE A RELEASE | |
49 | ||
a0668280 ER |
50 | - git checkout master |
51 | - git pull --rebase | |
56dc3756 | 52 | - ./autogen.sh |
6f90b667 ER |
53 | - make distcheck |
54 | - update configure.ac release and commit it | |
55 | - make dist | |
56 | - ./make-tag.sh | |
57 | - upload tarball to distfiles, update rc-scripts.spec:HEAD |