Bug 1405633

Summary: java-comment-preprocessor-6.0.1-3.fc25 FTBFS
Product: [Fedora] Fedora Reporter: Merlin Mathesius <mmathesi>
Component: java-comment-preprocessorAssignee: Pavel Kajaba <pkajaba>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 25CC: mizdebsk, pkajaba, praiskup, psabata, sgallagh
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-02-22 05:48:25 UTC Type: Bug
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: 1400162    
Deadline: 2016-12-30   
Attachments:
Description Flags
proposed patch to correct FTBFS issue none

Description Merlin Mathesius 2016-12-16 21:49:35 UTC 
java-comment-preprocessor-6.0.1-3.fc25 fails to build from source.

This affects Base Runtime, Rawhide, and F25.

The reason for this is undeclared build dependencies on maven-plugin-plugin, maven-source-plugin, and maven-plugin-testing-harness.

This can be corrected by adding BuildRequires for maven-plugin-plugin, maven-source-plugin, and maven-plugin-testing-harness.
Comment 1 Merlin Mathesius 2016-12-16 21:52:52 UTC 
Created attachment 1232769 [details]
proposed patch to correct FTBFS issue
Comment 2 Merlin Mathesius 2016-12-16 21:53:34 UTC 
Please apply the patch or respond with a reason it should not be applied within two weeks. After that time, a provenpackager will go ahead and apply it.
Comment 3 Pavel Raiskup 2016-12-19 06:13:35 UTC 
Thanks for the patch!

http://pkgs.fedoraproject.org/cgit/rpms/java-comment-preprocessor.git/commit/?id=f8e2f80f8cb2698bd9d
Comment 4 Merlin Mathesius 2016-12-19 15:49:27 UTC 
Thank you for the quick fix.
Comment 5 Mikolaj Izdebski 2017-01-31 23:44:36 UTC 
Patch was committed, but build not submitted. Reopening.
Comment 6 Mikolaj Izdebski 2017-02-01 00:07:17 UTC 
Fixed in rawhide only, https://koji.fedoraproject.org/koji/buildinfo?buildID=837511

F25 is still not fixed, I'll leave it up to the maintainer.
Comment 7 Pavel Raiskup 2017-02-20 09:39:32 UTC 
Mikolaj, do we need build in F25?
Comment 8 Mikolaj Izdebski 2017-02-21 22:18:03 UTC 
(In reply to Pavel Raiskup from comment #7)
> Mikolaj, do we need build in F25?

I personally don't have any need for fixing F25, but AFAIK modularity work is based on F25.
Comment 9 Merlin Mathesius 2017-02-21 22:40:15 UTC 
Base Runtime / Modularity is fine with just the Rawhide build of the package.
Comment 10 Pavel Raiskup 2017-02-22 05:48:25 UTC 
OK, closing.  The commit is merged in f25 (without build).