launchpad-reviewers team mailing list archive

[Simone <mp+439147@xxxxxxxxxxxxxxxxxx>]

Simone has proposed merging ~pelpsi/launchpad:bzrsync-is-executable-for-no-apparent-reason into launchpad:master.

Commit message:
Bzrsync.py was executable for no apparent reason

Fixed the permissions chmod -x bzrsync.py 
and removed shebang line.

LP: #736619

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~pelpsi/launchpad/+git/launchpad/+merge/439147
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~pelpsi/launchpad:bzrsync-is-executable-for-no-apparent-reason into launchpad:master.

diff --git a/lib/lp/codehosting/scanner/bzrsync.py b/lib/lp/codehosting/scanner/bzrsync.py
old mode 100755
new mode 100644
index 8ba1c48..db9c67b
--- a/lib/lp/codehosting/scanner/bzrsync.py
+++ b/lib/lp/codehosting/scanner/bzrsync.py
@@ -1,5 +1,3 @@
-#!/usr/bin/python3
-#
 # Copyright 2009-2018 Canonical Ltd.  This software is licensed under the
 # GNU Affero General Public License version 3 (see the file LICENSE).