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