Overview

Request 836091 accepted

OK

Loading...

Dominique Leuenberger's avatar
31+mkdir -p %{buildroot}%{_prefix}%{_sysconfdir}/pam.d
32+cat > %{buildroot}%{_prefix}%{_sysconfdir}/pam.d/physlock <<-'EOF'

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

Request History
Dr. Werner Fink's avatar

WernerFink created request

OK


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Saul Goodman's avatar

licensedigger accepted review

ok


Richard Brown's avatar

RBrownSUSE added as a reviewer

Being evaluated by staging project "openSUSE:Factory:Staging:adi:109"


Richard Brown's avatar

RBrownSUSE accepted review

Picked "openSUSE:Factory:Staging:adi:109"


Dominique Leuenberger's avatar

dimstar accepted review


Dominique Leuenberger's avatar

dimstar_suse accepted review

Staging Project openSUSE:Factory:Staging:adi:109 got accepted.


Dominique Leuenberger's avatar

dimstar_suse approved review

Staging Project openSUSE:Factory:Staging:adi:109 got accepted.


Dominique Leuenberger's avatar

dimstar_suse accepted request

Staging Project openSUSE:Factory:Staging:adi:109 got accepted.

openSUSE Build Service is sponsored by