[
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