all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Drew Adams" <drew.adams@oracle.com>
To: "'Chong Yidong'" <cyd@stupidchicken.com>
Cc: emacs-devel@gnu.org
Subject: RE: Merging Finder into package mechanism
Date: Sun, 31 Oct 2010 15:29:07 -0700	[thread overview]
Message-ID: <1DFC8CFFD80647C086EAC377F7AAA837@us.oracle.com> (raw)
In-Reply-To: <87eib72orn.fsf@stupidchicken.com>

> Which package is this?

It doesn't matter.  It's not my library, if that's what you're worried about.
It uses `finder-package-info' only in this way:

(cadr (assoc (format "%s.el" (symbol-name feature))
             finder-package-info))

That's all.  So please say how to do the equivalent using package.el or
whatever.  That should be simple since you say the same info is still available
(somewhere).

> > If you truly "merged" those two variables, why couldn't you provide
> > compatibility - via defalias or something?
> 
> Preferably not.  If almost no one is using the old data, it's 
> better to make a clean break than to lug around two copies of
> the same data.

Since when does `def(var)alias' make a copy?  I didn't say anything about
creating copies.

In any case, where is the remaining copy of the data that you still lug around?
What guidance do you provide to update code that uses `finder-package-info' so
that it can DTRT after Emacs 23?

If the data is still available then it should be simple to update the code above
for Emacs 24+.  Please indicate how.

And how do you know that "almost no one is using the old data"?  Did you
survey/poll/search or just divine this?  You incorrectly assumed/claimed that
"third-party code is unlikely to make use of it."  Now you suppose that _almost_
no one is using it.  Based on what?

I came across one library that uses it, just by accident.

Googling indicates others, including `ee-finder.el' by Juri, which does this:

(mapcar
 (lambda (package)
   (mapcar
    (lambda (field-name)
      (cond ((eq field-name 'file)     (nth 0 package))
            ((eq field-name 'synopsis) (nth 1 package))
            ((eq field-name 'keywords) (nth 2 package))))
    field-names))
 finder-package-info)

And a post from Tim Cross entitled "Warning regarding current emacs 24" says
that Emacsspeak needs `finder-package-info' and that your change "breaks
emacspeak-finder.el, which in turn breaks the emacspeak config step."
 
http://www.cs.vassar.edu/~priestdo/emacspeak/2010/msg00149.html

No doubt all such 3rd-party coders can eventually make adjustments for your
"clean break".  But please document what's involved so they know what to do and
can begin to do it.

> > Also, I do not, in any case, see any `package--builtins-base'
> > in Emacs 24.
> 
> The system has been tweaked a couple of times since I posted 
> that.

Surely some person(s) did the tweaking, and so should update NEWS accordingly.
So far, we have user-visible changes with no explanation in the user-visible
NEWS.

> I'll add a placeholder NEWS entry as a reminder, but
> there will likely be more changes before Emacs 24 is released.

Please keep NEWS updated correctly wrt this breakage.  NEWS is for users.  If
you can update change logs for developers then you can also update NEWS for
users.  You don't use "placeholders" in change logs, do you?




  reply	other threads:[~2010-10-31 22:29 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-28 20:08 Merging Finder into package mechanism Chong Yidong
2010-08-28 20:34 ` Drew Adams
2010-08-28 23:38 ` Juri Linkov
2010-08-29  1:44   ` Chong Yidong
2010-08-31  0:27     ` Juri Linkov
2010-08-31  1:29       ` Chong Yidong
2010-08-31 22:39         ` Juri Linkov
2010-09-01  6:54           ` Stefan Monnier
2010-09-01 22:05             ` Juri Linkov
2010-09-02  7:54               ` Stefan Monnier
2010-09-03 18:30           ` Chong Yidong
2010-09-03 20:45             ` Juri Linkov
2010-08-30  0:46 ` Chong Yidong
2010-08-31  0:27   ` Juri Linkov
2010-08-31  1:42     ` Chong Yidong
2010-08-31 22:43       ` Juri Linkov
2010-10-30 20:59 ` Drew Adams
2010-10-31  0:23   ` Chong Yidong
2010-10-31 22:29     ` Drew Adams [this message]
2010-11-01 16:11       ` Chong Yidong
2010-11-01 17:51         ` Drew Adams
2010-11-01 18:58           ` Eli Zaretskii
2010-11-01 22:03             ` Drew Adams
2010-11-02  3:54               ` Eli Zaretskii
2010-11-01 21:31           ` Chong Yidong
2010-11-01 22:03             ` Drew Adams

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=1DFC8CFFD80647C086EAC377F7AAA837@us.oracle.com \
    --to=drew.adams@oracle.com \
    --cc=cyd@stupidchicken.com \
    --cc=emacs-devel@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.