Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cbi-dev] Empty source bundles

On Tue, May 15, 2012 at 5:01 PM, Thanh Ha <thanh.ha@xxxxxxxxxxx> wrote:
Hi Paul,

I incorrectly recalled this morning that there were ~5 .source bundles that were removed from features. I did a grep of all the feature.xml files and only found 2 listed below that were commented out.


eclipse.platform.releng/features/org.eclipse.sdk.tests/feature.xml:
<!--
   igorf: org.eclipse.sdk.tests does not have any sources, afaict
   <plugin
         id="org.eclipse.sdk.tests.source"
         download-size="0"
         install-size="0"
         version="0.0.0"/>
-->



eclipse.platform.releng/features/org.eclipse.sdk/feature.xml:
<!--
   thanh: needed to comment this out for build to succeed.
   Not sure if this was the right thing to do.
   <includes
         id="org.eclipse.cvs.source"
         version="0.0.0"/>
-->



OK, these were the ones I was looking for, I'll find out if we can just remove them upstream.

 

The reason for my confusion was that there was actually additional plugins that we removed in the features.xml as well, I've listed the ones that I could find below.



Why are these commented out?  AFAIK CBI-Platform build is picking up these platforms from the p2-natives repo.  Can they be included since they're coming from a pre-built repo?  I don't believe we would want these changes pushed upstream.

I remember the bundle-shape changes.  We were supposed to push those upstream (I thought we had already, but I guess not).  Any build.properties changes we would have to review before we pushed them upstream.

re:changes.

I think we should open one bug per project (Platform, JDT, PDE, Equinox) and list the repos that they should cover.  Then committers can pull their changes.

Later,
PW


--
Paul Webster
Hi floor.  Make me a sammich! - GIR

Back to the top