Bug 414698 - [CBI] webtools.common.fproj patch for building with Maven/Tycho
Summary: [CBI] webtools.common.fproj patch for building with Maven/Tycho
Status: RESOLVED FIXED
Alias: None
Product: WTP Common Tools
Classification: WebTools
Component: Faceted Project Framework (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.6 M2   Edit
Assignee: Carl Anderson CLA
QA Contact: Konstantin Komissarchik CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 412211
  Show dependency tree
 
Reported: 2013-08-08 15:26 EDT by Thanh Ha CLA
Modified: 2013-09-10 13:30 EDT (History)
2 users (show)

See Also:


Attachments
webtools.common.fproj.patch (25.79 KB, patch)
2013-08-08 15:26 EDT, Thanh Ha CLA
ccc: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thanh Ha CLA 2013-08-08 15:26:34 EDT
Created attachment 234208 [details]
webtools.common.fproj.patch

Patch for webtools.common.fproj, all modules are buildable with CBI.
Comment 1 Carl Anderson CLA 2013-09-10 13:17:25 EDT
I am not sure about the change to org.eclipse.wst.common.fproj.feature.patch - that is a feature patch used for patching a feature after a release, and should not be part of the normal WTP build.
Comment 2 Thanh Ha CLA 2013-09-10 13:25:49 EDT
(In reply to Carl Anderson from comment #1)
> I am not sure about the change to org.eclipse.wst.common.fproj.feature.patch
> - that is a feature patch used for patching a feature after a release, and
> should not be part of the normal WTP build.

Should I create a new patch that excludes it from building?


To exclude it from a regular build we simply need to remove the module declaration in the repo root pom.xml file. We could leave the bundle pom.xml existing in case someone needs to build the patch via cbi too. As long as the root pom does not include it, it won't be built.
Comment 3 Carl Anderson CLA 2013-09-10 13:28:41 EDT
I went ahead and committed all of the changes.  I believe that we should go back and remove the two .feature.patch entries from the root pom.xml, but for now I will leave it as is, to get the CBI build working.

Is there a way to comment out that line in a pom.xml?

Also, I copied the .gitignore from webtools.common.tests, since it is much more complete.
Comment 4 Thanh Ha CLA 2013-09-10 13:30:55 EDT
(In reply to Carl Anderson from comment #3)
> I went ahead and committed all of the changes.  I believe that we should go
> back and remove the two .feature.patch entries from the root pom.xml, but
> for now I will leave it as is, to get the CBI build working.
> 
> Is there a way to comment out that line in a pom.xml?
> 
> Also, I copied the .gitignore from webtools.common.tests, since it is much
> more complete.

Yes it's just standard xml so you can add a comment with:

<!-- single line comment -->

or 

<!--
    multi
    line
    comment
-->