Messages in this thread | | | Subject | Re: [PATCH] tools/power: Don't make man pages executable | From | Laura Abbott <> | Date | Tue, 19 Dec 2017 16:36:26 -0800 |
| |
On 12/19/2017 03:48 PM, Rafael J. Wysocki wrote: > On 12/19/2017 8:54 PM, Laura Abbott wrote: >> rpm-lint flagged these as being executable: >> >> kernel-tools.x86_64: W: spurious-executable-perm /usr/share/man/man8/turbostat.8.gz >> kernel-tools.x86_64: W: spurious-executable-perm /usr/share/man/man8/x86_energy_perf_policy.8.gz >> >> Fix this >> >> Signed-off-by: Laura Abbott <labbott@redhat.com> > > Please CC PM stuff to linux-pm (which makes it easier to handle). >
Neither of these are listed in MAINTAINERS so I missed sending to linux-pm. I'll remember for next time, do you want me to resend for tracking?
Thanks, Laura
>> --- >> tools/power/x86/turbostat/Makefile | 2 +- >> tools/power/x86/x86_energy_perf_policy/Makefile | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/tools/power/x86/turbostat/Makefile b/tools/power/x86/turbostat/Makefile >> index a9bc914a8fe8..2ab25aa38263 100644 >> --- a/tools/power/x86/turbostat/Makefile >> +++ b/tools/power/x86/turbostat/Makefile >> @@ -25,4 +25,4 @@ install : turbostat >> install -d $(DESTDIR)$(PREFIX)/bin >> install $(BUILD_OUTPUT)/turbostat $(DESTDIR)$(PREFIX)/bin/turbostat >> install -d $(DESTDIR)$(PREFIX)/share/man/man8 >> - install turbostat.8 $(DESTDIR)$(PREFIX)/share/man/man8 >> + install -m 644 turbostat.8 $(DESTDIR)$(PREFIX)/share/man/man8 >> diff --git a/tools/power/x86/x86_energy_perf_policy/Makefile b/tools/power/x86/x86_energy_perf_policy/Makefile >> index 2447b1bbaacf..f4534fb8b951 100644 >> --- a/tools/power/x86/x86_energy_perf_policy/Makefile >> +++ b/tools/power/x86/x86_energy_perf_policy/Makefile >> @@ -24,5 +24,5 @@ install : x86_energy_perf_policy >> install -d $(DESTDIR)$(PREFIX)/bin >> install $(BUILD_OUTPUT)/x86_energy_perf_policy $(DESTDIR)$(PREFIX)/bin/x86_energy_perf_policy >> install -d $(DESTDIR)$(PREFIX)/share/man/man8 >> - install x86_energy_perf_policy.8 $(DESTDIR)$(PREFIX)/share/man/man8 >> + install -m 644 x86_energy_perf_policy.8 $(DESTDIR)$(PREFIX)/share/man/man8 > > > Thanks, > > Rafael > >
| |