PR#2: Only do %post chown if /var is writable - rpms/mock

Colin Walters • 2 years ago

0af1cd

mock.spec

file modified

+6 -3

		`@@ -17,7 +17,7 @@`
		`Summary: Builds packages inside chroots`
		`Name: mock`
		`Version: 1.4.7`
		`- Release: 2%{?dist}`
		`+ Release: 3.post%{?dist}`
		`License: GPLv2+`
		`# Source is created by`
		`# git clone https://github.com/rpm-software-management/mock.git`
		`@@ -172,8 +172,11 @@`
		`install -d %{buildroot}/var/cache/mock`

		`%post`
		`- # fix cache permissions from old installs`
		`- chmod 2775 %{_localstatedir}/cache/%{name}`
		`+ # fix cache permissions from old installs; test for writability in`
		`+ # advance for compatibility with rpm-ostree: https://bugzilla.redhat.com/show_bug.cgi?id=1352154`
		`+ if test -w %{_localstatedir}/cache/%{name}; then`
		`+ chmod 2775 %{_localstatedir}/cache/%{name}`
		`+ fi`

		`%check`
		`# ignore the errors for now, just print them and hopefully somebody will fix it one day`