From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxim Cournoyer Subject: bug#38309: Recent $EMACSLOADPATH changes crash gnome-session Date: Sun, 24 Nov 2019 12:45:51 +0900 Message-ID: <87d0diuec0.fsf@gmail.com> References: <87y2w8vzeq.fsf@lassieur.org> <8736egaw7g.fsf@gmail.com> <87k17rvmfw.fsf@gmail.com> <87eexy1nah.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:470:142:3::10]:55395) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iYirc-0004iq-Jx for bug-guix@gnu.org; Sat, 23 Nov 2019 22:47:09 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iYirb-00073Q-59 for bug-guix@gnu.org; Sat, 23 Nov 2019 22:47:08 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:49227) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1iYirb-00073K-1i for bug-guix@gnu.org; Sat, 23 Nov 2019 22:47:07 -0500 Sender: "Debbugs-submit" Resent-Message-ID: In-Reply-To: <87eexy1nah.fsf@gnu.org> ("Ludovic \=\?utf-8\?Q\?Court\=C3\=A8s\=22'\?\= \=\?utf-8\?Q\?s\?\= message of "Sat, 23 Nov 2019 19:05:10 +0100") List-Id: Bug reports for GNU Guix List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-guix-bounces+gcggb-bug-guix=m.gmane.org@gnu.org Sender: "bug-Guix" To: Ludovic =?UTF-8?Q?Court=C3=A8s?= Cc: 38309@debbugs.gnu.org, a@ajgrf.com Hello, Ludovic Court=C3=A8s writes: > Hi Maxim, > > Maxim Cournoyer skribis: > >> Really sorry about this ugly regression :-/. > > Should we revert 47b3b4c2aa49e21f4cc32c97ff7bbbd069bb849c so we can > address this without pressure in the meantime? > > I think I missed the discussions around this patch, but we should get > Alex Kost into the loop (Alex did the initial work in that area, if I=E2= =80=99m > not mistaken.) > > Thanks, > Ludo=E2=80=99. There would be a couple more commits to include in the revert to undo the changes (one to the build system, others to adapt the renaming of the emacs-set-load-path phase for some packages: --8<---------------cut here---------------start------------->8--- 63edbb65e4 * origin/master gnu: emacs-protobuf-mode: Rename the set-emacs-l= oad-path phase. ffb2316548 * gnu: emacs-erlang: Rename the set-emacs-load-path phase. ed94123667 * gnu: emacs-pdf-tools: Adapt phase name. 418febb54f * gnu: emacs-scel: Fix build. 1bb39982f1 * gnu: emacs-realgud: Fix build. c51d4c7746 * gnu: emacs-pdf-tools: Fix build. b44357d02a * gnu: emacs-forge: Fix build. 47b3b4c2aa * gnu: emacs: Adapt the autoloads auxiliary code to use EMACSLOA= DPATH. e1d31e6457 * build-system: emacs: Simplify the SET-EMACS-LOAD-PATH phase. 215a45d9b8 * gnu: emacs: Locate Elisp libraries via EMACSLOADPATH. --8<---------------cut here---------------end--------------->8--- The bug in questino seems to have to do with a regression in recent versions PCRE, that trigger a crash on large environment variables as reported in [0]. There's a new version of PCRE2 (10.34) available; but I'm unsure if it addresses this particular problem: https://pcre.org/changelog.txt. If it doesn't we should report the problem to them. This could happen with any other environment variable than EMACSLOADPATH; it's just a matter of having an environment variable grow sufficiently long to trigger it. I'm testing a patch that may workaround this problem successfully right now (that would reduce the length of EMACSLOADPATH) I'll send an incomplete patch shortly for discussing the idea or testing. In case it's not clear; I'd rather workaround the issue or have it fixed at its source, ideally. WDYT? Maxim [0] https://bugzilla.redhat.com/show_bug.cgi?id=3D1430103