GNU bug report logs - #11915
libraries that don't provide a feature or the wrong one

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 11915 in the body.
You can then email your comments to 11915 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jonas Bernoulli <jonas <at> bernoul.li>
To: bug-gnu-emacs <at> gnu.org
Subject: libraries that don't provide a feature or the wrong one
Date: Wed, 11 Jul 2012 22:52:47 +0200

* in the emacs-24 branch:

bug-reference.el and map-ynp.el do not provide a feature.
misearch.el provides `multi-isearch' instead of `misearch'.

userlock.el, patcomp.el, forms-d2.el and forms-pass.el also
don't provide a feature but it seems this intentional.
However I think they then should not appear in `packages-builtins'.
The last two could possibly be added to the forms package.

* in the elpa branch:

markchars.el does not provide a feature.

Message #8 received at 11915 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Jonas Bernoulli <jonas <at> bernoul.li>
Cc: 11915 <at> debbugs.gnu.org
Subject: Re: bug#11915: libraries that don't provide a feature or the wrong one
Date: Thu, 12 Jul 2012 10:33:00 -0400

> * in the emacs-24 branch:
> bug-reference.el and map-ynp.el do not provide a feature.

Looks like a bug in bug-reference.el.
For map-ynp.el, I think it's normal because that file is preloaded.

> misearch.el provides `multi-isearch' instead of `misearch'.

That should be fixed, and is probably a left-over from adapting the
filename to 8+3 limitations.

> userlock.el, patcomp.el, forms-d2.el and forms-pass.el also
> don't provide a feature but it seems this intentional.

userlock should arguably be preloaded.  patcomp.el is obsolete and not
worth fixing, and the other two indeed aren't really Lisp packages so
don't need a `provide'.

> However I think they then should not appear in `packages-builtins'.

You might be right (at least the etc/forms/*.el files shouldn't be
there), tho I'm not sure how important this is.

> * in the elpa branch:
> markchars.el does not provide a feature.

That's an oversight, yes, thank you,


        Stefan

Message #11 received at 11915 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: jonas <at> bernoul.li, 11915 <at> debbugs.gnu.org
Subject: Re: bug#11915: libraries that don't provide a feature or the wrong one
Date: Thu, 12 Jul 2012 18:22:48 +0300

> From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
> Date: Thu, 12 Jul 2012 10:33:00 -0400
> Cc: 11915 <at> debbugs.gnu.org
> 
> > misearch.el provides `multi-isearch' instead of `misearch'.
> 
> That should be fixed, and is probably a left-over from adapting the
> filename to 8+3 limitations.

AFAIK, there was never such a limitation in Emacs sources.

Message #16 received at 11915-done <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Jonas Bernoulli <jonas <at> bernoul.li>
Cc: 11915-done <at> debbugs.gnu.org
Subject: Re: bug#11915: libraries that don't provide a feature or the wrong one
Date: Wed, 18 Jul 2012 07:44:38 -0400

>>> * in the emacs-24 branch:
>>> bug-reference.el and map-ynp.el do not provide a feature.
>> Looks like a bug in bug-reference.el.
> This bug does not appear to be fixed yet.

Should be fixed now.

>> For map-ynp.el, I think it's normal because that file is preloaded.

I added:

> ;; Package: emacs
> to it's header.

>>> misearch.el provides `multi-isearch' instead of `misearch'.
>> That should be fixed,
> This does not appear to be fixed yet.

Should be fixed now.

>>> userlock.el, patcomp.el, forms-d2.el and forms-pass.el also
>>> don't provide a feature but it seems this intentional.
>> userlock should arguably be preloaded.
> Then it should also not be an "independent package" but be part of the
> "emacs" package just like `map-ynp.el'.

Done.

>> patcomp.el is obsolete and not worth fixing,
> Shouldn't it be moved to lisp/obsolete then (or since it was only used by
> maintainers removed right away)?  Also it should not appear in
> `package--builtins'.

It's already been moved to obsolete in trunk a while ago.

>>> However I think they then should not appear in `packages-builtins'.
>> You might be right (at least the etc/forms/*.el files shouldn't be
>> there), tho I'm not sure how important this is.
> It would be easy to make them part of the `forms' package.

Oh, I see, you're looking at the 24.1 code, but in the trunk these files
have been moved to "etc/forms" so they should not be a problem any more.

>>> * in the elpa branch:
>>> markchars.el does not provide a feature.
>> That's an oversight, yes, thank you,
> This does not appear to be fixed yet.

Should be fixed now.


        Stefan

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.