[Bug 1898312] Review Request: jakarta-json - Jakarta JSON Processing

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

 



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



--- Comment #5 from Jerry James <loganjerry@xxxxxxxxx> ---
(In reply to Fabio Valentini from comment #4)
> 1) You can mark maven packages to not get installed with "%mvn_package", so
> you don't have to remove them manually:
> Here's an example in jakarta-el:
> https://src.fedoraproject.org/rpms/jakarta-el/blob/master/f/jakarta-el.
> spec#_71

Oh, good.  I had hoped there was some way of doing that.  Thanks for the
pointer.

> 2) According to the "renaming packages" documentation, the obsoleted NVRs
> should be < 1.0.4-12, because 1.0.4-11 was the last available NVR of jsonp
> and jsonp-javadoc.

Fixed.

> 3) There are weird Requires for jakarta-json:
> 
>     mvn(org.apache.maven.plugins:maven-compiler-plugin)
>     mvn(org.apache.maven.plugins:maven-source-plugin)
>     mvn(org.codehaus.mojo:build-helper-maven-plugin)
> 
> Looks like the top-level maven project is a "parent POM", so that should be
> fine.

I'm not sure the parent POM should even be packaged.  Let me see if there is
any sign that it's really needed and if not, I will not install it either.

Thanks for the review!  What can I do for you?  Toss me something from your
TODO list.


-- 
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
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux