Re: [PATCH] Add 'pre-inst-env' wrapper script

2017-09-22 Thread Mathieu Lirzin
Hans-Bernhard Bröker writes: > Am 22.09.2017 um 12:26 schrieb Mathieu Lirzin: > >> My patch was missing this configure.ac change: >> >> --8<---cut here---start->8--- >> AC_CONFIG_FILES([pre-inst-env:build-aux/pre-inst-env.in], >>[chmod +x pre-inst-env]) >>

Re: [PATCH] Add 'pre-inst-env' wrapper script

2017-09-22 Thread Hans-Bernhard Bröker
Am 22.09.2017 um 12:26 schrieb Mathieu Lirzin: My patch was missing this configure.ac change: --8<---cut here---start->8--- AC_CONFIG_FILES([pre-inst-env:build-aux/pre-inst-env.in], [chmod +x pre-inst-env]) --8<---cut here---end

Re: [PATCH] Add 'pre-inst-env' wrapper script

2017-09-22 Thread Mathieu Lirzin
Hans-Bernhard Bröker writes: > Am 22.09.2017 um 13:03 schrieb Gavin Smith: >> On Fri, Sep 22, 2017 at 12:26:46PM +0200, Mathieu Lirzin wrote: >>> I think you forgot to include build-aux/pre-inst-env.in in revision >>> r7954. >> >> OK I have committed that file. > > Ah, but there doesn't appear to

Re: [PATCH] Add 'pre-inst-env' wrapper script

2017-09-22 Thread Hans-Bernhard Bröker
Am 22.09.2017 um 13:03 schrieb Gavin Smith: On Fri, Sep 22, 2017 at 12:26:46PM +0200, Mathieu Lirzin wrote: I think you forgot to include build-aux/pre-inst-env.in in revision r7954. OK I have committed that file. Ah, but there doesn't appear to be a (correct) makefile rule that actually us

Re: [PATCH] Add 'pre-inst-env' wrapper script

2017-09-22 Thread Gavin Smith
On Fri, Sep 22, 2017 at 12:26:46PM +0200, Mathieu Lirzin wrote: > I think you forgot to include build-aux/pre-inst-env.in in revision > r7954. OK I have committed that file. > Additionnally some cleanups could be done such as removing the > 'top_builddir' and 'top_srcdir' environment variables ha

Re: [PATCH] Add 'pre-inst-env' wrapper script

2017-09-22 Thread Mathieu Lirzin
Gavin Smith writes: > On Sat, Aug 26, 2017 at 06:41:47PM +0200, Mathieu Lirzin wrote: > >> While hacking on 'texi2any' and testing my changes with manuals located >> in various places in the filesystem, I found it cumbersome to launch the >> non installed 'makeinfo' like this: >> >> TEXINFO_DE

Re: [PATCH] Add 'pre-inst-env' wrapper script

2017-09-22 Thread Gavin Smith
On Sat, Aug 26, 2017 at 06:41:47PM +0200, Mathieu Lirzin wrote: > Hello, > > While hacking on 'texi2any' and testing my changes with manuals located > in various places in the filesystem, I found it cumbersome to launch the > non installed 'makeinfo' like this: > > TEXINFO_DEV_SOURCE=1 top_srcd