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

gil cattaneo <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?([email protected] |
                   |t)                          |



--- Comment #6 from gil cattaneo <[email protected]> ---
(In reply to Ding-Yi Chen from comment #4)
> Thanks gil,
> 
> Come to think of it, since I also need enunciate-core and
> enunciate-jersey-rt, should I make the enunciate as main package instead,
> and provide enunciate-core-rt, enunciate-core, enunciate-jeresy-rt as
> sub-packages?

yes, is ok for me. but is available a new version:
https://github.com/stoicflame/enunciate/releases/tag/v2.5.0
you should add a comment, in the spec file, as to why you want to import this
old version

maybe you could remove:
%mvn_package ":enunciate-parent" enunciate
and change or remove the main package in

%files -f .mfiles-%{name}-parent
%license license.txt notice.txt

....

%files core-rt -f .mfiles-%{name}-core-rt
%doc README.md
%license license.txt notice.txt

or

%files parent -f .mfiles-%{name}-parent
%license license.txt notice.txt

...

i dont understand this:

# remove enunciate-core-annotations, because it is in other package 
rm -f %{buildroot}/%{_javadir}/enunciate/enunciate-core-annotations.jar
rm -f
%{buildroot}/%{_datadir}/maven-metadata/enunciate-enunciate-core-annotations.xml
rm -f
%{buildroot}/%{_datadir}/maven-poms/enunciate/enunciate-core-annotations.pom

you should disable this module, and use the other package

and for last you should add missing build-requires:
mvn(com.sun.jersey:jersey-server:1)
mvn(com.sun.jersey:jersey-servlet:1)
mvn(com.sun.jersey.contribs:jersey-spring:1)

-- 
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
[email protected]
https://lists.fedoraproject.org/admin/lists/[email protected]

Reply via email to