Bug 819599 - add maven pom
Summary: add maven pom
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: jnr-x86asm
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Mikolaj Izdebski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-05-07 16:37 UTC by gil cattaneo
Modified: 2012-10-10 12:02 UTC (History)
2 users (show)

Fixed In Version: 0.1-5
Clone Of:
Environment:
Last Closed: 2012-10-10 12:02:39 UTC
Type: Bug
Embargoed:

Attachments (Terms of Use)
add maven pom (3.36 KB, patch)
2012-05-07 16:37 UTC, gil cattaneo 		no flags Details | Diff
add maven pom -rebase (2.78 KB, patch)
2012-10-09 17:55 UTC, gil cattaneo 		no flags Details | Diff
add maven pom -rebase (3.46 KB, patch)
2012-10-09 18:23 UTC, gil cattaneo 		no flags Details | Diff
add maven pom -rebase (2.48 KB, patch)
2012-10-10 11:49 UTC, gil cattaneo 		no flags Details | Diff
Show Obsolete (3) View All

Description gil cattaneo 2012-05-07 16:37:43 UTC 
Created attachment 582710 [details]
add maven pom

Description of problem:
add maven pom

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


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 gil cattaneo 2012-10-09 17:55:29 UTC 
Created attachment 624212 [details]
add maven pom -rebase
Comment 2 gil cattaneo 2012-10-09 18:23:46 UTC 
Created attachment 624218 [details]
add maven pom -rebase
Comment 3 Mikolaj Izdebski 2012-10-10 05:43:44 UTC 
Why do you change artifact version to 1.0.0? I mean this line:
> sed -i "s|<version>0.1</version>|<version>1.0.0</version>|" pom.xml

Also instead of %patch0 it would be better to write:
> %pom_xpath_remove pom:extensions
Comment 4 gil cattaneo 2012-10-10 11:41:52 UTC 
(In reply to comment #3)
> Why do you change artifact version to 1.0.0? I mean this line:
> > sed -i "s|<version>0.1</version>|<version>1.0.0</version>|" pom.xml
> 
this should be removed

> Also instead of %patch0 it would be better to write:
> > %pom_xpath_remove pom:extensions
yes but i havent this macro available until a short time ago

thanks
Comment 5 gil cattaneo 2012-10-10 11:49:13 UTC 
Created attachment 624807 [details]
add maven pom -rebase
Comment 6 Mikolaj Izdebski 2012-10-10 12:02:39 UTC 
Applied in rawhide. Thanks.
Note You need to log in before you can comment on or make changes to this bug.