After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 593454 - __init__.py doesn't need to be executable
__init__.py doesn't need to be executable
Status: RESOLVED FIXED
Product: libgweather
Classification: Core
Component: general
2.27.x
Other Linux
: Normal minor
: 2.22.0
Assigned To: libgweather-maint
libgweather-maint
Depends on:
Blocks:
 
 
Reported: 2009-08-28 23:45 UTC by Emilio Pozuelo Monfort
Modified: 2009-09-08 23:44 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Don't install __init__.py with executable permissions (754 bytes, patch)
2009-08-28 23:47 UTC, Emilio Pozuelo Monfort 		committed Details | Review

Description Emilio Pozuelo Monfort 2009-08-28 23:45:27 UTC 
Hi,

__init__.py doesn't need to be executable. In fact, one of the two __init__.py files libgweather installs is set executable.

The attached patch makes it not be installed executable. I've tested it and it does the right thing, and gweather keeps importing fine.
Comment 1 Emilio Pozuelo Monfort 2009-08-28 23:47:47 UTC 
Created attachment 141973 [details] [review]
Don't install __init__.py with executable permissions

The patch is against 2.26 but I guess it will apply fine in trunk
Comment 2 Vincent Untz 2009-09-08 23:44:03 UTC 
Thanks!