* build failure emacs-26 branch on Mingw64
@ 2018-01-01 11:34 Stephen Leake
2018-01-01 12:33 ` Stephen Leake
0 siblings, 1 reply; 5+ messages in thread
From: Stephen Leake @ 2018-01-01 11:34 UTC (permalink / raw)
To: emacs-devel
Building emacs-26 branch on Mingw64, in a fresh checkout, configure
fails with:
config.status: executing src/epaths.h commands
sed: -e expression #1, char 79: unknown option to `s'
make: *** [Makefile:383: epaths-force-w32] Error 1
config.status: error: 'src/epaths.h' could not be made.
I'll investigate further.
--
-- Stephe
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: build failure emacs-26 branch on Mingw64
2018-01-01 11:34 build failure emacs-26 branch on Mingw64 Stephen Leake
@ 2018-01-01 12:33 ` Stephen Leake
2018-01-01 12:51 ` Andreas Schwab
0 siblings, 1 reply; 5+ messages in thread
From: Stephen Leake @ 2018-01-01 12:33 UTC (permalink / raw)
To: emacs-devel
Stephen Leake <stephen_leake@stephe-leake.org> writes:
> Building emacs-26 branch on Mingw64, in a fresh checkout, configure
> fails with:
>
> config.status: executing src/epaths.h commands
> sed: -e expression #1, char 79: unknown option to `s'
> make: *** [Makefile:383: epaths-force-w32] Error 1
> config.status: error: 'src/epaths.h' could not be made.
>
> I'll investigate further.
The sed command that gives the error is:
-e 's;\(#.*PATH_SITELOADSEARCH\).*$$;\1 "'"$${w32locallisppath//;/\\;}"'";'
There is an unquoted ; in the replacement, but quoting that does not fix
the error.
I'm guessing the reference to 'w32locallisppath' is supposed to replace
the Unix path separator : with the Windows path separator ; . But the
shell syntax for that would be:
${w32locallisppath/:/;}
The escaped make syntax would then be:
$${w32locallisppath/:/\\;}
but that doesn't fix the error either. And 'msys-to-w32' has already
done that substitution, so doing it here is wrong.
I could not find a variation that fixes the error.
'git blame' says this line was last edited in 2014:
7c86a2a7d82 (Dani Moncayo 2014-11-08 15:55:09 +0100 388) -e 's;\(#.*PATH_SITELOADSEARCH\).*$$;\1 "'"$${w32locallisppath//;/\\;}"'";' \
so this has apparently been working for a long time; I don't understand
why it's failing for me now.
I don't specify --enable-locallisppath, so I can work around this by
simply deleting that line from Makefile.in. That lets configure complete.
--
-- Stephe
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: build failure emacs-26 branch on Mingw64
2018-01-01 12:33 ` Stephen Leake
@ 2018-01-01 12:51 ` Andreas Schwab
2018-01-01 13:15 ` Richard Copley
0 siblings, 1 reply; 5+ messages in thread
From: Andreas Schwab @ 2018-01-01 12:51 UTC (permalink / raw)
To: Stephen Leake; +Cc: emacs-devel
On Jan 01 2018, Stephen Leake <stephen_leake@stephe-leake.org> wrote:
> The sed command that gives the error is:
>
> -e 's;\(#.*PATH_SITELOADSEARCH\).*$$;\1 "'"$${w32locallisppath//;/\\;}"'";'
>
> There is an unquoted ; in the replacement, but quoting that does not fix
> the error.
The regexp separator for the s command should be changed to something
else than `;' to avoid a quoting nightmare.
> I'm guessing the reference to 'w32locallisppath' is supposed to replace
> the Unix path separator : with the Windows path separator ; .
No, it replaces `;' with `\;' so that it can be interpolated into the
replacement part of the s command.
Andreas.
--
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: build failure emacs-26 branch on Mingw64
2018-01-01 12:51 ` Andreas Schwab
@ 2018-01-01 13:15 ` Richard Copley
2018-01-01 17:51 ` Stephen Leake
0 siblings, 1 reply; 5+ messages in thread
From: Richard Copley @ 2018-01-01 13:15 UTC (permalink / raw)
To: Stephen Leake; +Cc: Andreas Schwab, emacs-devel
On 1 January 2018 at 12:51, Andreas Schwab <schwab@linux-m68k.org> wrote:
> On Jan 01 2018, Stephen Leake <stephen_leake@stephe-leake.org> wrote:
>
>> The sed command that gives the error is:
>>
>> -e 's;\(#.*PATH_SITELOADSEARCH\).*$$;\1 "'"$${w32locallisppath//;/\\;}"'";'
>>
>> There is an unquoted ; in the replacement, but quoting that does not fix
>> the error.
>
> The regexp separator for the s command should be changed to something
> else than `;' to avoid a quoting nightmare.
>
>> I'm guessing the reference to 'w32locallisppath' is supposed to replace
>> the Unix path separator : with the Windows path separator ; .
>
> No, it replaces `;' with `\;' so that it can be interpolated into the
> replacement part of the s command.
Building the emacs-26 branch from scratch works fine for me.
If I insert an echo command in Makefile (see patch below),
then run "make epaths-force-w32", the output is
s;\(#.*PATH_SITELOADSEARCH\).*$;\1
"%emacs_dir%/share/emacs/26.0.90/site-lisp\;%emacs_dir%/share/emacs/site-lisp";
Stephe, what do you get?
The patch:
--- c:/projects/emacs/Makefile
+++ #<buffer Makefile>
@@ -384,6 +384,7 @@
w32prefix=`${srcdir}/build-aux/msys-to-w32 "${prefix}"`; \
w32locallisppath=`${srcdir}/build-aux/msys-to-w32 "${locallisppath}"` ; \
w32locallisppath=$${w32locallisppath//$${w32prefix}/"%emacs_dir%"} ; \
+ echo 's;\(#.*PATH_SITELOADSEARCH\).*$$;\1
"'"$${w32locallisppath//;/\\;}"'";' ; \
sed < ${srcdir}/nt/epaths.nt > epaths.h.$$$$ \
-e 's;\(#.*PATH_SITELOADSEARCH\).*$$;\1
"'"$${w32locallisppath//;/\\;}"'";' \
-e '/^.*#/s/@VER@/${version}/g' \
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: build failure emacs-26 branch on Mingw64
2018-01-01 13:15 ` Richard Copley
@ 2018-01-01 17:51 ` Stephen Leake
0 siblings, 0 replies; 5+ messages in thread
From: Stephen Leake @ 2018-01-01 17:51 UTC (permalink / raw)
To: emacs-devel
Richard Copley <rcopley@gmail.com> writes:
> On 1 January 2018 at 12:51, Andreas Schwab <schwab@linux-m68k.org> wrote:
>> On Jan 01 2018, Stephen Leake <stephen_leake@stephe-leake.org> wrote:
>>
>>> The sed command that gives the error is:
>>>
>>> -e 's;\(#.*PATH_SITELOADSEARCH\).*$$;\1 "'"$${w32locallisppath//;/\\;}"'";'
>>>
>>> There is an unquoted ; in the replacement, but quoting that does not fix
>>> the error.
>>
>> The regexp separator for the s command should be changed to something
>> else than `;' to avoid a quoting nightmare.
>>
>>> I'm guessing the reference to 'w32locallisppath' is supposed to replace
>>> the Unix path separator : with the Windows path separator ; .
>>
>> No, it replaces `;' with `\;' so that it can be interpolated into the
>> replacement part of the s command.
>
> Building the emacs-26 branch from scratch works fine for me.
I'm guessing you are using the same tools I am.
Are you running configure from the mingw64 shell? Which make is
first in path?
As discussed in the thread on my other build failure, this is fixed by
using /usr/bin/make
> If I insert an echo command in Makefile (see patch below), then run
> "make epaths-force-w32",
You can get the same effect by removing the '@'; that makes the entire
command visible.
--
-- Stephe
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-01-01 17:51 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-01-01 11:34 build failure emacs-26 branch on Mingw64 Stephen Leake
2018-01-01 12:33 ` Stephen Leake
2018-01-01 12:51 ` Andreas Schwab
2018-01-01 13:15 ` Richard Copley
2018-01-01 17:51 ` Stephen Leake
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.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).