Bug 861413

Summary: system-config-printer.py should not be executable
Product: [Fedora] Fedora Reporter: Michal Nowak <mnowak>
Component: system-config-printerAssignee: Tim Waugh <twaugh>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: unspecified    
Version: 17CC: jpopelka, twaugh
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-10-01 14:03:32 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Michal Nowak 2012-09-28 14:20:08 UTC
Description of problem:

/usr/share/system-config-printer/system-config-printer.py is currently executable file:

    -rwxr-xr-x. /usr/share/system-config-printer/system-config-printer.py

and because of this zeitgeist-datahub is looking for it's desktop file (which does not exist) and eventually fails with

    ** (process:1256): WARNING **: recent-manager-provider.vala:125: Desktop file for system-config-printer.py was not found

in ~/.xsession-error file.

Since s-c-printer is being executed via /usr/bin/s-c-printer or via it's desktop file, s-c-printer.py does not need executable bit.

Version-Release number of selected component (if applicable):

system-config-printer-1.3.9-1.fc17.x86_64

Comment 1 Jiri Popelka 2012-10-01 12:51:47 UTC
Seems it needs to be executable:

# chmod -x /usr/share/system-config-printer/system-config-printer.py

# system-config-printer
/bin/system-config-printer: line 3: /usr/share/system-config-printer/system-config-printer.py: Permission denied
/bin/system-config-printer: line 3: exec: /usr/share/system-config-printer/system-config-printer.py: cannot execute: Permission denied

Comment 2 Tim Waugh 2012-10-01 14:03:32 UTC
Yes, the /usr/bin script executes it.