This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[RFA-v2] contrib/ari: Avoid permission issue for script
- From: "Pierre Muller" <pierre dot muller at ics-cnrs dot unistra dot fr>
- To: "'Tom Tromey'" <tromey at redhat dot com>
- Cc: <gdb-patches at sourceware dot org>
- Date: Mon, 5 Nov 2012 12:06:35 +0100
- Subject: [RFA-v2] contrib/ari: Avoid permission issue for script
- References: <11045.9555559375$1351807023@news.gmane.org> <87a9uzq5j1.fsf@fleche.redhat.com>
> Pierre> I can lift the problem for the main script,
> Pierre> by calling it using:
> Pierre> sh ../../src/gdb/contrib/ari/ create-web-ari-in-src.sh,
> Pierre> but this script calls update-web-ari.sh script
> Pierre> without using '. script'...
>
> I think you probably want $SHELL rather than '.' here.
> '.' runs it in the current shell, but the previous code launched a new
> shell.
You are right of course.
Here is a new version of the patch.
> Pierre> Should I add myself as contrib/ari maintainer?
>
> Sure.
Submitted as a seperate thread.
ChangeLog entry:
2012-11-05 Pierre Muller <muller@sourceware.org>
* contrib/ari/create-web-ari-in-src.sh: Avoid problem if script
is not executable.
Index: create-web-ari-in-src.sh
===================================================================
RCS file: /cvs/src/src/gdb/contrib/ari/create-web-ari-in-src.sh,v
retrieving revision 1.3
diff -u -r1.3 create-web-ari-in-src.sh
--- create-web-ari-in-src.sh 2 Nov 2012 18:19:36 -0000 1.3
+++ create-web-ari-in-src.sh 5 Nov 2012 11:03:29 -0000
@@ -67,7 +67,7 @@
fi
# Launch update-web-ari.sh in same directory as current script.
-${scriptpath}/update-web-ari.sh ${srcdir} ${tempdir} ${webdir} gdb
+${SHELL} ${scriptpath}/update-web-ari.sh ${srcdir} ${tempdir} ${webdir} gdb
if [ -f "${webdir}/index.html" ] ; then
echo "ARI output can be viewed in file \"${webdir}/index.html\""