[CEDET-devel] Re[1]: EDE related changes

[Eric M. L. <er...@si...>]

Those all sound like good ideas to me.  Please check it in.  For some
reason cedet-autogen for the Semantic package was adding the wisent
and bovine directories to my autoload file anyway so I hadn't worried
about it.

How do you generate the email w/ the changelog and patches?  I find
the process somewhat tedious when I do it myself by hand.  Is it
scriptable?  I'm tempted to add "ede-prepare-patch-email" or some such
if it is.

Eric

>>> David PONCE <dav...@wa...> seems to think that:
>Hi Eric,
>
>I submit you these changes to ede-proj-elisp.el and to some EDE project
>files in CEDET:
>
>- In ede-proj-elisp.el I added a new `autoload-dirs' slot to the
>  class `ede-proj-target-elisp-autoloads'.  So it is possible to
>  customize which directories to scan for autoloads in a project.
>
>  In the semantic project file I used that new feature to also scan
>  the "bovine" and "wisent" sub-directories for autoloads.  I
>  re-generated the cedet packages Makefile to take into account that
>  change.
>
>- In the EIEIO project file I added "speedbar" as an aux-packages to
>  the target "eieio" to ensure that eieio libraries will be
>  byte-compiled with the right version of speedbar (that is, the cedet
>  one).  Without that I got speedbar runtime errors on undefined
>  variable `speedbar-attached-frame' which was referenced in
>  byte-compiled code from the version of speedbar in standard
>  distribution of Emacs.
>
>Following is the change log, the patch is attached.  If you agree with
>those changes, I will commit them.
>
>What do you think?
>
>David
>
>2003-09-10  David Ponce  <da...@dp...>
>
>	* ede/ede-proj-elisp.el
>
>	(ede-proj-target-elisp-autoloads): Add autoload-dirs slot.
>	(ede-emacs-cedet-autogen-compiler): Use $(LOADDIRS) instead of "."
>	in compiler's commands.
>	(ede-proj-makefile-insert-variables): Method of class
>	ede-proj-target-elisp-autoloads creates the LOADDIRS variable
>	definition.
>
>	* ede/Makefile
>
>	Re-generated.
>
>	* eieio/Project.ede
>
>	(eieio target): Add speedbar to aux-packages to ensure that eieio
>	will be byte-compiled with the speedbar library in cedet.
>
>	* eieio/Makefile
>
>	Re-generated.
>
>	* semantic/Project.ede
>
>	(autoloads target): Add "bovine" and "wisent" sub-directories to
>	autoload-dirs.
>
>	* semantic/Makefile
>
>	Re-generated.
>
>	* speedbar/Makefile
>
>	Re-generated.
>	
>

-- 
          Eric Ludlam:                 za...@gn..., er...@si...
   Home: http://www.ludlam.net            Siege: www.siege-engine.com
Emacs: http://cedet.sourceforge.net               GNU: www.gnu.org