---
easy-build.sh | 28
1 files changed, 16 insertions(+), 12 deletions(-)
diff --git a/easy-build.sh b/easy-build.sh
index c586c1e..d85d810 100755
--- a/easy-build.sh
+++ b/easy-build.sh
@@ -25,19 +25,23 @@ if [ $# -eq 0 ]; then
fi
desktop=
-if [ $1 = -d ];
On Friday 07 August 2009, Ian Campbell wrote:
+while getopts d:h OPT ; do
Is getopts also supported in dash?
+ case $OPT in
+ d)
+ case $OPTARG in
+ # Note: gnome is the special gnome task, not the generic
task
+
On Fri, Aug 07, 2009 at 02:05:52PM +0200, Frans Pop wrote:
On Friday 07 August 2009, Ian Campbell wrote:
+while getopts d:h OPT ; do
Is getopts also supported in dash?
Yes, and in POSIX.
Maybe it would be good to also support --help if -h is added.
getopts can't do longoptions. I'd say
On Fri, 2009-08-07 at 14:05 +0200, Frans Pop wrote:
On Friday 07 August 2009, Ian Campbell wrote:
+while getopts d:h OPT ; do
Is getopts also supported in dash?
I believe so.
I just tried it and it looks like the script is already not dash-ready:
$ dash -x ./easy-build.sh -h
+ set -e
+
On Fri, 2009-08-07 at 13:33 +0100, Ian Campbell wrote:
I just tried it and it looks like the script is already not
dash-ready:
$ dash -x ./easy-build.sh -h
+ set -e
+ export CF=CONF.sh
+ . CONF.sh
.: 1: CONF.sh: not found
I'm no expert but it looks like dash obeys $PATH when executing
On Fri, 2009-08-07 at 13:45 +0100, Ian Campbell wrote:
On Fri, 2009-08-07 at 13:33 +0100, Ian Campbell wrote:
I just tried it and it looks like the script is already not
dash-ready:
$ dash -x ./easy-build.sh -h
+ set -e
+ export CF=CONF.sh
+ . CONF.sh
.: 1: CONF.sh: not found
Ian Campbell wrote:
On Fri, 2009-08-07 at 13:33 +0100, Ian Campbell wrote:
I just tried it and it looks like the script is already not
dash-ready:
$ dash -x ./easy-build.sh -h
+ set -e
+ export CF=CONF.sh
+ . CONF.sh
.: 1: CONF.sh: not found
I'm no expert but it looks like dash obeys $PATH
Hi,
one nitpick:
On Fri, Aug 7, 2009 at 13:33:33 +0100, Ian Campbell wrote:
+shift $(expr $OPTIND - 1)
maybe 'shift $((OPTIND - 1))' so you don't fork expr?
Cheers,
Julien
--
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact
On Friday 07 August 2009, Ian Campbell wrote:
# Set configuration file to be used for the build and source it
-export CF=CONF.sh
+export CF=./CONF.sh
I've already committed that (for all scripts that set/source it).
--
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a
9 matches
Mail list logo