Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 459323 - Remove org.eclipse.jdt.ui.examples.projects from Maven build
Summary: Remove org.eclipse.jdt.ui.examples.projects from Maven build
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.5   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.5 M6   Edit
Assignee: Dani Megert CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 420078
  Show dependency tree
 
Reported: 2015-02-06 10:26 EST by Mat Booth CLA
Modified: 2015-02-26 08:30 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mat Booth CLA 2015-02-06 10:26:45 EST
Patch contains some improvements for o.e.jdt.ui.examples.projects:
    
    * Set "strictly compatible JRE" pref to warning (so it works out of the box for everyone)
    * Bump BREE to 1.5 (this is highest BREE used by any eclipse.jdt.ui bundle)
    * Drop useless non-javadoc comments
    * Avoid use of nested jar (prevents workspace "dirt" at build time)
Comment 1 Mat Booth CLA 2015-02-06 10:29:36 EST
Patch submitted for gerrit review:

https://git.eclipse.org/r/41285
Comment 2 Mat Booth CLA 2015-02-06 10:31:16 EST
The "workspace dirt" issue is related to bug 420078 and bug 419506
Comment 3 Dani Megert CLA 2015-02-09 06:26:40 EST
(In reply to Mat Booth from comment #0)
>     * Avoid use of nested jar (prevents workspace "dirt" at build time)

Please see bug 419524 comment 10:
"
I should note, despite some of my mis-statements from years ago, there's nothing about "nested jars", per se, that causes dirty working trees.
"

Is there any "dirt" produced by the jdt.ui.examples?
Comment 4 Dani Megert CLA 2015-02-09 06:30:14 EST
(In reply to Mat Booth from comment #0)
> Patch contains some improvements for o.e.jdt.ui.examples.projects:
>     
>     * Set "strictly compatible JRE" pref to warning (so it works out of the
> box for everyone)

Yes, instead of an error you get a warning that you later ignore, and then cause a build breakage by introducing compile errors into the build. See e.g. bug 458005 comment 4.
Comment 5 Alexander Kurtakov CLA 2015-02-10 06:05:47 EST
(In reply to Dani Megert from comment #3)
> (In reply to Mat Booth from comment #0)
> >     * Avoid use of nested jar (prevents workspace "dirt" at build time)
> 
> Please see bug 419524 comment 10:
> "
> I should note, despite some of my mis-statements from years ago, there's
> nothing about "nested jars", per se, that causes dirty working trees.
> "
> 
> Is there any "dirt" produced by the jdt.ui.examples?

Steps to reproduce the dirt:
git clone git://git.eclipse.org/gitroot/jdt/eclipse.jdt.ui.git
cd eclipse.jdt.ui
mvn clean verify -Pbuild-individual-bundles
mvn clean -Pbuild-individual-bundles
git status 

Actual result:
On branch master
Your branch is up-to-date with 'origin/master'.
Untracked files:
  (use "git add <file>..." to include in what will be committed)

	org.eclipse.jdt.ui.examples.projects/jdtexamples.jar

Expected:
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean
Comment 6 Mat Booth CLA 2015-02-26 05:38:30 EST
Hi Dani,

I was looking at this bug again and I stumbled across bug 278968, where this project was removed from the build around the Eclipse 3.6 timeframe.

Did this regress when the project was converted to tycho build?

If so, should we just remove it from the build again? (Simple edit to the pom.xml)
Comment 7 Dani Megert CLA 2015-02-26 08:27:39 EST
(In reply to Mat Booth from comment #6)
> Hi Dani,
> 
> I was looking at this bug again and I stumbled across bug 278968, where this
> project was removed from the build around the Eclipse 3.6 timeframe.
> 
> Did this regress when the project was converted to tycho build?
> 
> If so, should we just remove it from the build again? (Simple edit to the
> pom.xml)

Good catch Mat!

Removed with http://git.eclipse.org/c/jdt/eclipse.jdt.ui.git/commit/?id=9cde5051fbfae3c3fb00fbe0717c2ffd414b8d99