all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Emacs master build broken [MSYS2/UCRT64]
@ 2023-08-18 21:35 Angelo Graziosi
  2023-08-18 21:54 ` Angelo Graziosi
  2023-08-19  6:06 ` Eli Zaretskii
  0 siblings, 2 replies; 4+ messages in thread
From: Angelo Graziosi @ 2023-08-18 21:35 UTC (permalink / raw)
  To: emacs-devel@gnu.org

I have built master two day ago on MSYS2/UCRT64 using their patches (https://github.com/msys2/MINGW-packages/tree/master/mingw-w64-emacs).

Now a recent change (Po Lu f2f2e6a082a541 ?) to src/sysdep.c has broken the build because the MSYS2 people patche does not apply:

patching file nt/cmdproxy.c
Hunk #1 succeeded at 38 (offset 3 lines).
patching file src/sysdep.c
Hunk #1 succeeded at 2985 (offset 292 lines).
Hunk #2 FAILED at 2704.
1 out of 2 hunks FAILED -- saving rejects to file src/sysdep.c.rej
Error: Failure patching nt/cmdproxy.c

where

$ cat emacs-master/src/sysdep.c.rej
--- src/sysdep.c.orig   2022-06-23 20:59:02.252891700 +0200
+++ src/sysdep.c        2022-06-23 21:05:31.998322600 +0200
@@ -2704,7 +2706,11 @@
   if (err | (ADDRESS_SANITIZER
             ? fflush (stderr) != 0 || ferror (stderr)
             : close_stream (stderr) != 0))
+#ifndef _UCRT // XXX: close_stream(stderr) fails for some reason
     _exit (EXIT_FAILURE);
+#else
+    ;
+#endif
 }


 #ifndef DOS_NT

Maybe you now how this should be fixed...

TIA



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Emacs master build broken [MSYS2/UCRT64]
  2023-08-18 21:35 Emacs master build broken [MSYS2/UCRT64] Angelo Graziosi
@ 2023-08-18 21:54 ` Angelo Graziosi
  2023-08-19  6:06 ` Eli Zaretskii
  1 sibling, 0 replies; 4+ messages in thread
From: Angelo Graziosi @ 2023-08-18 21:54 UTC (permalink / raw)
  To: emacs-devel@gnu.org


> Il 18/08/2023 23:35 CEST Angelo Graziosi ha scritto:
> 
>  
> I have built master two day ago on MSYS2/UCRT64 using their patches (https://github.com/msys2/MINGW-packages/tree/master/mingw-w64-emacs).
> 
> Now a recent change (Po Lu f2f2e6a082a541 ?) to src/sysdep.c has broken the build because the MSYS2 people patche does not apply:
> 
> patching file nt/cmdproxy.c
> Hunk #1 succeeded at 38 (offset 3 lines).
> patching file src/sysdep.c
> Hunk #1 succeeded at 2985 (offset 292 lines).
> Hunk #2 FAILED at 2704.
> 1 out of 2 hunks FAILED -- saving rejects to file src/sysdep.c.rej
> Error: Failure patching nt/cmdproxy.c
> 
> where
> 
> $ cat emacs-master/src/sysdep.c.rej
> --- src/sysdep.c.orig   2022-06-23 20:59:02.252891700 +0200
> +++ src/sysdep.c        2022-06-23 21:05:31.998322600 +0200
> @@ -2704,7 +2706,11 @@
>    if (err | (ADDRESS_SANITIZER
>              ? fflush (stderr) != 0 || ferror (stderr)
>              : close_stream (stderr) != 0))
> +#ifndef _UCRT // XXX: close_stream(stderr) fails for some reason
>      _exit (EXIT_FAILURE);
> +#else
> +    ;
> +#endif
>  }
> 
> 
>  #ifndef DOS_NT
> 
> Maybe you now how this should be fixed...
> 
> TIA

For completeness, the cited patches apply to commit 22d031f644d38e385f422ffc4855385d9052659b but _not_ to commit f2f2e6a082a541c60eb25ad6d30707e111082811 (whose parent is 22d031...)



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Emacs master build broken [MSYS2/UCRT64]
  2023-08-18 21:35 Emacs master build broken [MSYS2/UCRT64] Angelo Graziosi
  2023-08-18 21:54 ` Angelo Graziosi
@ 2023-08-19  6:06 ` Eli Zaretskii
  2023-08-19  7:44   ` Angelo Graziosi
  1 sibling, 1 reply; 4+ messages in thread
From: Eli Zaretskii @ 2023-08-19  6:06 UTC (permalink / raw)
  To: Angelo Graziosi; +Cc: emacs-devel

> Date: Fri, 18 Aug 2023 23:35:57 +0200 (CEST)
> From: Angelo Graziosi <angelo.g0@libero.it>
> 
> I have built master two day ago on MSYS2/UCRT64 using their patches (https://github.com/msys2/MINGW-packages/tree/master/mingw-w64-emacs).
> 
> Now a recent change (Po Lu f2f2e6a082a541 ?) to src/sysdep.c has broken the build because the MSYS2 people patche does not apply:

Please take this up with the MSYS2 folks: they should update their
patches.  We don't maintain their patches here, and cannot be
responsible for not breaking them as development of Emacs continues.



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Emacs master build broken [MSYS2/UCRT64]
  2023-08-19  6:06 ` Eli Zaretskii
@ 2023-08-19  7:44   ` Angelo Graziosi
  0 siblings, 0 replies; 4+ messages in thread
From: Angelo Graziosi @ 2023-08-19  7:44 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: emacs-devel


> Il 19/08/2023 08:06 CEST Eli Zaretskii ha scritto:
> 
>  
> > Date: Fri, 18 Aug 2023 23:35:57 +0200 (CEST)
> > From: Angelo Graziosi <angelo.g0@libero.it>
> > 
> > I have built master two day ago on MSYS2/UCRT64 using their patches (https://github.com/msys2/MINGW-packages/tree/master/mingw-w64-emacs).
> > 
> > Now a recent change (Po Lu f2f2e6a082a541 ?) to src/sysdep.c has broken the build because the MSYS2 people patche does not apply:
> 
> Please take this up with the MSYS2 folks: they should update their
> patches.  We don't maintain their patches here, and cannot be
> responsible for not breaking them as development of Emacs continues.

I corrected those patches and informed MSYS2 people. Thanks in any case...



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-08-19  7:44 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-18 21:35 Emacs master build broken [MSYS2/UCRT64] Angelo Graziosi
2023-08-18 21:54 ` Angelo Graziosi
2023-08-19  6:06 ` Eli Zaretskii
2023-08-19  7:44   ` Angelo Graziosi

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.