Bug 1114566

Summary: Be explicit about curl support
Product: [Fedora] Fedora Reporter: Ville Skyttä <ville.skytta>
Component: mupdfAssignee: Pavel Zhukov <pavel>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: rawhideCC: dominik, pavel, pzhukov
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-10-28 19:27:13 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:

Description Ville Skyttä 2014-06-30 11:16:08 UTC
Since you asked in bug 1102901:

mupdf compiles in curl support if curl-devel is installed.

If that's desirable, it needs to be made explicit for reproducible builds by adding BuildRequires: curl-devel.

If that's not desirable, that needs to be made explicit too, either by adding BuildConflicts: curl-devel or otherwise ensuring that curl support is not built in even if curl-devel is installed.

Comment 1 Ville Skyttä 2014-07-04 23:45:04 UTC
1.5-3 changelog mentions adding curl-devel BR, but doesn't actually add it.

http://pkgs.fedoraproject.org/cgit/mupdf.git/commit/?id=c5c814f

Comment 2 Dominik 'Rathann' Mierzejewski 2016-08-02 20:24:31 UTC
This still not fixed, please do not close until it is.

Comment 3 Pavel Zhukov 2016-10-28 19:27:13 UTC
As I can see this was fixed 2 years ago:
http://pkgs.fedoraproject.org/cgit/rpms/mupdf.git/commit/?id=fbcd9c181f72e828e53be544812d7dc9cf837fc4

Did I miss something?

Comment 4 Dominik 'Rathann' Mierzejewski 2016-10-31 01:40:13 UTC
Indeed it was, thanks. It would've been easier to see if the dependencies were specified one per line and sorted.

Comment 5 Pavel Zhukov 2016-10-31 06:31:44 UTC
(In reply to Dominik 'Rathann' Mierzejewski from comment #4)
> Indeed it was, thanks. It would've been easier to see if the dependencies
> were specified one per line and sorted.

Please use grep next time.