GNU bug report logs - #70842
30.0.50; Native compilation and gmake in src

Previous Next

Package: emacs;

Reported by: Gerd Möllmann <gerd.moellmann <at> gmail.com>

Date: Thu, 9 May 2024 06:00:02 UTC

Severity: normal

Found in version 30.0.50

Done: Andrea Corallo <acorallo <at> gnu.org>

To reply to this bug, email your comments to 70842 AT debbugs.gnu.org.
There is no need to reopen the bug first.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#70842; Package emacs. (Thu, 09 May 2024 06:00:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Gerd Möllmann <gerd.moellmann <at> gmail.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Thu, 09 May 2024 06:00:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Gerd Möllmann <gerd.moellmann <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Cc: Andrea Corallo <acorallo <at> gnu.org>
Subject: 30.0.50; Native compilation and gmake in src
Date: Thu, 09 May 2024 07:58:49 +0200
(CC'd to Andrea.)

1. Start from a clean build with native compilation

2. Modify a .c file in src to trigger a recompilation

3. gmake in src

=> error "trying to dump non fixed-up eln file" when dumping
bootstrap-emacs.pdmp. Always.

I find this very inconvenient, esp. since this never happens without
native compilation. Only thing that works currently is to make in the
toplevel directory.

The reason for this is src/Makefile uses ELN_DESTDIR and BIN_DESTDIR in
rules, but does not contain definitions for them. In a toplevel make,
both are defined on the command line when invoking "gmake -C src
ELN_DESTDIR=...".

When making in src, ELN_DESTDIR is therefore empty, and temacs is called
with with --eln-dest empty. Loadup.el then does its thing and calls
load--fixup-all-elns before dumping. That function decides to not set
native compilation units' file member to a cons, and "trying to dump non
fixed-up eln file" is then issued when dumping such a CU.

I don't know what the right fix for this is, maybe load--fixup-all-elns
should work differently. As a workaround, it apparently suffices to add
default values to src/Makefile like so:

ELN_DESTDIR=not-set
BIN_DESTDIR=not-set

In GNU Emacs 30.0.50 (build 1, aarch64-apple-darwin23.4.0, NS
 appkit-2487.50 Version 14.4.1 (Build 23E224)) of 2024-05-04 built on
 pro2.fritz.box
Repository revision: fdfadd6f467063dc83032b4e0dbe4e7b96690670
Repository branch: master
Windowing system distributor 'Apple', version 10.3.2487
System Description:  macOS 14.4.1

Configured using:
 'configure --cache-file
 /var/folders/1d/k_6t25f94sl83szqbf8gpkrh0000gn/T//config.cache.master
 --with-native-compilation CC=clang'




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#70842; Package emacs. (Sat, 25 May 2024 07:39:02 GMT) Full text and rfc822 format available.

Message #8 received at 70842 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Gerd Möllmann <gerd.moellmann <at> gmail.com>
Cc: acorallo <at> gnu.org, 70842 <at> debbugs.gnu.org
Subject: Re: bug#70842: 30.0.50; Native compilation and gmake in src
Date: Sat, 25 May 2024 10:37:58 +0300
Ping!  Andrea, if you agree with Gerd's proposal, would you please
install such a change?

> Cc: Andrea Corallo <acorallo <at> gnu.org>
> From: Gerd Möllmann <gerd.moellmann <at> gmail.com>
> Date: Thu, 09 May 2024 07:58:49 +0200
> 
> (CC'd to Andrea.)
> 
> 1. Start from a clean build with native compilation
> 
> 2. Modify a .c file in src to trigger a recompilation
> 
> 3. gmake in src
> 
> => error "trying to dump non fixed-up eln file" when dumping
> bootstrap-emacs.pdmp. Always.
> 
> I find this very inconvenient, esp. since this never happens without
> native compilation. Only thing that works currently is to make in the
> toplevel directory.
> 
> The reason for this is src/Makefile uses ELN_DESTDIR and BIN_DESTDIR in
> rules, but does not contain definitions for them. In a toplevel make,
> both are defined on the command line when invoking "gmake -C src
> ELN_DESTDIR=...".
> 
> When making in src, ELN_DESTDIR is therefore empty, and temacs is called
> with with --eln-dest empty. Loadup.el then does its thing and calls
> load--fixup-all-elns before dumping. That function decides to not set
> native compilation units' file member to a cons, and "trying to dump non
> fixed-up eln file" is then issued when dumping such a CU.
> 
> I don't know what the right fix for this is, maybe load--fixup-all-elns
> should work differently. As a workaround, it apparently suffices to add
> default values to src/Makefile like so:
> 
> ELN_DESTDIR=not-set
> BIN_DESTDIR=not-set
> 
> In GNU Emacs 30.0.50 (build 1, aarch64-apple-darwin23.4.0, NS
>  appkit-2487.50 Version 14.4.1 (Build 23E224)) of 2024-05-04 built on
>  pro2.fritz.box
> Repository revision: fdfadd6f467063dc83032b4e0dbe4e7b96690670
> Repository branch: master
> Windowing system distributor 'Apple', version 10.3.2487
> System Description:  macOS 14.4.1
> 
> Configured using:
>  'configure --cache-file
>  /var/folders/1d/k_6t25f94sl83szqbf8gpkrh0000gn/T//config.cache.master
>  --with-native-compilation CC=clang'
> 
> 
> 
> 




Reply sent to Andrea Corallo <acorallo <at> gnu.org>:
You have taken responsibility. (Sat, 25 May 2024 15:23:02 GMT) Full text and rfc822 format available.

Notification sent to Gerd Möllmann <gerd.moellmann <at> gmail.com>:
bug acknowledged by developer. (Sat, 25 May 2024 15:23:02 GMT) Full text and rfc822 format available.

Message #13 received at 70842-done <at> debbugs.gnu.org (full text, mbox):

From: Andrea Corallo <acorallo <at> gnu.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Gerd Möllmann <gerd.moellmann <at> gmail.com>,
 70842-done <at> debbugs.gnu.org
Subject: Re: bug#70842: 30.0.50; Native compilation and gmake in src
Date: Sat, 25 May 2024 11:22:08 -0400
Eli Zaretskii <eliz <at> gnu.org> writes:

> Ping!  Andrea, if you agree with Gerd's proposal, would you please
> install such a change?

Sorry missed this.

Installed and closing.

Thanks

  Andrea




This bug report was last modified 7 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.