Bug 414719 - [CBI] webtools.releng patches for building with Maven/Tycho
Summary: [CBI] webtools.releng patches for building with Maven/Tycho
Status: RESOLVED FIXED
Alias: None
Product: WTP Releng
Classification: WebTools
Component: releng (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: David Williams CLA
QA Contact: Carl Anderson CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 412211 419265
  Show dependency tree
 
Reported: 2013-08-08 16:37 EDT by Thanh Ha CLA
Modified: 2013-10-24 09:48 EDT (History)
3 users (show)

See Also:


Attachments
webtools.releng.patch (82.14 KB, patch)
2013-08-08 16:37 EDT, Thanh Ha CLA
ccc: iplog+
Details | Diff
webtools.releng.aggregator.patch (23.37 KB, patch)
2013-08-08 16:41 EDT, Thanh Ha CLA
ccc: iplog+
Details | Diff
aggregator.zip (32.32 KB, application/zip)
2013-10-17 14:57 EDT, Thanh Ha CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Thanh Ha CLA 2013-08-08 16:37:58 EDT
Created attachment 234234 [details]
webtools.releng.patch

Patch for webtools.releng, several modules are not buildable due to duplicate ID or missing bundles. I've documented them in comments inside the root pom.xml.

I'm not entirely sure if some of these modules should be built or not.

One feature.xml from org.eclipse.wtp.xml.dev.assembly.feature was modified to add a missing ".qualifier" to the version.
Comment 1 Thanh Ha CLA 2013-08-08 16:41:12 EDT
Created attachment 234235 [details]
webtools.releng.aggregator.patch

A new aggregator will need to be created. locally I called mine webtools.releng.aggregator.git but any alternative names can be used if there is a preference.


As a note, the aggregator does not yet include profiles for signing the build with Eclipse infrastructure. We should add this once the patches are merged and we can test against the repos.
Comment 2 Thanh Ha CLA 2013-09-25 13:40:11 EDT
Carl,

Looks like this is the last of the patches to be merged. I guess after webtools.releng.patch gets merged we will need to create a webtools.releng.aggregator repo somewhere.

Should I go ahead and create a new repo for this under /gitroot/webtools?
Comment 3 Carl Anderson CLA 2013-09-26 16:11:57 EDT
I had problems applying the following changes:

features/org.eclipse.jst.doc.isv.feature/pom.xml (this feature lives in webtools.javaee now - neither are currently built)

features/org.eclipse.wtp.package.jee.feature/pom.xml (this feature has the same id as org.eclipse.wtp.jee.feature - not part of the WTP build, this is EPP support)

plugins/org.eclipse.javaee.intro/pom.xml (this plugin has the same id as org.eclipse.wtp.epp.package.jee.intro - not part of the WTP build, this is EPP support)

plugins/org.eclipse.wtp.jee.intro/pom.xml (this plugin has the same id as org.eclipse.wtp.intro - not part of the WTP build, this is EPP support)

plugins/org.eclipse.wtp.package.jee/pom.xml (this plugin has the same id as org.eclipse.wtp.jee - not part of the WTP build, this is EPP support)

Since we do not build/ship these as part of WTP 3.6, I did not worry about adding their pom files.
Comment 4 Thanh Ha CLA 2013-09-26 16:18:36 EDT
(In reply to Carl Anderson from comment #3)
> I had problems applying the following changes:
> 
> features/org.eclipse.jst.doc.isv.feature/pom.xml (this feature lives in
> webtools.javaee now - neither are currently built)
> 
> features/org.eclipse.wtp.package.jee.feature/pom.xml (this feature has the
> same id as org.eclipse.wtp.jee.feature - not part of the WTP build, this is
> EPP support)
> 
> plugins/org.eclipse.javaee.intro/pom.xml (this plugin has the same id as
> org.eclipse.wtp.epp.package.jee.intro - not part of the WTP build, this is
> EPP support)
> 
> plugins/org.eclipse.wtp.jee.intro/pom.xml (this plugin has the same id as
> org.eclipse.wtp.intro - not part of the WTP build, this is EPP support)
> 
> plugins/org.eclipse.wtp.package.jee/pom.xml (this plugin has the same id as
> org.eclipse.wtp.jee - not part of the WTP build, this is EPP support)
> 
> Since we do not build/ship these as part of WTP 3.6, I did not worry about
> adding their pom files.

Yes if they are not used in the build I would just leave them out. I created poms for everything simply because I did not know what should / is built.
Comment 5 Carl Anderson CLA 2013-10-17 14:03:06 EDT
Thanh, please apply the webtools.releng.aggregator.patch.  I am frustrated with my inability to do so with EGit.
Comment 6 Thanh Ha CLA 2013-10-17 14:21:05 EDT
(In reply to Carl Anderson from comment #5)
> Thanh, please apply the webtools.releng.aggregator.patch.  I am frustrated
> with my inability to do so with EGit.

Hi Carl, It just occurred to me that I wouldn't be able to do this since I'm not a committter. Would it make sense if I became a committer for wtp releng? or do we have other options?
Comment 7 David Williams CLA 2013-10-17 14:45:09 EDT
(In reply to Carl Anderson from comment #5)
> Thanh, please apply the webtools.releng.aggregator.patch.  I am frustrated
> with my inability to do so with EGit.

And why is that? When I look, webtools.releng.aggregator is empty. 
If that is the case, then a "patch" is not appropriate form ... should be just a set of files to "share". Which would become the initial contents.
Comment 8 Thanh Ha CLA 2013-10-17 14:55:51 EDT
(In reply to David Williams from comment #7)
> (In reply to Carl Anderson from comment #5)
> > Thanh, please apply the webtools.releng.aggregator.patch.  I am frustrated
> > with my inability to do so with EGit.
> 
> And why is that? When I look, webtools.releng.aggregator is empty. 
> If that is the case, then a "patch" is not appropriate form ... should be
> just a set of files to "share". Which would become the initial contents.

I'm not sure how Egit handles it but the patch works with commandline git. I just cloned the empty repo and ran:

    git am webtools.releng.aggregator.patch


I will try attaching a zip of the repo I produced that contains all the files in case that helps.
Comment 9 Thanh Ha CLA 2013-10-17 14:57:06 EDT
Created attachment 236610 [details]
aggregator.zip
Comment 10 David Williams CLA 2013-10-17 15:53:13 EDT
I've shoe-horned something into the repository ... not sure it's quite right ... but maybe now that its none-empty more progress could be made. 

Good luck!
Comment 11 Thanh Ha CLA 2013-10-17 16:01:53 EDT
(In reply to David Williams from comment #10)
> I've shoe-horned something into the repository ... not sure it's quite right
> ... but maybe now that its none-empty more progress could be made. 
> 
> Good luck!

Thanks David, Looks like my patches are there now. I will test and make sure they still work as expected.
Comment 12 Carl Anderson CLA 2013-10-23 18:03:47 EDT
I believe that David handled this request.  (Thanks!)  If there is something left to do, please reopen this bug.
Comment 13 Thanh Ha CLA 2013-10-24 09:48:36 EDT
(In reply to Carl Anderson from comment #12)
> I believe that David handled this request.  (Thanks!)  If there is something
> left to do, please reopen this bug.

Yes this one's done we still need the patches from bug 419265 though.