unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#10396: 24.0.92; update-directory-autoloads now adds path element to filename in autoload definition
@ 2011-12-29 13:37 David Engster
  2011-12-30 11:42 ` Stefan Monnier
  0 siblings, 1 reply; 6+ messages in thread
From: David Engster @ 2011-12-29 13:37 UTC (permalink / raw)
  To: 10396

This might not be a bug, but I couldn't find anything in NEWS regarding
this issue, and it breaks my personal autoload setup.

Recipe:

* mkdir -p ~/foo/bar

* Create a file 'test.el' in foo/bar, containing

  ;;;###autoload
  (defun testthis ()
  )

* cd ~/foo and call

  emacs --batch --eval '(let ((generated-autoload-file "~/foo/loaddefs.el")) (update-directory-autoloads "~/foo/bar")))'

* The file ~/foo/loaddefs.el will have this autoload definition:

  (autoload 'testthis "bar/test" "\

  Using emacs v23.3, I only get

  (autoload 'testthis "test" "\


-David


In GNU Emacs 24.0.92.6 (i686-pc-linux-gnu, X toolkit, Xaw3d scroll bars)
 of 2011-12-28 on spaten
Windowing system distributor `The X.Org Foundation', version 11.0.11103000
configured using `configure  '--prefix=/opt/emacs' '--without-gsettings' '--without-gconf' '--without-dbus' '--with-x-toolkit=lucid''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: C
  value of $LC_MESSAGES: C
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_US.UTF-8
  value of $XMODIFIERS: nil
  locale-coding-system: nil
  default enable-multibyte-characters: t





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#10396: 24.0.92; update-directory-autoloads now adds path element to filename in autoload definition
  2011-12-29 13:37 bug#10396: 24.0.92; update-directory-autoloads now adds path element to filename in autoload definition David Engster
@ 2011-12-30 11:42 ` Stefan Monnier
  2011-12-31 10:53   ` David Engster
  0 siblings, 1 reply; 6+ messages in thread
From: Stefan Monnier @ 2011-12-30 11:42 UTC (permalink / raw)
  To: David Engster; +Cc: 10396

> This might not be a bug, but I couldn't find anything in NEWS regarding
> this issue, and it breaks my personal autoload setup.

It's a feature: since foo/bar is not in `load-path', autoload.el figures
that only "bar/test" will properly load the file.
So you just need to set `load-path' to include `foo/bar' like you
probably already do elsewhere.


        Stefan





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#10396: 24.0.92; update-directory-autoloads now adds path element to filename in autoload definition
  2011-12-30 11:42 ` Stefan Monnier
@ 2011-12-31 10:53   ` David Engster
  2012-01-27  8:39     ` Chong Yidong
  0 siblings, 1 reply; 6+ messages in thread
From: David Engster @ 2011-12-31 10:53 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: 10396

Stefan Monnier writes:
>> This might not be a bug, but I couldn't find anything in NEWS regarding
>> this issue, and it breaks my personal autoload setup.
>
> It's a feature: since foo/bar is not in `load-path', autoload.el figures
> that only "bar/test" will properly load the file.
> So you just need to set `load-path' to include `foo/bar' like you
> probably already do elsewhere.

But it's a new feature, right? Emacs 23.3 doesn't seem to do that.

If it's new, I think it should be mentioned in NEWS; it is an important
change for anyone creating Emacs packages, since I now have to make sure
that `load-path' during autoload-generation and during actual usage of
the package is consistent.

-David





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#10396: 24.0.92; update-directory-autoloads now adds path element to filename in autoload definition
  2011-12-31 10:53   ` David Engster
@ 2012-01-27  8:39     ` Chong Yidong
  2012-01-28 18:41       ` David Engster
  0 siblings, 1 reply; 6+ messages in thread
From: Chong Yidong @ 2012-01-27  8:39 UTC (permalink / raw)
  To: David Engster; +Cc: 10396

David Engster <deng@randomsample.de> writes:

> Stefan Monnier writes:
>>> This might not be a bug, but I couldn't find anything in NEWS regarding
>>> this issue, and it breaks my personal autoload setup.
>>
>> It's a feature: since foo/bar is not in `load-path', autoload.el figures
>> that only "bar/test" will properly load the file.
>> So you just need to set `load-path' to include `foo/bar' like you
>> probably already do elsewhere.
>
> But it's a new feature, right? Emacs 23.3 doesn't seem to do that.
>
> If it's new, I think it should be mentioned in NEWS; it is an important
> change for anyone creating Emacs packages, since I now have to make sure
> that `load-path' during autoload-generation and during actual usage of
> the package is consistent.

Could you suggest a NEWS entry?  That would be helpful, since I don't
understand how the current autoload behavior breaks your setup.





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#10396: 24.0.92; update-directory-autoloads now adds path element to filename in autoload definition
  2012-01-27  8:39     ` Chong Yidong
@ 2012-01-28 18:41       ` David Engster
  2012-02-26  9:51         ` Chong Yidong
  0 siblings, 1 reply; 6+ messages in thread
From: David Engster @ 2012-01-28 18:41 UTC (permalink / raw)
  To: Chong Yidong; +Cc: 10396

Chong Yidong writes:
> David Engster <deng@randomsample.de> writes:
>
>> Stefan Monnier writes:
>>>> This might not be a bug, but I couldn't find anything in NEWS regarding
>>>> this issue, and it breaks my personal autoload setup.
>>>
>>> It's a feature: since foo/bar is not in `load-path', autoload.el figures
>>> that only "bar/test" will properly load the file.
>>> So you just need to set `load-path' to include `foo/bar' like you
>>> probably already do elsewhere.
>>
>> But it's a new feature, right? Emacs 23.3 doesn't seem to do that.
>>
>> If it's new, I think it should be mentioned in NEWS; it is an important
>> change for anyone creating Emacs packages, since I now have to make sure
>> that `load-path' during autoload-generation and during actual usage of
>> the package is consistent.
>
> Could you suggest a NEWS entry?  That would be helpful, since I don't
> understand how the current autoload behavior breaks your setup.

Here's a suggestion:

When generating autoloads, `update-directory-autoloads' now longer
assumes that every inspected file will be in your `load-path', but will
now generate relative names according to your current `load-path' setting.

Stefan, please correct me if this is not a proper description of the new
behavior.  Also, I'm not sure if this new feature only affects
`update-directory-autoloads', but also other autoload functions.

-David





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#10396: 24.0.92; update-directory-autoloads now adds path element to filename in autoload definition
  2012-01-28 18:41       ` David Engster
@ 2012-02-26  9:51         ` Chong Yidong
  0 siblings, 0 replies; 6+ messages in thread
From: Chong Yidong @ 2012-02-26  9:51 UTC (permalink / raw)
  To: David Engster; +Cc: 10396

David Engster <deng@randomsample.de> writes:

> Here's a suggestion:
>
> When generating autoloads, `update-directory-autoloads' now longer
> assumes that every inspected file will be in your `load-path', but will
> now generate relative names according to your current `load-path' setting.

I went ahead and committed this NEWS entry.





^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2012-02-26  9:51 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-29 13:37 bug#10396: 24.0.92; update-directory-autoloads now adds path element to filename in autoload definition David Engster
2011-12-30 11:42 ` Stefan Monnier
2011-12-31 10:53   ` David Engster
2012-01-27  8:39     ` Chong Yidong
2012-01-28 18:41       ` David Engster
2012-02-26  9:51         ` Chong Yidong

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).