Bug 1853808 - ccache is not required
Summary: ccache is not required
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: abi-compliance-checker
Version: 32
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Richard Shaw
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-07-04 11:33 UTC by Magnus Fromreide
Modified: 2020-07-24 12:11 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-07-24 12:11:16 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Magnus Fromreide 2020-07-04 11:33:47 UTC
Description of problem:
abi-compliance-checker drags in ccache

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

How reproducible:
Always

Steps to Reproduce:
1. Install abi-compliance-checker on a host without ccache

Actual results:
ccache is installed

Expected results:
ccache does not get installed

Additional info:
According to https://lvc.github.io/abi-compliance-checker/ there is a conflict with ccache < 3.1.2 so the spec file line

Requires: ccache >= 3.1.2

should have been

Conflicts: ccache < 3.1.2

but according to the Packager Documentation (https://docs.fedoraproject.org/en-US/packaging-guidelines/Conflicts/#_optional_functionality) Fedora only care about two releases back and ccache have been sufficiently new for that long so I suppose the right thing to do is to delete this Requires line.

Comment 1 Richard Shaw 2020-07-05 12:17:50 UTC
Thanks for pointing this out. I have changed it to Conflicts as you never know who is going to use these spec files to build for older system. I'm not sure what version of ccache is in the various EL releases.

Is pulling in ccache causing a problem for you? Do I need to push updates? Or is simply updating the spec file for future builds sufficient?

Comment 2 Magnus Fromreide 2020-07-05 15:54:19 UTC
Thanks for the quick fix.
Updating for future builds should be enough, I was just surprised about ccache beeing installed and looked into why that was.


Note You need to log in before you can comment on or make changes to this bug.