* fix bash autoconfiguration
@ 2019-01-17 2:11 David Bremner
2019-01-17 2:11 ` [PATCH 1/2] configure: don't use special variable BASH David Bremner
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: David Bremner @ 2019-01-17 2:11 UTC (permalink / raw)
To: notmuch
Testing by people running with /bin/sh = bash welcome. The bug shows
up as a shebang of /bin/sh in /usr/bin/notmuch-emacs-mua
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/2] configure: don't use special variable BASH
2019-01-17 2:11 fix bash autoconfiguration David Bremner
@ 2019-01-17 2:11 ` David Bremner
2019-01-17 2:11 ` [PATCH 2/2] Debian: use new variable to force bash location David Bremner
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: David Bremner @ 2019-01-17 2:11 UTC (permalink / raw)
To: notmuch
bash, in it's wisdom, sets that variable when invoked as /bin/sh, but
then doesn't act as bash, at least not to the degree we need to run
scripts.
---
configure | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/configure b/configure
index b2200be0..ea22587b 100755
--- a/configure
+++ b/configure
@@ -53,7 +53,7 @@ fi
# Set several defaults (optionally specified by the user in
# environment variables)
-BASH=${BASH:-bash}
+BASHCMD=${BASHCMD:-bash}
PERL=${PERL:-perl}
CC=${CC:-cc}
CXX=${CXX:-c++}
@@ -560,13 +560,13 @@ else
fi
printf "Checking for bash... "
-if command -v ${BASH} > /dev/null; then
+if command -v ${BASHCMD} > /dev/null; then
have_bash=1
- bash_absolute=$(command -v ${BASH})
+ bash_absolute=$(command -v ${BASHCMD})
printf "Yes (%s).\n" "$bash_absolute"
else
have_bash=0
- printf "No. (%s not found)\n" "${BASH}"
+ printf "No. (%s not found)\n" "${BASHCMD}"
fi
printf "Checking for perl... "
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] Debian: use new variable to force bash location
2019-01-17 2:11 fix bash autoconfiguration David Bremner
2019-01-17 2:11 ` [PATCH 1/2] configure: don't use special variable BASH David Bremner
@ 2019-01-17 2:11 ` David Bremner
2019-01-17 7:15 ` fix bash autoconfiguration Tomi Ollila
2019-01-30 14:11 ` David Bremner
3 siblings, 0 replies; 5+ messages in thread
From: David Bremner @ 2019-01-17 2:11 UTC (permalink / raw)
To: notmuch
This should no longer be needed on the official debian autobuilders,
but it might help others with merged /usr build environments.
---
debian/rules | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/debian/rules b/debian/rules
index c8ecb1ac..d056edb6 100755
--- a/debian/rules
+++ b/debian/rules
@@ -6,7 +6,7 @@ python3_all = py3versions -s | xargs -n1 | xargs -t -I {} env {}
dh $@ --with python2,python3,elpa
override_dh_auto_configure:
- BASH=/bin/bash ./configure --prefix=/usr \
+ BASHCMD=/bin/bash ./configure --prefix=/usr \
--libdir=/usr/lib/$$(dpkg-architecture -q DEB_TARGET_MULTIARCH) \
--includedir=/usr/include \
--mandir=/usr/share/man \
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: fix bash autoconfiguration
2019-01-17 2:11 fix bash autoconfiguration David Bremner
2019-01-17 2:11 ` [PATCH 1/2] configure: don't use special variable BASH David Bremner
2019-01-17 2:11 ` [PATCH 2/2] Debian: use new variable to force bash location David Bremner
@ 2019-01-17 7:15 ` Tomi Ollila
2019-01-30 14:11 ` David Bremner
3 siblings, 0 replies; 5+ messages in thread
From: Tomi Ollila @ 2019-01-17 7:15 UTC (permalink / raw)
To: David Bremner, notmuch
On Wed, Jan 16 2019, David Bremner wrote:
> Testing by people running with /bin/sh = bash welcome. The bug shows
> up as a shebang of /bin/sh in /usr/bin/notmuch-emacs-mua
LGTM
Tomi
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: fix bash autoconfiguration
2019-01-17 2:11 fix bash autoconfiguration David Bremner
` (2 preceding siblings ...)
2019-01-17 7:15 ` fix bash autoconfiguration Tomi Ollila
@ 2019-01-30 14:11 ` David Bremner
3 siblings, 0 replies; 5+ messages in thread
From: David Bremner @ 2019-01-30 14:11 UTC (permalink / raw)
To: notmuch
David Bremner <david@tethera.net> writes:
> Testing by people running with /bin/sh = bash welcome. The bug shows
> up as a shebang of /bin/sh in /usr/bin/notmuch-emacs-mua
>
Last call for testing by people with /bin/sh = bash.
d
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-01-30 14:11 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-17 2:11 fix bash autoconfiguration David Bremner
2019-01-17 2:11 ` [PATCH 1/2] configure: don't use special variable BASH David Bremner
2019-01-17 2:11 ` [PATCH 2/2] Debian: use new variable to force bash location David Bremner
2019-01-17 7:15 ` fix bash autoconfiguration Tomi Ollila
2019-01-30 14:11 ` David Bremner
Code repositories for project(s) associated with this public inbox
https://yhetil.org/notmuch.git/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).