[Bug 854239] Review Request: gpars - Groovy Parallel Systems

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=854239



--- Comment #6 from Michael Simacek <msimacek@xxxxxxxxxx> ---
(In reply to gil cattaneo from comment #5)
> (In reply to Michael Simacek from comment #4)
> > My mistake, I had different version of groovy in my local repo which ended
> > up in mock.
> 
> If you have also groovy 2.x in your repo then you have a broken package or
> you have not built the ant sub-module, or worse you also don't package the
> groovy-all 2.x jar (Note: groovy 2.x with all sub-modules only without
> external deps)

I no longer have it in the local repo so there shouldn't be any such problem
anymore.

> 
> > Package Review
> > ==============
> > 
> > Legend:
> > [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
> > [ ] = Manual review needed
> > 
> > 
> > Issues:
> > =======
> > - Missing BuildRequires: javapackages-local and aqute-bndlib
> Already fixed now use aqute-bnd, you should (re-)download the latest src rpm
> > - Doesn't build with netty3 in rawhide - the jar path is different and
> >   the <version> in POM doesn't match
> Can you post the correct jar path (/usr/..../netty3<?>.jar)

/usr/share/java/netty3-3.6.6.jar

and change the <version> tag in the POM to just "3", otherwise the specified
version doesn't match the compat provider.

> > - Bundled extra166y - I think this part should be packaged separately
> Re-open https://bugzilla.redhat.com/show_bug.cgi?id=854234 with only this
> module?

Rather open a new one with just the extra part. The other parts of jsr-166 seem
to be already present in JDK, so I'd like to avoid duplication. Name it
extra166y or jsr-166-extra.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]