[fedora-java] Packaging m2e-wtp

[Roland Grunberg]

> On 02/20/2015 02:54 PM, Mikolaj Izdebski wrote:
> > On 02/20/2015 01:45 PM, Julius Schwartzenberg wrote:
> >> I added a few more dependencies and now I'm running into this error:
> >>
> >> [ERROR] Failed to execute goal
> >> org.eclipse.tycho:tycho-packaging-plugin:0.21.0:package-plugin
> >> (default-package-plugin) on project org.eclipse.m2e.wtp.overlay:
> >> Execution default-package-plugin of goal
> >> org.eclipse.tycho:tycho-packaging-plugin:0.21.0:package-plugin failed:
> >> One of setGitDir or setWorkTree must be called. -> [Help 1]
> >
> > My guess is that upstream is using older version of Tycho and you'll
> > need to port the build to Tycho 0.21.0 (patch pom.xml).
> 
> I'm patching pom.xml as part of the build now to use Tycho 0.21.0 (was
> indeed 0.20.0), but I still receive the same error.

The issue is related to something else, but the reason this wouldn't
have worked is because your maven rpm build would resolve the version
of Fedora installed on your system (through rpm). Changing the version
property would have no effect, so in these cases one has to look at
what changed in the newer version and adapt the pom files to reflect
that.

> From: "Mikolaj Izdebski" <mizdebsk at redhat.com>
> To: java-devel at lists.fedoraproject.org
> Sent: Wednesday, February 25, 2015 7:24:18 AM
> Subject: Re: [fedora-java] Packaging m2e-wtp
> 
> This is a bug in tycho-extras. I've just filed it:
> https://bugzilla.redhat.com/show_bug.cgi?id=1196167
> 
> As workaround you can add to %prep:
> %pom_remove_plugin :tycho-packaging-plugin

As mentioned in the bug, the error is a bit cryptic but expected. You
basically need to decide whether you want to include a shallow copy of
the git history in the sources to allow generating qualifiers based on
the last commit made to the sub-project, or not. If you don't want build
qualifiers based on commit date, then I believe Mikolaj's suggestion of
removing the packaging plugin entirely should work (it will run anyways
with default settings).

Cheers,
-- 
Roland Grunberg