Bug 889144

Summary: maven-checkstyle-plugin: Migrate from maven-doxia to doxia subpackages
Product: [Fedora] Fedora Reporter: Mikolaj Izdebski <mizdebsk>
Component: maven-checkstyle-pluginAssignee: Michal Srb <msrb>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: java-sig-commits, msrb, sochotni, SpikeFedora
Target Milestone: ---Keywords: Patch
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 2.9.1-4 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-01-04 16:12:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 909869    
Attachments:
Description Flags
spec file patch
none
spec file patch none

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.