[yocto] [eclipse-poky][RFC 0/6] Introducing a YoctoSDKCMakeProjectNature

Wed Apr 24 21:27:13 PDT 2013

Hi Atanas,

Please see my comments below...

Thanks,

Jessica

From: Atanas Gegov [mailto:atanas.gegov.oss at gmail.com]
Sent: Wednesday, April 24, 2013 8:48 AM
To: Zhang, Jessica
Cc: yocto at yoctoproject.org
Subject: Re: [yocto] [eclipse-poky][RFC 0/6] Introducing a YoctoSDKCMakeProjectNature

Hi Jessica,

thank you for the comments!

1.      Yes, the idea is that at the end there are the current category ("Yocto Project ADT Autotools Project" with some Autotools-based project templates) and a new category "Yocto Project ADT CMake Project" with e.g. its own "Hello World C++ CMake Project". The latter category and the templates will be implemented in the upcoming patch series.

Great that's what I'd expected .

2.      Could you please give me a small example how it should look like?Should the method addNatures receive a list of natures to add? What is the case if more natures are added to a project (e.g. SDKNature + AutotoolsNature + EmptyNature)? Or do you think of enum based on the buildsystem natures only (Autotools vs. CMake) and not based on all 4 natures?

I took another look of the class, we should be able to even further simplify the API of addNatures.  We can make all the natures as instance variables of the class.  In process method we assign them values and totally eliminate passing natures in addNatures API.  So the code should be(please ignore the incorrect capitalization due to auto correction):

Public class NewYoctoProjectTemplateProcess extends ProcessRunner {

...

            Private Boolean isEmptyProject;

            Private Boolean isAutotoolsProject;

            Private Boolean isCMakeProject;

...

            Public void process(...) {

            ...

                        isEmptyProject = ...

                        isAutotoolsProject = ...

                        isCMakeProject = ...

            ...

                        addNatures(project, false, monitor);

            ...

            }

            ...

            Private void addNatures(IProject project, Boolean projectExists, IProgressMonitor monitor) {

                        ...

            }

3.      I just created an account in bugzilla.yoctoproject.org<http://bugzilla.yoctoproject.org/>, so you can assign me the CMake support entry.

Super.  You'll get your first assignment J

Thanks,
Atanas

On Tue, Apr 23, 2013 at 8:50 PM, Zhang, Jessica <jessica.zhang at intel.com<mailto:jessica.zhang at intel.com>> wrote:

Hi Atanas,

Overall, the patches look good.  Couple comments:

1. So from the end user perspective, after the 2nd patch set, user will see "Yocto Project ADT CMake Project" category when they want to create a new project?
2. Can we change the API for addNatures by using an enum type for projectNature instead of explicitly listing all of them out.  This way is easier for code maintenance and API usability.
3. Don't know whether you've created a bugzilla account at bugzilla.yoctoproject.org<http://bugzilla.yoctoproject.org> or not, if not, please do, since I'd hope you're able to help fixing certain bugs in this area down the road.  Also, from project planning process wise, we're right in the 1.5 planning, and I'd like to create an entry in the bugzilla for extending eclipse for CMake support and assign that to you, so it's easier to track what are the community contribution to the project.

Thanks,
Jessica

-----Original Message-----
From: yocto-bounces at yoctoproject.org<mailto:yocto-bounces at yoctoproject.org> [mailto:yocto-bounces at yoctoproject.org<mailto:yocto-bounces at yoctoproject.org>] On Behalf Of Atanas Gegov
Sent: Monday, April 22, 2013 5:06 AM
To: yocto at yoctoproject.org<mailto:yocto at yoctoproject.org>
Subject: [yocto] [eclipse-poky][RFC 0/6] Introducing a YoctoSDKCMakeProjectNature

From: Atanas Gegov <atanas.gegov at bmw-carit.de<mailto:atanas.gegov at bmw-carit.de>>

Hi,

This patch series introduces a YoctoSDKCMakeProjectNature as a new nature for CMake-based Yocto Projects in the IDE. It requires the basic YoctoSDKProjectNature and is an alternative to the YoctoSDKAutotoolsProjectNature.

This does not affect the currently existing Autotools template projects.

Upcoming patch series will add the last missing parts for having CMake projects in the IDE: a ManagedBuilder for CMake (CMake toolchain for building in Eclipse) and some template projects. However, I already wanted to post this series for a review and make sure that the concept is acceptable, so that we can build on top of it.

Cheers, Atanas

P.S: This patch series apply on the current master "plugins/sdk.ide:
Extracted error message to YoctoSDKMessages"
(7142c5fc82a41a9d5f86c48b289db0ef44f5a897).

Atanas Gegov (6):
  plugins/sdk.ide: Prepared templates for CMake nature
  plugins/sdk.ide: Added a new YoctoSDKCMakeProjectNature
  plugins/sdk.ide: Enable the creation of CMake projects
  plugins/sdk.ide: Update CMake projects on Yocto settings change
  plugins/sdk.ide: Renamed template process
  plugins/sdk.ide: Fixed whitespace issues

 .../OSGI-INF/l10n/bundle.properties                |    1 +
 plugins/org.yocto.sdk.ide/plugin.xml               |   25 +-
 .../ide/natures/YoctoSDKCMakeProjectNature.java    |   92 +++++++
 .../src/org/yocto/sdk/ide/utils/YoctoSDKUtils.java |    7 +-
 .../sdk/ide/wizard/NewYoctoCProjectTemplate.java   |  245 -------------------
 .../ide/wizard/NewYoctoProjectTemplateProcess.java |  256 ++++++++++++++++++++
 .../EmptyCAutotoolsProject/template.xml            |    1 +
 .../EmptyCPPAutotoolsProject/template.xml          |    1 +
 .../HelloWorldCAutotoolsProject/template.xml       |    1 +
 .../HelloWorldCGTKAutotoolsProject/template.xml    |    1 +
 .../HelloWorldCPPAutotoolsProject/template.xml     |    1 +
 11 files changed, 384 insertions(+), 247 deletions(-)  create mode 100644 plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/natures/YoctoSDKCMakeProjectNature.java
 delete mode 100644 plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/wizard/NewYoctoCProjectTemplate.java
 create mode 100644 plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/wizard/NewYoctoProjectTemplateProcess.java

--
1.7.9.5

_______________________________________________
yocto mailing list
yocto at yoctoproject.org<mailto:yocto at yoctoproject.org>
https://lists.yoctoproject.org/listinfo/yocto

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/yocto/attachments/20130425/517db8fb/attachment.html>