Mats Lidell writes:
But it is at the same time a real hack and is trying to solve the
problem with auto-autoloads.el containing other stuff than
autoloads. This is also probably only needed for auctex which might
suggest other ways to solve it.
I've already heard from one reviewer worrying about the hacky aspects.
But we don't actually have a patch to review. The few lines in
XEmacs.rules are not a big deal, and figuring out a better way to do
what's necessary for AUCTeX requires knowing what gets done.
The worst that can happen IMO is that Norbert will have to set the set
of xemacs-packages to build to "everything but AUCTeX" until problems
get worked out. Somewhat more distressing to you but quite convenient
for him, Norbert could just revert the whole commit. :-) So I don't
think it's anything to worry about too much.
My opinion (as a single reviewer, I haven't consulted with anybody) is
that you should just go and ask Norbert when it would be convenient
for him for you to commit what you've got (he'll probably say "now"
:-), and we can go from there.
Maybe the documentation should be "Used for special
auto-autoloads
generation by auctex"?
It would be better to be more specific. If you can't easily describe
it in words, something like "See AUCTeX's XEmacs Makefile ca. line
XXX" is sufficient.
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-beta