Overview

Request 445416 superseded

- Import preset macros from systemd-rpm-macro and convert them into
scripts.
Presets are not supposed to be shipped by other packages as the
default policy needs to be reviewed by the secteam. Therefore don't
encourage packagers to do that and make those macros private to this
package.
Also convert them into real scripts so they're much easier to
maintain and debug. Also working in %posttrans is unneeded as this
package is assumed to be the only one to update the preset.

- There's no need to reload the daemon configuration as presets are
not part of the dameon config. (forwarded request 445123 from fbui)


Dominique Leuenberger's avatar
+if [ $1 -gt 1 ] ; then
+   #
+   # Save the old state so we can detect which package have its
+   # default changed later.
+   #
+   # Note: the old version of the script is used here.
+   #
+   %{_prefix}/lib/%{generic_name}/branding-preset-states save
+fi

Note: old package version does potentially not have the script at all - so it can't be found

Request History
Marcus Meissner's avatar

msmeissn created request

- Import preset macros from systemd-rpm-macro and convert them into
scripts.
Presets are not supposed to be shipped by other packages as the
default policy needs to be reviewed by the secteam. Therefore don't
encourage packagers to do that and make those macros private to this
package.
Also convert them into real scripts so they're much easier to
maintain and debug. Also working in %posttrans is unneeded as this
package is assumed to be the only one to update the preset.

- There's no need to reload the daemon configuration as presets are
not part of the dameon config. (forwarded request 445123 from fbui)


Saul Goodman's avatar

licensedigger accepted review


Factory Auto's avatar

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

Please review sources


Factory Auto's avatar

factory-auto added factory-repo-checker as a reviewer

Please review build success


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Dominique Leuenberger's avatar

dimstar_suse set openSUSE:Factory:Staging:B as a staging project

Being evaluated by staging project "openSUSE:Factory:Staging:B"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Picked openSUSE:Factory:Staging:B


Factory Repo Checker's avatar

factory-repo-checker reopened review

systemd-presets-branding-openSUSE is still building for repository openSUSE_Factory


Jan Engelhardt's avatar

jengelh accepted review


Yuchen Lin's avatar

maxlin_factory added openSUSE:Factory:Staging:D as a reviewer

Being evaluated by staging project "openSUSE:Factory:Staging:D"


Yuchen Lin's avatar

maxlin_factory accepted review

Moved to openSUSE:Factory:Staging:D


Factory Repo Checker's avatar

factory-repo-checker reopened review


Factory Repo Checker's avatar

factory-repo-checker accepted review

Builds for repo openSUSE:Factory:Staging:D/standard


Dominique Leuenberger's avatar

dimstar_suse accepted review

Removing from openSUSE:Factory:Staging:D, re-evaluation needed


Dominique Leuenberger's avatar

dimstar_suse approved review

Removing from openSUSE:Factory:Staging:D, re-evaluation needed


Dominique Leuenberger's avatar

dimstar_suse added factory-staging as a reviewer

Requesting new staging review


Marcus Meissner's avatar

msmeissn superseded request

superseded by 446738

openSUSE Build Service is sponsored by