Thread: [CEDET-devel] A CEDET's common package
Brought to you by:
zappo
From: David P. <dav...@wa...> - 2003-09-22 14:01:54
Attachments:
common-new.tar.gz
|
Hi Eric, For now CEDET lacks a true CEDET's common package. Here is a proposal to supply one. First I created these new files (in the attached tarball): common.el This file just provide the `common-version' information for inversion. common-load.el Initialize common stuff. For now it just loads common autoloads from the generated common-loaddefs.el file. Project.ede The CEDET's common stuff project used to generate a Makefile. Then I made minor changes to the main Makefile and cedet.el to take into account the common package (see the patch at end). Also, it would be good to move sformat.el and working.el from semantic to the common package which is now mandatory. What do you think? Index: Makefile =================================================================== RCS file: /cvsroot/cedet/cedet/Makefile,v retrieving revision 1.2 diff -c -r1.2 Makefile *** Makefile 17 Sep 2003 08:54:34 -0000 1.2 --- Makefile 22 Sep 2003 13:49:28 -0000 *************** *** 29,34 **** --- 29,35 ---- ## The CEDET's packages installed CEDET_PACKAGES=\ + common \ ede \ speedbar \ eieio \ Index: common/cedet.el =================================================================== RCS file: /cvsroot/cedet/cedet/common/cedet.el,v retrieving revision 1.3 diff -c -r1.3 cedet.el *** common/cedet.el 17 Sep 2003 08:56:12 -0000 1.3 --- common/cedet.el 22 Sep 2003 13:49:28 -0000 *************** *** 82,87 **** --- 82,88 ---- (defconst cedet-packages '( ;;PACKAGE MIN-VERSION + (common "0.1" ) (cogre "0.4" ) (ede "1.0beta3" ) (eieio "0.18" ) |
From: Eric M. L. <er...@si...> - 2003-09-22 17:35:39
|
>>> David PONCE <dav...@wa...> seems to think that: >Hi Eric, > >For now CEDET lacks a true CEDET's common package. Here is a proposal >to supply one. > >First I created these new files (in the attached tarball): > >common.el > This file just provide the `common-version' information for > inversion. Hmmmm. The name "common" just seems too... I dunno, common? The name doesn't say anything to me. 'common' makes sense as an entity to a project, but `common-version' in the global namespace does not. `CEDET' would say something, but I'm not sure it says the right thing. >common-load.el > Initialize common stuff. For now it just loads common autoloads > from the generated common-loaddefs.el file. This makes sense, but I'm not sure having a file with the word `common' in the file name is good for the Emacs global namespace since it is specific to CEDET. Perhaps the `common' directory could just be called `cedet' and mark it as the location for miscellaneous bits not large enough to warrant a project directory, but still quite specific to CEDET. >Project.ede > The CEDET's common stuff project used to generate a Makefile. > >Then I made minor changes to the main Makefile and cedet.el to take >into account the common package (see the patch at end). > >Also, it would be good to move sformat.el and working.el from semantic >to the common package which is now mandatory. This is certainly a good idea. Thanks Eric -- 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 |
From: David P. <dav...@wa...> - 2003-09-22 17:59:25
|
[...] > This makes sense, but I'm not sure having a file with the word > `common' in the file name is good for the Emacs global namespace > since it is specific to CEDET. As you, I am not completely satisfied with the name "common". > Perhaps the `common' directory could just be called `cedet' and mark > it as the location for miscellaneous bits not large enough to warrant > a project directory, but still quite specific to CEDET. [...] I agree with that (I thought to it too). An advantage is that the library cedet.el already exists, and provide the cedet-version info. needed by inversion (IMO it makes sense that the cedet version serve as reference for common stuff). Is there a simple way to do that renaming in CVS? Or should we create the new cedet directory then add the content of the common directory into it, and finally remove the common directory from the repository. David |
From: Eric M. L. <er...@si...> - 2003-09-22 18:29:42
|
>>> David Ponce <dav...@wa...> seems to think that: >[...] > > This makes sense, but I'm not sure having a file with the word > > `common' in the file name is good for the Emacs global namespace > > since it is specific to CEDET. > >As you, I am not completely satisfied with the name "common". > > > Perhaps the `common' directory could just be called `cedet' and mark > > it as the location for miscellaneous bits not large enough to warrant > > a project directory, but still quite specific to CEDET. >[...] > >I agree with that (I thought to it too). An advantage is that the >library cedet.el already exists, and provide the cedet-version >info. needed by inversion (IMO it makes sense that the cedet version >serve as reference for common stuff). > >Is there a simple way to do that renaming in CVS? Or should we create >the new cedet directory then add the content of the common directory >into it, and finally remove the common directory from the repository. [ ... ] The only way to do this CVS change in one pass is if you have access to the CVS repository file system. (ie, we could download a repository backup, make the change, and request an upload via the sourceforge system administrators... Ick!) A description of doing it via CVS commands is in the CVS info pages and is pretty easy too. If you intend to do this, that would be great. Thanks Eric -- 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 |