Bug 796831

Summary: Missing POM file in jettison
Product: [Fedora] Fedora Reporter: Juan Hernández <juan.hernandez>
Component: jettisonAssignee: Sandro Mathys <sandro>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rawhideCC: sandro
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: jettison-1.3.1-4.fc17 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-03-13 18:30:28 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Proposed patch to build with maven and add POM file
none
Proposed patch to remove the %clean section
none
Proposed patch to remove the %clean section and add the maven fragments none

Description Juan Hernández 2012-02-23 18:09:14 UTC
Created attachment 565346 [details]
Proposed patch to build with maven and add POM file

Description of problem:

The jettison package available in rawhide doesn't include the POM file, this complicates the build of other components that need jettison, like RESTEasy, for example.

Version-Release number of selected component (if applicable):

jettison-1.3.1-1.fc17.noarch

How reproducible:

Always.

Steps to Reproduce:

1. Build with maven anything that needs jettison.
  
Actual results:

The build fails.

Expected results:

The build should succeed.

Additional info:

I am attaching a patch that changes the spec file and adds a patch to fix the POM provided by the upstream project.

The patched spec builds correctly in Koji:

http://koji.fedoraproject.org/koji/taskinfo?taskID=3813411

Comment 1 Fedora Update System 2012-03-06 13:59:05 UTC
jettison-1.3.1-3.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/jettison-1.3.1-3.fc17

Comment 2 Sandro Mathys 2012-03-06 14:00:38 UTC
Thanks for providing a patch. Included in F17 and rawhide.

Comment 3 Fedora Update System 2012-03-07 07:23:06 UTC
Package jettison-1.3.1-3.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing jettison-1.3.1-3.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-3206/jettison-1.3.1-3.fc17
then log in and leave karma (feedback).

Comment 4 Juan Hernández 2012-03-12 17:37:48 UTC
Somehow the patch didn't apply correctly and the creation of the maven dependencies map ended in the %clean section. I am attaching a patch to fix that.

Sandro, I am open to apply the do the fix/build/update of the package if you give me permission to do so. Let me know what you think. Thanks in advance!

Comment 5 Juan Hernández 2012-03-12 17:38:33 UTC
Created attachment 569466 [details]
Proposed patch to remove the %clean section

Comment 6 Juan Hernández 2012-03-12 17:48:48 UTC
Created attachment 569471 [details]
Proposed patch to remove the %clean section and add the maven fragments

Comment 7 Sandro Mathys 2012-03-12 19:00:54 UTC
Thanks for the patch. Applied and built packages for F17/F18, also pushed the update for F17.

Juan, if you wish to, I'm happy to turn ownership of the package over to you - you clearly have the better knowledge about java stuff and more interest in the package, too.

But if I'm to remain owner I'd prefer to apply patches myself unless I don't think I can get it done in due time, in which case I'd let you know through the bug report.

Apologies that I was a bit slow with the first patch, I've been traveling when you submitted it and had to catch up with other tasks first after I returned.

Either way, thank you for asking my permission prior to just acting, I really appreciate that :)

Comment 8 Juan Hernández 2012-03-12 19:20:11 UTC
Thanks for applying the patch. I was faster than I would expect from anyone. If in the future you need help with this package please let me know.

Comment 9 Juan Hernández 2012-03-12 19:21:09 UTC
(In reply to comment #8)
> Thanks for applying the patch. I was faster than I would expect from anyone. If
> in the future you need help with this package please let me know.

I meant "It was faster ...", not "I was faster ...", sorry for the typo.

Comment 10 Fedora Update System 2012-03-13 18:30:28 UTC
jettison-1.3.1-4.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.