all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@IRO.UMontreal.CA>
To: David Engster <deng@randomsample.de>
Cc: "Eric M. Ludlam" <zappo@gnu.org>,
	Stephen Leake <stephen_leake@stephe-leake.org>,
	emacs-devel@gnu.org
Subject: Re: Noisy byte compilation on master
Date: Mon, 16 Feb 2015 18:14:47 -0500	[thread overview]
Message-ID: <jwvvbj11mty.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <87bnktpnns.fsf@engster.org> (David Engster's message of "Mon, 16 Feb 2015 22:11:19 +0100")

> The next problem are those "Obsolete name arg 'foo' to constructor"
> messages.  I cannot simply omit the names as this won't work with older
> EIEIO.

Indeed, you'd have to use `make-instance'.
Or you could use a macro which either drops the name argument or passes
it depending on the version of EIEIO with which it's compiled.

>   (require 'ede/proj-elisp)
>   (child-of-class-p 'ede-proj-target-elisp 'ede-target)
> used to work fine, but now `child-of-class-p' has those cl-check-types
> in it.  Is that on purpose?

Looks like a bug.  It's been changed to accept class arguments (rather
than class names), but it should also accept class names, as before.

>>> Also, if I understand you correctly, this would mean that
>>> core packages cannot depend on CEDET.
>> Indeed, that'd be the downside.
> A pretty major one, I would say.

Agreed.

> That's not what I mean.

I don't understand, then.

> For instance, there's the requirement that changes have to
> appear under the date they entered Emacs trunk,

Obviously, you won't be affected by this requirement any more once Emacs
starts generating the ChangeLogs mechanically.

> and that you should not have several entries from the same author for
> the same day.

Not sure where you ot that idea, but we don't have such a requirement.
I even often setup such multiple-entries-from-the-guy-same-day *by
hand*, when I feel like it helps structure the ChangeLog.

> Then there's the problem that the Changelog will also contain changes
> from files that are only upstream, which you'll have to delete.

How would that affect you, since "we" (i.e. Paul) write the scripts that
(will) generate the ChangeLogs (IIUC we'll generate a single ChangeLog
for the whole Emacs tree)?

> And on top of that, CEDET's files are scattered across the Emacs
> codebase, affecting many different Changelogs: not only that for
> 'lisp/cedet', but also 'lisp/emacs-lisp', 'admin', 'etc', 'doc', and
> there's even one for 'test'!

But once we start generating the ChangeLogs mechanically you won't need
to care any more (maybe we'll need to care if we want to generate
equivalent ChangeLogs, but so far the intention has been to do
something simpler).

> All of this is a *huge* pain,

I understand it's annoying, but I don't understand why you think that us
moving to auto-generated ChangeLogs won't solve those problems for you.


        Stefan



  reply	other threads:[~2015-02-16 23:14 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-01 16:06 Noisy byte compilation on master Eli Zaretskii
2015-02-02 16:31 ` Stefan Monnier
2015-02-02 16:35   ` Eli Zaretskii
2015-02-02 18:01   ` Stephen Leake
2015-02-02 23:23     ` Stefan Monnier
2015-02-03  7:03       ` David Engster
2015-02-04 15:45         ` Stefan Monnier
2015-02-15 15:58           ` David Engster
2015-02-16  2:56             ` Stefan Monnier
2015-02-16 21:11               ` David Engster
2015-02-16 23:14                 ` Stefan Monnier [this message]
2015-02-17  6:31                   ` David Engster
2015-02-17 23:06                     ` Stefan Monnier
2015-02-18 17:53                       ` David Engster
2015-02-18 19:35                         ` Stefan Monnier
2015-02-18 19:51                           ` David Engster
2015-02-18 22:44                             ` Stefan Monnier

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

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

  git send-email \
    --in-reply-to=jwvvbj11mty.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=deng@randomsample.de \
    --cc=emacs-devel@gnu.org \
    --cc=stephen_leake@stephe-leake.org \
    --cc=zappo@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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.