Merge lp:~townsend/click/no-follow-symlinks into lp:click/devel

Proposed by Michael Vogt
Status: Merged
Merged at revision: 579
Proposed branch: lp:~townsend/click/no-follow-symlinks
Merge into: lp:click/devel
Diff against target: 29 lines (+4/-3)
2 files modified
click/install.py (+1/-1)
debian/changelog (+3/-2)
To merge this branch: bzr merge lp:~townsend/click/no-follow-symlinks
Reviewer Review Type Date Requested Status
Michael Vogt Pending
Review via email: mp+271646@code.launchpad.net

This proposal supersedes a proposal from 2015-09-17.

Commit message

Don't follow symlinks when stat()ing files.

To post a comment you must log in.
Revision history for this message
Michael Vogt (mvo) wrote : Posted in a previous version of this proposal

Thanks, this looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'click/install.py'
--- click/install.py 2014-12-03 12:42:21 +0000
+++ click/install.py 2015-09-18 13:36:32 +0000
@@ -418,7 +418,7 @@
418 for dirpath, dirnames, filenames in os.walk(inst_dir):418 for dirpath, dirnames, filenames in os.walk(inst_dir):
419 for entry in dirnames + filenames:419 for entry in dirnames + filenames:
420 entry_path = os.path.join(dirpath, entry)420 entry_path = os.path.join(dirpath, entry)
421 entry_mode = os.stat(entry_path).st_mode421 entry_mode = os.lstat(entry_path).st_mode
422 new_entry_mode = entry_mode | stat.S_IRGRP | stat.S_IROTH422 new_entry_mode = entry_mode | stat.S_IRGRP | stat.S_IROTH
423 if entry_mode & stat.S_IXUSR:423 if entry_mode & stat.S_IXUSR:
424 new_entry_mode |= stat.S_IXGRP | stat.S_IXOTH424 new_entry_mode |= stat.S_IXGRP | stat.S_IXOTH
425425
=== modified file 'debian/changelog'
--- debian/changelog 2015-07-02 09:45:22 +0000
+++ debian/changelog 2015-09-18 13:36:32 +0000
@@ -1,10 +1,11 @@
1click (0.4.39.1-0ubuntu1) UNRELEASED; urgency=medium1click (0.4.39.1+15.10.20150702-0ubuntu1) wily; urgency=medium
22
3 [ Michael Vogt ]
3 * lp:~mvo/click/lp1456328-15.10-devlibs:4 * lp:~mvo/click/lp1456328-15.10-devlibs:
4 - add ubuntu-sdk-15.10-dev15 - add ubuntu-sdk-15.10-dev1
5 - remove ubuntu-core-15.04-dev16 - remove ubuntu-core-15.04-dev1
67
7 -- Michael Vogt <michael.vogt@ubuntu.com> Thu, 02 Jul 2015 09:05:10 +02008 -- CI Train Bot <ci-train-bot@canonical.com> Thu, 02 Jul 2015 11:43:15 +0000
89
9click (0.4.39) vivid; urgency=low10click (0.4.39) vivid; urgency=low
1011

Subscribers

People subscribed via source and target branches