bug#44481: 28.0.50; Byte compilation doesn't work with --no-build-details

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

bug#44481: 28.0.50; Byte compilation doesn't work with --no-build-details

Philipp Stephani

When byte-compiling the Emacs sources with --no-build-details, compiling
any file that loads TRAMP fails:

$ rm lisp/eshell/em-tramp.elc && make BYTE_COMPILE_EXTRA_FLAGS=--no-build-details
[…]
  ELC      eshell/em-tramp.elc

In toplevel form:
eshell/em-tramp.el:34:13: Error: Wrong type argument: stringp, nil
[…]

This is likely because TRAMP doesn't cope with a nil `system-name'.  I
haven't checked all the details, but there are a few occasions in
tramp.el where it assumes that `system-name' is a string.  That
assumption was once valid, but with --no-build-details, it's nil.

First, we should either document that `system-name' can be nil, or
instead of using nil, inject a dummy system name such as
"unknown.invalid" (".invalid" being a reserved TLD).

Depending on the outcome of that, TRAMP should be adapted to cope with
such an unknown system name.


In GNU Emacs 28.0.50 (build 116, x86_64-pc-linux-gnu, GTK+ Version 3.24.20, cairo version 1.16.0)
Repository revision: 527413fb2ff8c073d89ee2d22d38a67c74678b27
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12008000
System Description: Debian GNU/Linux rodete

Configured using:
 'configure --disable-build-details --enable-gcc-warnings=warn-only
 --enable-gtk-deprecation-warnings --without-pop --with-mailutils
 --enable-checking=all --enable-check-lisp-object-type --with-modules
 'CFLAGS=-O0 -ggdb3''

Configured features:
XPM JPEG TIFF GIF PNG CAIRO SOUND DBUS GSETTINGS GLIB NOTIFY INOTIFY
LIBSELINUX GNUTLS FREETYPE HARFBUZZ ZLIB TOOLKIT_SCROLL_BARS GTK3 X11
XDBE XIM MODULES THREADS LIBSYSTEMD JSON PDUMPER

Important settings:
  value of $LANG: en_US.utf8
  value of $XMODIFIERS: @im=ibus
  locale-coding-system: utf-8-unix

Major mode: Lisp Interaction

Minor modes in effect:
  tooltip-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message rmc dired dired-loaddefs rfc822
mml easymenu mml-sec epa epg epg-config gnus-util rmail rmail-loaddefs
time-date mm-decode mm-bodies mm-encode mail-parse rfc2231 mailabbrev
gmm-utils mailheader sendmail rfc2047 rfc2045 ietf-drums mm-util
mail-prsvr mail-utils phst skeleton derived edmacro kmacro pcase ffap
thingatpt url url-proxy url-privacy url-expand url-methods url-history
url-cookie url-domsuf url-util url-parse auth-source cl-seq eieio
eieio-core cl-macs eieio-loaddefs password-cache json map url-vars
mailcap rx gnutls puny dbus xml subr-x seq byte-opt gv bytecomp
byte-compile cconv compile text-property-search comint ansi-color ring
cl-loaddefs cl-lib tooltip eldoc electric uniquify ediff-hook vc-hooks
lisp-float-type mwheel term/x-win x-win term/common-win x-dnd tool-bar
dnd fontset image regexp-opt fringe tabulated-list replace newcomment
text-mode elisp-mode lisp-mode prog-mode register page tab-bar menu-bar
rfn-eshadow isearch timer select scroll-bar mouse jit-lock font-lock
syntax facemenu font-core term/tty-colors frame minibuffer cl-generic
cham georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao
korean japanese eucjp-ms cp51932 hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese composite charscript charprop
case-table epa-hook jka-cmpr-hook help simple abbrev obarray
cl-preloaded nadvice button loaddefs faces cus-face macroexp files
window text-properties overlay sha1 md5 base64 format env code-pages
mule custom widget hashtable-print-readable backquote threads dbusbind
inotify dynamic-setting system-font-setting font-render-setting cairo
move-toolbar gtk x-toolkit x multi-tty make-network-process emacs)

Memory information:
((conses 16 69348 6897)
 (symbols 48 8639 1)
 (strings 32 24444 1317)
 (string-bytes 1 791193)
 (vectors 16 13890)
 (vector-slots 8 188852 5440)
 (floats 8 26 30)
 (intervals 56 212 0)
 (buffers 992 11))

--
Google Germany GmbH
Erika-Mann-Straße 33
80636 München

Geschäftsführer: Paul Manicle, Halimah DeLaine Prado
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg

Diese E-Mail ist vertraulich.  Falls Sie diese fälschlicherweise erhalten haben
sollten, leiten Sie diese bitte nicht an jemand anderes weiter, löschen Sie
alle Kopien und Anhänge davon und lassen Sie mich bitte wissen, dass die E-Mail
an die falsche Person gesendet wurde.

This e-mail is confidential.  If you received this communication by mistake,
please don’t forward it to anyone else, please erase all copies and
attachments, and please let me know that it has gone to the wrong person.



Reply | Threaded
Open this post in threaded view
|

bug#44481: 28.0.50; Byte compilation doesn't work with --no-build-details

Glenn Morris-3

See also https://debbugs.gnu.org/34448 .

I would guess Tramp isn't the only thing expecting system-name to be a string.



Reply | Threaded
Open this post in threaded view
|

bug#44481: 28.0.50; Byte compilation doesn't work with --no-build-details

Michael Albinus
Glenn Morris <[hidden email]> writes:

> See also https://debbugs.gnu.org/34448 .
>
> I would guess Tramp isn't the only thing expecting system-name to be a string.

Perhaps system-name returns better "localhost" but nil? Or at least the
empty string, at build time with --no-build-details?

Best regards, Michael.



Reply | Threaded
Open this post in threaded view
|

bug#44481: 28.0.50; Byte compilation doesn't work with --no-build-details

Michael Albinus
Michael Albinus <[hidden email]> writes:

> Since Tramp must also work for older Emacs versions, it needs an own fix
> anyway.
>
> Philipp, does the appended patch work for you?

Ping.

Best regards, Michael.



Reply | Threaded
Open this post in threaded view
|

bug#44481: 28.0.50; Byte compilation doesn't work with --no-build-details

Michael Albinus
Michael Albinus <[hidden email]> writes:

>> Since Tramp must also work for older Emacs versions, it needs an own fix
>> anyway.
>>
>> Philipp, does the appended patch work for you?
>
> Ping.

It's a pity that you don't react. Closing the bug, it is fixed for Tramp.

Best regards, Michael.



Reply | Threaded
Open this post in threaded view
|

bug#44481: 28.0.50; Byte compilation doesn't work with --no-build-details

Philipp Stephani
Am Sa., 21. Nov. 2020 um 16:09 Uhr schrieb Michael Albinus
<[hidden email]>:

>
> Michael Albinus <[hidden email]> writes:
>
> >> Since Tramp must also work for older Emacs versions, it needs an own fix
> >> anyway.
> >>
> >> Philipp, does the appended patch work for you?
> >
> > Ping.
>
> It's a pity that you don't react.

Sorry for the delay. In general, there's almost never a need to wait
for me to respond - the initial bug report is hopefully clear and
reproducible, so if your patch fixes the initial report, feel free to
immediately push it and close the report without waiting for me. If
something doesn't work, I'll reopen the bug.

> Closing the bug, it is fixed for Tramp.

Thanks.