Bug 441331 - Integrate e4 project template into PDE
Summary: Integrate e4 project template into PDE
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.4   Edit
Hardware: PC All
: P3 enhancement (vote)
Target Milestone: 4.5 M7   Edit
Assignee: Olivier Prouvost CLA
QA Contact:
URL:
Whiteboard:
Keywords: greatfix, noteworthy
Depends on: 446418
Blocks: 463821 465680
  Show dependency tree
 
Reported: 2014-08-07 06:24 EDT by Lars Vogel CLA
Modified: 2015-04-28 08:30 EDT (History)
5 users (show)

See Also:
Vikas.Chandra: review+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2014-08-07 06:24:12 EDT
I plan to contribute our e4 tools wizard for creating Eclipse 4 RCP applications to PDE. This is a work item for me.
Comment 1 Vikas Chandra CLA 2014-12-01 08:34:27 EST
Is this on for 4.5 M4? If yes, we should plan early enough.
Comment 2 Vikas Chandra CLA 2014-12-07 03:41:56 EST
If this cant go in 4.5M4, probably we can move it to 4.5 M5
Comment 3 Lars Vogel CLA 2014-12-08 01:31:54 EST
(In reply to Vikas Chandra from comment #2)
> If this cant go in 4.5M4, probably we can move it to 4.5 M5

I could hold up by other work, sorry for this, I try to implement that for M5.
Comment 4 Vikas Chandra CLA 2015-01-27 06:39:37 EST
Lars, I am moving this to 4.5M6 as today is the last day to commit for 4.5M5
Comment 5 Eclipse Genie CLA 2015-03-10 19:27:17 EDT
New Gerrit change created: https://git.eclipse.org/r/43621
Comment 6 Eclipse Genie CLA 2015-03-16 03:32:19 EDT
New Gerrit change created: https://git.eclipse.org/r/43893
Comment 7 Olivier Prouvost CLA 2015-03-16 05:06:55 EDT
I created this review : 

https://git.eclipse.org/r/#/c/43893/

But the hudson build failed with this error : 

https://hudson.eclipse.org/platform/job/eclipse.pde.ui-Gerrit/267/console

So, I tried to build it at home, using mvn -X clean install and I got this error :

Something wrong with org.eclipse.pde.api.tools ? 

Any Idea ?
Thanks 

[ERROR] Cannot resolve project dependencies:
[ERROR]   Software being installed: org.eclipse.pde.api.tools 1.0.700.qualifier
[ERROR]   Missing requirement: org.eclipse.pde.api.tools 1.0.700.qualifier requires 'bundle org.eclipse.osgi [3.4.0,4.0.0)' but it could not be found
[ERROR] 
[ERROR] See http://wiki.eclipse.org/Tycho/Dependency_Resolution_Troubleshooting for help.
[ERROR] Cannot resolve dependencies of MavenProject: org.eclipse.pde:org.eclipse.pde.api.tools:1.0.700-SNAPSHOT @ /Users/olivier/git/eclipse.pde.ui/apitools/org.eclipse.pde.api.tools/pom.xml: See log for details -> [Help 1]
org.apache.maven.MavenExecutionException: Cannot resolve dependencies of MavenProject: org.eclipse.pde:org.eclipse.pde.api.tools:1.0.700-SNAPSHOT @ /Users/olivier/git/eclipse.pde.ui/apitools/org.eclipse.pde.api.tools/pom.xml
	at org.eclipse.tycho.core.maven.TychoMavenLifecycleParticipant.afterProjectsRead(TychoMavenLifecycleParticipant.java:92)
	at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:317)
	at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:155)
	at org.apache.maven.cli.MavenCli.execute(MavenCli.java:584)
	at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:213)
	at org.apache.maven.cli.MavenCli.main(MavenCli.java:157)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:483)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229)
	at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415)
	at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356)
Caused by: org.eclipse.tycho.artifacts.DependencyResolutionException: Cannot resolve dependencies of MavenProject: org.eclipse.pde:org.eclipse.pde.api.tools:1.0.700-SNAPSHOT @ /Users/olivier/git/eclipse.pde.ui/apitools/org.eclipse.pde.api.tools/pom.xml
	at org.eclipse.tycho.p2.resolver.P2ResolverImpl.resolveDependencies(P2ResolverImpl.java:202)
	at org.eclipse.tycho.p2.resolver.P2ResolverImpl.resolveDependencies(P2ResolverImpl.java:116)
	at org.eclipse.tycho.p2.resolver.P2DependencyResolver.doResolveDependencies(P2DependencyResolver.java:367)
	at org.eclipse.tycho.p2.resolver.P2DependencyResolver.resolveDependencies(P2DependencyResolver.java:335)
	at org.eclipse.tycho.core.resolver.DefaultTychoResolver.resolveProject(DefaultTychoResolver.java:117)
	at org.eclipse.tycho.core.maven.TychoMavenLifecycleParticipant.afterProjectsRead(TychoMavenLifecycleParticipant.java:87)
	... 13 more
Caused by: org.eclipse.tycho.p2.util.resolution.ResolverException: See log for details
	at org.eclipse.tycho.p2.util.resolution.ProjectorResolutionStrategy.resolve(ProjectorResolutionStrategy.java:88)
	at org.eclipse.tycho.p2.util.resolution.AbstractResolutionStrategy.resolve(AbstractResolutionStrategy.java:42)
	at org.eclipse.tycho.p2.resolver.P2ResolverImpl.resolveDependencies(P2ResolverImpl.java:195)
	... 18 more
[ERROR] 
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MavenExecutionException
Comment 9 Lars Vogel CLA 2015-03-18 08:58:35 EDT
(In reply to Olivier Prouvost from comment #7)
> I created this review : 
> 
> https://git.eclipse.org/r/#/c/43893/
> 
> But the hudson build failed with this error : 
> 
> https://hudson.eclipse.org/platform/job/eclipse.pde.ui-Gerrit/267/console

Build failure should be fixed, if you push an update it should build again.
Comment 10 Olivier Prouvost CLA 2015-03-18 10:10:16 EDT
Yes It worked !
Comment 11 Lars Vogel CLA 2015-03-19 05:12:57 EDT
Vikas, can this be integrated into early M7?

We have a fork of the PDE wizard in eclipse.platform.ui.tools and this would allow us to get rid of it.
Comment 12 Vikas Chandra CLA 2015-03-24 05:05:13 EDT
Hi Lars,

What will be the impact of this change on PDE ? Or will this change/refactoring just affect e4 applications ?
Comment 13 Lars Vogel CLA 2015-03-24 05:20:42 EDT
(In reply to Vikas Chandra from comment #12)
> Hi Lars,
> 
> What will be the impact of this change on PDE ? Or will this
> change/refactoring just affect e4 applications ?

Except this and potential more future templates I do not see a necessary change in PDE. We want to use the existing PDE template framework.
Comment 15 Vikas Chandra CLA 2015-03-24 06:28:24 EDT
Thanks Olivier/Lars

Looks fine. All the tests run fine. See previous comment for commit id.
Comment 16 Lars Vogel CLA 2015-03-24 21:47:22 EDT
This should be added to the N&N for M7. Thanks a bunch  Olivier.
Comment 17 Lars Vogel CLA 2015-04-28 08:30:22 EDT
Tested in 4.5.0.I20150428-0100. Looks good. I will suggest minor changes in the labels via Bug 465680.