[arch-general] [PATCH 01/48] Bashification of initscripts

Allan McRae allan at archlinux.org
Wed Jul 7 00:03:49 EDT 2010


Here is a quick review on all these patches.   I recommend that the lvm 
and crypttab changes get a decent amount of testing before these go live 
as they are the biggest changes being done.


Tighten up the console size finding code a bit.
   Add some white space in test construct:
     if ((STAT_COL==0)); then
     if (( STAT_COL == 0 )); then
   and throughout these patches


Simplify the code that clears USECOLOR.
   The following condition is removed with no commit message to explain why
     if [ $? = 3 ]; then
       TERM_COLOURS=8


Replace trivial use of grep with bash regex conditional.
   - if [ -n "$CONSOLEMAP" ] && echo "$LOCALE" | /bin/grep -qi utf ; then
   + [[ $CONSOLEMAP && $LOCALE =~ UTF|utf ]] && CONSOLEMAP=""

   Use ... && ${LOCALE,,} == utf ]] to accurately replicate the grep


Replace slightly too long echo staement with a here document.
                                   ^^ typo
   I actually find the echo more readable


Change the daemon runnign loop to use a case statement.
Quote daemon names.
   Merge these commits


Both rc.single and rc.shutdown use the same code to kill everything.
     + # $1 = where we are being called from.
     + # This is used to determine which hooks to run.
   ->  Add separater line here...
     + # Find daemons NOT in the DAEMONS array. Shut these down first

   Why has this been removed:
     -if [ -x /etc/rc.local.shutdown ]; then
     - /etc/rc.local.shutdown
     -fi
   Ah... it has been moved to another place in another commit.  Please 
document these sorts of changes in your commit message and preferably do 
the entire move in one commit.


Flatten LVM deactivation if block in rc.shutdown.
   This change does not do the same thing and I do not see where it gets 
replicated
     -if [ -f /etc/crypttab -a -n "$(/bin/grep -v ^# /etc/crypttab | 
/bin/grep -v ^$)" ]; then
     +if [[ -f /etc/crypttab ]]; then

   Also another:
   +if [[ $USELVM =~ yes|YES
   -> ${USELVM,,} == yes


bashify bringing up the loopback adaptor.
   Add a commit message as that is doing a lot more than bashifing.


Bashify locale setting.
   $LOCALE =~ utf|UTF


Allan


More information about the arch-general mailing list