Bug 796831 - Missing POM file in jettison
Summary: Missing POM file in jettison
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: jettison
Version: rawhide
Hardware: Unspecified
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Sandro Mathys
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-23 18:09 UTC by Juan Hernández
Modified: 2012-03-13 18:30 UTC (History)
1 user (show)

Fixed In Version: jettison-1.3.1-4.fc17
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-03-13 18:30:28 UTC
Type: ---


Attachments (Terms of Use)
Proposed patch to build with maven and add POM file (5.45 KB, patch)
2012-02-23 18:09 UTC, Juan Hernández
no flags Details | Diff
Proposed patch to remove the %clean section (1.28 KB, patch)
2012-03-12 17:38 UTC, Juan Hernández
no flags Details | Diff
Proposed patch to remove the %clean section and add the maven fragments (1.49 KB, patch)
2012-03-12 17:48 UTC, Juan Hernández
no flags Details | Diff

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.


Note You need to log in before you can comment on or make changes to this bug.