This request is superseded by
request 836091
(Show diff)
Overview
Request 836059 superseded
ditto
- Created by WernerFink
- In state superseded
- Supersedes 836051
- Superseded by 836091
- Open review for opensuse-review-team
- Open review for openSUSE:Factory:Staging:adi:109
Loading...
Request History
WernerFink created request
ditto
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
dimstar_suse added as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:109"
dimstar_suse accepted review
Picked "openSUSE:Factory:Staging:adi:109"
superseded by 836091
That took me two seconds to understand why this looks so weird :)
you likely want to use distconfdir here.
On old systems, you have this defined to be /etc, on newer systems it is pre-defined to be /usr/etc.
The way you package it now, it has potential to fail when distconfdir is not defined, as then you create the files in prefix/sysconfdir (aka /usr/etc) but in the files section you then use distconfdir, which has been redefined to sysconfdir, aka /etc