[Comm] python

[php-coder]

--- Anton Gorlov 2005-05-14 12:49:06 +0400
+++ php-coder 2005-05-14 18:09:04 +0700

    хм... :)) Можно скрипты ваши покритиковать/высказать свои мысли
    по-поводу?

> #!/bin/bash

    Как говорят на ЛОРе: "Башизмы давить!" (с)

    А почему: #!/bin/sh    

> echo YOU MUST BE ROOT TO RUN THIS SCRIPT
> echo

    Просто так выводить надпись не интересно :)) Вот, тут рядом был скрипт
    lav@ там была проверка на рутность. Адаптируя к вашему скрипту, я бы
    написал так:

    if [ "$UID" != "0" ]; then
       echo -e "\e[31;1;40mYou must be root to run this script!\e[0m" > /dev/stderr
       exit 1
    fi
      
> mkdir /var/log/ibs

    Для надежности я бы добавил опцию -p

> chown apache /var/log/ibs

    Это вы тут владельца меняете или группу?

> chkconfig --add ibs

    chkconfig ibs on

> echo Enter system \(default admin\) password:
> read pass

    read -p "Enter system (default admin) password: " pass

> enc_pass=`/usr/local/ibs/sc/md5crypt.py $pass`

    У всех свои вкусы конечно, но мне больше по душе такая форма: $(command)

    
    Удачи!

-- 
+ php-coder
+ php-coder[at]altlinux[dot]ru