Announcement - xxv-1.0

UloPe · 16. November 2007

Zitat

Originally posted by Hulk
Das sehe ich nicht so, /bin/sh ist auf allen System der kleinste gemeinsame Nenner, zeigt immer auf eine zu POSIX konforme Shell. Im allgemeinen meist per symbolischen Link auf bash/csh/ksh .... Wenn das nicht der Fall ist, wurde das manuell geaendert. Auf Debian heisst die Shell dash

Da ist mir durchaus bewust.

Evtl. habe ich mich unklar ausgedrückt:
Das Skript läuft mit nicht bash shells einfach nicht! (Da es functions usw nutzt.)
Als Test:

Code

dash xxv-1.0/install.sh

Von daher ist es extrem sinnvoll das shebang zu ändern.

UloPe

anbr · 16. November 2007

Zitat
Original von UloPe
Als Test:
Code
dash xxv-1.0/install.sh
Von daher ist es extrem sinnvoll das shebang zu ändern.

Ich würde es eher als sinnvoller erachten, die stellen anpassen, damit sie mit jeder "Standard" Shell funktioniert... wenn z.b. Dash damit Probleme hat...

Bitte teste doch bitte folgende Modifikationen

Diff

--- install.sh  (Revision 1205)
+++ install.sh  (Arbeitskopie)
@@ -66,7 +66,7 @@
 
 PATH=$PATH:/sbin:/bin:/usr/sbin:/usr/bin
 
-function usage()
+usage()
 {
        echo ""
        echo "usage: $(basename $0) [-h] [-p] [-u]"
@@ -79,7 +79,7 @@
        exit 0
 }
 
-function killRunning()
+killRunning()
 {
        local KILLED=0
        ps a | grep xxvd | grep perl | grep -v grep | while read PID UNWANTED
@@ -92,7 +92,7 @@
 }
 
 # $1 - the Perl module to check for.
-function checkPerlModule()
+checkPerlModule()
 {
        [ -z "$1" ] && return 1
 
@@ -120,7 +120,7 @@
        fi
 }
 
-function perlModules()
+perlModules()
 {
        checkPerlModule CGI
        checkPerlModule Compress::Zlib
@@ -158,7 +158,7 @@
        checkPerlModule XML::Simple
 }
 
-function makeDir()
+makeDir()
 {
        [ -z "$1" ] && return 1
        local DIR=$1
@@ -184,7 +184,7 @@
        return 0
 }
 
-function doInstall()
+doInstall()
 {
        echo ""
        echo "********* Installing xxv *************"
@@ -346,7 +346,7 @@
        exit 0
 }
 
-function doUninstall()
+doUninstall()
 {
        echo ""
        echo "********* Uninstalling xxv *************"

Alles anzeigen

Cu,
Andreas

Announcement - xxv-1.0

Jetzt mitmachen!

Teilen