[Bug 1352091] Review Request: enunciate - Build-time enhancement tool for Java-based Web services projects

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

 



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

gil cattaneo <puntogil@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(puntogil@libero.i |
                   |t)                          |



--- Comment #6 from gil cattaneo <puntogil@xxxxxxxxx> ---
(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
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx




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