Bug 889144 - maven-checkstyle-plugin: Migrate from maven-doxia to doxia subpackages
Summary: maven-checkstyle-plugin: Migrate from maven-doxia to doxia subpackages
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: maven-checkstyle-plugin
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michal Srb
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 909869
TreeView+ depends on / blocked
 
Reported: 2012-12-20 10:40 UTC by Mikolaj Izdebski
Modified: 2013-02-11 11:01 UTC (History)
4 users (show)

Fixed In Version: 2.9.1-4
Clone Of:
Environment:
Last Closed: 2013-01-04 16:12:29 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)
spec file patch (1.68 KB, patch)
2012-12-20 12:11 UTC, Michal Srb 		no flags Details | Diff
spec file patch (1.71 KB, patch)
2013-01-02 14:17 UTC, Michal Srb 		no flags Details | Diff
Show Obsolete (1) View All

Description Mikolaj Izdebski 2012-12-20 10:40:27 UTC 
maven-doxia was split into multiple subpackages. Your package currently requires or build-requires maven-doxia. Please fix your requires or build requires of your package maven-checkstyle-plugin so that it requires proper subpackages of doxia.
Comment 1 Michal Srb 2012-12-20 12:11:27 UTC 
Created attachment 666627 [details]
spec file patch
Comment 2 Mikolaj Izdebski 2012-12-21 10:10:54 UTC 
I don't understand your choice of subpackages to add as Requires. In my opinion you should replace maven-dixia with doxia-decoration-model, doxia-sink-api and doxia-site-renderer. It would also be nice to ad these as BuildRequires too.
Comment 3 Michal Srb 2013-01-02 14:17:53 UTC 
Created attachment 671448 [details]
spec file patch

You're right Mikolaj. It should be fixed in attached patch. Note doxia-decoration-model and doxia-site-renderer are provided by maven-doxia-sitetools package.
Comment 4 Mikolaj Izdebski 2013-01-04 15:56:32 UTC 
Commited by sochotni.
Note You need to log in before you can comment on or make changes to this bug.