unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Will Tower <william.emerson.tower@gmail.com>
Cc: 54491@debbugs.gnu.org
Subject: bug#54491: 27.2; left-margin variable breaks autoload generation
Date: Mon, 21 Mar 2022 14:43:18 +0200	[thread overview]
Message-ID: <83v8w7eas9.fsf@gnu.org> (raw)
In-Reply-To: <87a6djhgjv.fsf@gmail.com> (message from Will Tower on Mon, 21 Mar 2022 04:05:52 -0400)

> From: Will Tower <william.emerson.tower@gmail.com>
> Date: Mon, 21 Mar 2022 04:05:52 -0400
> 
> Steps to reproduce (arch linux, GNU Emacs 27.2 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.33, cairo version 1.17.4) of 2022-03-11)
> 1. Delete all installed versions of magit (our test package)
> 	a. for me this looks like calling M-x package-delete return magit until I can't delete anything else
> 2. from command line, emacs -Q
> 3. yank reproduction config file into a scratch buffer of clean emacs instance:
> 
> (custom-set-variables
>  '(left-margin 8)
> )
> (setq byte-compile-warnings '(cl-functions))
> 
> ;;Initialize package sources
> (require 'package)
> (setq package-archives
> 	'(
> 		("melpa" . "https://melpa.org/packages/")
> 		("org" . "https://orgmode.org/elpa/")
> 		("elpa" . "https://elpa.gnu.org/packages/")
> 	)
> )
> 
> 4. M-x eval-buffer return
> 5. M-x package-install return magit
> 6. Notice a bunch of a compiler warnings and errors, I usually receive something about autoloads 105
> 7. Examine magit-autoloads.el file
> 8. Notice that some of the generated autoloads have mismatched delimiters, (" appearing on the second line of an autoload, in a region that is supposed to be a comment is fairly common, like

I have hard time imagining how display margins, which are purely a
display feature, could ever affect the results of byte compilation or
any other processing of Lisp programs.

Could you perhaps show a simpler reproduction recipe, one that
wouldn't require installing 3rd-part packages from the various ELPAs?
We use autoloading in many Lisp files in the core, so if the problem
is real and general enough, any Lisp file that uses autoloading should
do, right?

Thanks.





  reply	other threads:[~2022-03-21 12:43 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-21  8:05 bug#54491: 27.2; left-margin variable breaks autoload generation Will Tower
2022-03-21 12:43 ` Eli Zaretskii [this message]
2022-03-21 13:48   ` Andreas Schwab
2022-03-21 14:49     ` Eli Zaretskii
2022-03-21 15:19 ` Lars Ingebrigtsen
2022-03-21 17:27   ` Eli Zaretskii
2022-03-21 18:17     ` Lars Ingebrigtsen
2022-06-12 14:33 ` chris

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83v8w7eas9.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=54491@debbugs.gnu.org \
    --cc=william.emerson.tower@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).