Bug 1074273 - fedora-gnat-project-common installs macros files to /etc/rpm
Summary: fedora-gnat-project-common installs macros files to /etc/rpm
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: fedora-gnat-project-common
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Björn Persson
QA Contact: Fedora Extras Quality Assurance
URL: https://lists.fedoraproject.org/piper...
Whiteboard:
Depends On:
Blocks: 1074261
TreeView+ depends on / blocked
 
Reported: 2014-03-09 10:52 UTC by Ville Skyttä
Modified: 2015-02-08 15:09 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-02-08 15:09:35 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Ville Skyttä 2014-03-09 10:52:29 UTC
The proper location for rpm macros files in rpm >= 4.11 is
%{_rpmconfigdir}/macros.d, please move them there from /etc/rpm.

If this package's specfile targets Fedora and EL >= 7 only, the
location for macro files can be simply changed from /etc/rpm to
%{_rpmconfigdir}/macros.d. If it is intended to work on EL 5 and/or 6
as well, you can define a macros dir for example as follows (all on
one line) and install the macros to %{macrosdir}:

%global macrosdir %(d=%{_rpmconfigdir}/macros.d; [ -d $d ] || d=%{_sysconfdir}/rpm; echo $d)

Comment 1 Björn Persson 2014-03-09 12:37:17 UTC
How about you read the file in question and then tell me what you think about it?

Comment 2 Ville Skyttä 2014-03-09 16:22:12 UTC
I think the file in /etc/rpm should be removed.

Comment 3 Björn Persson 2014-03-09 16:37:30 UTC
And in the spec you'll find a note that it's scheduled to be removed next year.

Comment 4 Ville Skyttä 2014-03-10 14:09:12 UTC
I think it would be better use of both of our time if you'd just close this bug along with an explanation if you have no intention to remove the file, or just remove it. I've read the specfile and the macros file per comment 1, and I feel that already was a waste of my time, not that it was that much.

To be clear, I still think there's no point in leaving the file behind.

Comment 5 Björn Persson 2015-02-08 15:09:35 UTC
Explanation file removed on schedule.


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