Thread: [CEDET-devel] [PATCH] Add compilation to ede-linux
Brought to you by:
zappo
From: Robert J. <rob...@fr...> - 2011-07-18 19:58:11
|
Hi Eric and others, I wonder if you would consider taking in that patch, which provides simple target and project compilation commands for linux kernel. As I'm much more confortable with Linux kernel than with elisp code, feel free to comment and/or change the submitted patch. Cheers. -- Robert --- >From b4ad4f28d8354e725c2b30bbb76909f64a64f14b Mon Sep 17 00:00:00 2001 From: Robert Jarzmik <rob...@at...> Date: Mon, 18 Jul 2011 21:26:55 +0200 Subject: [PATCH] ede-linux: add support for target and project compilation Add support to : - compile a target (ie. a Linux kernel directory) - compile the whole project (ie. a full Linux kernel) --- site-elisp/cedet-1.0/ede/ede-linux.el | 52 +++++++++++++++++++++++++++++++++ 1 files changed, 52 insertions(+), 0 deletions(-) diff --git a/site-elisp/cedet-1.0/ede/ede-linux.el b/site-elisp/cedet-1.0/ede/ede-linux.el index 250c744..ab28b6c 100644 --- a/site-elisp/cedet-1.0/ede/ede-linux.el +++ b/site-elisp/cedet-1.0/ede/ede-linux.el @@ -34,8 +34,25 @@ ;; * Add website (require 'ede) +(require 'ede-make) ;;; Code: +(defgroup project-linux nil + "File and tag browser frame." + :group 'tools + :group 'ede + ) + +(defcustom project-linux-compile-target-command (concat ede-make-command " -k -C %s SUBDIRS=%s") + "*Default command used to compile a target." + :group 'project-linux + :type 'string) + +(defcustom project-linux-compile-project-command (concat ede-make-command " -k -C %s") + "*Default command used to compile a project." + :group 'project-linux + :type 'string) + (defvar ede-linux-project-list nil "List of projects created by option `ede-linux-project'.") @@ -237,6 +254,41 @@ Knows about how the Linux source tree is organized." ) (or F (call-next-method)))) +(defmethod project-compile-project ((proj ede-linux-project) + &optional command) + "Compile the entire current project. +Argument COMMAND is the command to use when compiling." + (let* ((dir (ede-project-root-directory proj))) + + (require 'compile) + (if (not project-linux-compile-project-command) + (setq project-linux-compile-project-command compile-command)) + (if (not command) + (setq command + (format + project-linux-compile-project-command + dir))) + + (compile command))) + +(defmethod project-compile-target ((obj ede-linux-target-c) &optional command) + "Compile the current target. +Argument COMMAND is the command to use for compiling the target." + (let* ((proj (ede-target-parent obj)) + (root (ede-project-root proj)) + (dir (ede-project-root-directory root)) + (subdir (oref obj path))) + + (require 'compile) + (if (not project-linux-compile-project-command) + (setq project-linux-compile-project-command compile-command)) + (if (not command) + (setq command + (format + project-linux-compile-target-command + dir subdir))) + + (compile command))) (provide 'ede-linux) ;;; ede-linux.el ends here -- 1.7.4.4 |
From: Eric M. L. <eri...@gm...> - 2011-07-20 01:29:44
|
On 07/18/2011 03:57 PM, Robert Jarzmik wrote: > Hi Eric and others, > > I wonder if you would consider taking in that patch, which provides simple > target and project compilation commands for linux kernel. > > As I'm much more confortable with Linux kernel than with elisp code, feel free > to comment and/or change the submitted patch. Hi, Your patch looks pretty good. It is also longer than the "tiny patch" we can accept without getting papers signed for the FSF. Would you be able to sign a release so that your changes can be incorporated into Emacs? If you write to cop...@fs... and specify you want to contribute to CEDET and Emacs, they will hook you up with the right forms to sign that will enable us to include your patch. Thanks! Eric |
From: Robert J. <rob...@fr...> - 2011-08-06 09:45:09
|
"Eric M. Ludlam" <eri...@gm...> writes: > On 07/18/2011 03:57 PM, Robert Jarzmik wrote: >> Hi Eric and others, >> >> I wonder if you would consider taking in that patch, which provides simple >> target and project compilation commands for linux kernel. >> >> As I'm much more confortable with Linux kernel than with elisp code, feel free >> to comment and/or change the submitted patch. > > Hi, > > Your patch looks pretty good. It is also longer than the "tiny patch" we can > accept without getting papers signed for the FSF. Would you be able to sign a > release so that your changes can be incorporated into Emacs? > > If you write to cop...@fs... and specify you want to contribute to > CEDET and Emacs, they will hook you up with the right forms to sign that will > enable us to include your patch. Hi Eric, I signed and sent back the papers to the FSFs. It shouldn't be long before they get it back, around 3 days. Feel free to use the patch. Cheers. -- Robert |
From: Robert J. <rob...@fr...> - 2011-08-06 09:45:22
|
"Eric M. Ludlam" <eri...@gm...> writes: > On 07/18/2011 03:57 PM, Robert Jarzmik wrote: >> Hi Eric and others, >> >> I wonder if you would consider taking in that patch, which provides simple >> target and project compilation commands for linux kernel. >> >> As I'm much more confortable with Linux kernel than with elisp code, feel free >> to comment and/or change the submitted patch. > > Hi, > > Your patch looks pretty good. It is also longer than the "tiny patch" we can > accept without getting papers signed for the FSF. Would you be able to sign a > release so that your changes can be incorporated into Emacs? > > If you write to cop...@fs... and specify you want to contribute to > CEDET and Emacs, they will hook you up with the right forms to sign that will > enable us to include your patch. Hi Eric, I signed and sent back the papers to the FSFs. It shouldn't be long before they get it back, around 3 days. Feel free to use the patch. Cheers. -- Robert |
From: Robert J. <rob...@fr...> - 2011-09-24 10:05:16
|
"Eric M. Ludlam" <eri...@gm...> writes: > Hi, ... zip ... > If you write to cop...@fs... and specify you want to contribute to > CEDET and Emacs, they will hook you up with the right forms to sign that will > enable us to include your patch. Hi Eric, Could you include the patch now I'm registered in FSF database please ? Cheers. -- Robert |
From: Eric L. <eri...@gm...> - 2011-09-25 21:47:44
|
On 09/24/2011 06:05 AM, Robert Jarzmik wrote: > "Eric M. Ludlam"<eri...@gm...> writes: >> Hi, > ... zip ... >> If you write to cop...@fs... and specify you want to contribute to >> CEDET and Emacs, they will hook you up with the right forms to sign that will >> enable us to include your patch. > Hi Eric, > > Could you include the patch now I'm registered in FSF database please ? > > Cheers. > Thanks! Are there any volunteers who would like to merge and test Robert's patch to ede-linux? (Originally posted July 18th) Thanks Eric |
From: Eric M. L. <eri...@gm...> - 2012-01-21 03:51:21
|
On 09/24/2011 06:05 AM, Robert Jarzmik wrote: > "Eric M. Ludlam"<eri...@gm...> writes: >> Hi, > ... zip ... >> If you write to cop...@fs... and specify you want to contribute to >> CEDET and Emacs, they will hook you up with the right forms to sign that will >> enable us to include your patch. > > Hi Eric, > > Could you include the patch now I'm registered in FSF database please ? Sorry for the long delay. I have finally committed this change. Thanks for your patience. Eric |