unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Glenn Morris <rgm@gnu.org>
To: Michael Olson <mwolson@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: Please fix before the release of 23.1
Date: Sat, 09 Aug 2008 20:55:38 -0400	[thread overview]
Message-ID: <mkiqu9g039.fsf@fencepost.gnu.org> (raw)
In-Reply-To: <87ljzj3oad.fsf_-_@grepfind.mwolson.org> (Michael Olson's message of "Tue, 29 Jul 2008 22:59:38 -0700")

Michael Olson wrote:

> This is a pretty serious regression, so I want to bring it to the
> attention of the list again, in hopes it gets fixed before the release
> of 23.1.  I think the problem is in the byte-compiler.

If this were true, we wouldn't be able to build Emacs, since there are
numerous instances of such constructs in the source. Since we can, the
problem either does not exist, or cannot be as general as you suggest.
So please try to investigate further.

>> This is a bug of emacs-snapshot, not semantic.  The above compilation
>> error is triggered by the following kind of statement:
>>
>> (eval-when-compile
>>    (require 'something)
>>    (require 'something-else))
>>
>> The problem goes away when the lines are changed to:
>>
>> (eval-when-compile (require 'something))
>> (eval-when-compile (require 'something-else))
>>
>> This is a regression and needs to be fixed in upstream Emacs.




  reply	other threads:[~2008-08-10  0:55 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <877ibp4hh3.fsf@vorlon.ganneff.de>
2008-07-13 22:21 ` Bug#490722: semantic: invalid read syntax Michael Olson
2008-07-30  5:59   ` Please fix before the release of 23.1 (was: Bug#490722: semantic: invalid read syntax) Michael Olson
2008-08-10  0:55     ` Glenn Morris [this message]
2008-08-10  9:15       ` Please fix before the release of 23.1 Ulrich Mueller
2008-08-10  9:48         ` Ulrich Mueller
2008-08-10 22:15           ` Chong Yidong
2008-08-10 22:18             ` Chong Yidong
2008-08-11  7:18             ` bug#636: " Ulrich Mueller
2008-08-11  8:47               ` Alan Mackenzie
2008-08-11  8:54                 ` Andreas Schwab
2008-08-11 18:50                   ` Alan Mackenzie
2008-08-11 15:26               ` Michael Olson
2008-08-11 16:30               ` Chong Yidong
2008-08-11 19:01                 ` Alan Mackenzie
2008-08-11 21:42                   ` Chong Yidong
2008-08-13  9:24                     ` Alan Mackenzie
2008-08-11 22:00               ` Johan Bockgård
2008-09-05 14:54           ` Stefan Monnier
2008-08-10 18:51         ` Glenn Morris

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=mkiqu9g039.fsf@fencepost.gnu.org \
    --to=rgm@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=mwolson@gnu.org \
    /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).