emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* hard coded autoload file
@ 2013-04-25 17:53 Ivan Kanis
  2013-04-26  6:36 ` Bastien
  0 siblings, 1 reply; 3+ messages in thread
From: Ivan Kanis @ 2013-04-25 17:53 UTC (permalink / raw)
  To: org mode

Hello,

It seems each file has the name of the autoload file hard coded at the
end.

For example at the end org-element.el:

;; Local variables:
;; generated-autoload-file: "org-loaddefs.el"
;; End:

It sucks for me as I generate autoload with a different name. I made a
workaround so it's fine.

I think it would be better engineered no to have this stuff
repeated at the end of each file.

It would be better that it's specified somewhere in the autoconf.

If I work on this would the patch be accepted?
-- 
Not properly grounded, please bury computer.
    -- BOFH excuse #52

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

* Re: hard coded autoload file
  2013-04-25 17:53 hard coded autoload file Ivan Kanis
@ 2013-04-26  6:36 ` Bastien
  2013-04-26  7:55   ` Ivan Kanis
  0 siblings, 1 reply; 3+ messages in thread
From: Bastien @ 2013-04-26  6:36 UTC (permalink / raw)
  To: Ivan Kanis; +Cc: org mode

Hi Ivan,

Ivan Kanis <banana@kanis.fr> writes:

> It seems each file has the name of the autoload file hard coded at the
> end.
>
> For example at the end org-element.el:
>
> ;; Local variables:
> ;; generated-autoload-file: "org-loaddefs.el"
> ;; End:
>
> It sucks for me as I generate autoload with a different name. I made a
> workaround so it's fine.

Why do you generate autoloads with a different name?

> I think it would be better engineered no to have this stuff
> repeated at the end of each file.

We follow the Calc exemple here: .el files for which autoloads need to
be in Emacs loaddefs.el don't have this local variable set, and files
for which autoloads are put into org-loaddefs.el do have this
variable.

This is for reducing the number of Org autoloads in loaddefs.el as it
speeds up Emacs startup time.

> It would be better that it's specified somewhere in the autoconf.

> If I work on this would the patch be accepted?

I can imagine some ad hoc function putting the autoloads were they
belong, but this would be ad hoc.  I guess you'll have first to
explain why the current setup is problematic.  

-- 
 Bastien

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

* Re: hard coded autoload file
  2013-04-26  6:36 ` Bastien
@ 2013-04-26  7:55   ` Ivan Kanis
  0 siblings, 0 replies; 3+ messages in thread
From: Ivan Kanis @ 2013-04-26  7:55 UTC (permalink / raw)
  To: Bastien; +Cc: org mode

April, 26 at 8:36 Bastien wrote:

>> It sucks for me as I generate autoload with a different name. I made a
>> workaround so it's fine.
>
> Why do you generate autoloads with a different name?

I do my own packaging. I started before ELPA and el-get existed. I was
dissatisfied with the Debian way.

Now I am still doing it to keep third-party packages in sync with my
three computers.

>> I think it would be better engineered no to have this stuff
>> repeated at the end of each file.
>
> We follow the Calc exemple here: .el files for which autoloads need to
> be in Emacs loaddefs.el don't have this local variable set, and files
> for which autoloads are put into org-loaddefs.el do have this
> variable.

OK, I didn't know that Emacs has two autoload files. It makes sense now.

> I can imagine some ad hoc function putting the autoloads were they
> belong, but this would be ad hoc.  I guess you'll have first to
> explain why the current setup is problematic.  

There is no need to waste anymore time on this. I have a workaround.
-- 
The power of accurate observation is commonly called cynicism by those
who have not got it.
    -- George Bernard Shaw

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

end of thread, other threads:[~2013-04-26  7:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-04-25 17:53 hard coded autoload file Ivan Kanis
2013-04-26  6:36 ` Bastien
2013-04-26  7:55   ` Ivan Kanis

Code repositories for project(s) associated with this inbox:

	https://git.savannah.gnu.org/cgit/emacs/org-mode.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).