[Asterisk-code-review] Remove executable flag from files that are not executable. (testsuite[newmaster])

Kevin Harwell asteriskteam at digium.com
Thu Mar 8 11:25:09 CST 2018


Kevin Harwell has submitted this change and it was merged. ( https://gerrit.asterisk.org/8469 )

Change subject: Remove executable flag from files that are not executable.
......................................................................

Remove executable flag from files that are not executable.

This removes the executable flag from *.xml and all *.py files that do
not contain any line matching '^#!'.

Also removes a couple invalid shebangs in subscriptions tests.

ASTERISK-24292

Change-Id: I9aac68227e706a8d456110d6a5d46434ed8b1f57
(cherry picked from commit 233f6fe497fb2cf8d0c763a231eb370aebd41248)
---
M tests/channels/SIP/rfc2833_dtmf_detect/sipp/broken_dtmf.xml
M tests/channels/SIP/rfc2833_dtmf_detect/sipp/dtmf_baseline.xml
M tests/channels/SIP/sip_hold/sipp/phone_A_no_direction.xml
M tests/channels/SIP/sip_hold/sipp/phone_B_no_direction.xml
M tests/channels/pjsip/subscriptions/rls/rls_test.py
M tests/channels/pjsip/subscriptions/rls/rls_validation.py
M tests/channels/pjsip/transfers/attended_transfer/nominal/callee_remote/transfer.py
M tests/manager/bridge_actions/check_bridge.py
M tests/rest_api/applications/double-subscribe-device-state/subscribe_device_state.py
M tests/rest_api/bridges/bridge_record/bridges_record.py
M tests/rest_api/bridges/playback/basic/bridges_play.py
M tests/rest_api/bridges/playback/tones/bridges_play.py
M tests/rest_api/channels/originate_then_continue/channel_originate.py
M tests/rest_api/external_interaction/attended_transfer/attended_transfer.py
M tests/rest_api/external_interaction/blind_transfer/stasis_bridge_to_non_stasis_app/blind_transfer.py
M tests/rest_api/mailbox/baseline/mailbox_baseline.py
M tests/rest_api/recording/file_conflicts/recording.py
M tests/rest_api/recording/format_unavailable/recording.py
18 files changed, 0 insertions(+), 2 deletions(-)

Approvals:
  Corey Farrell: Looks good to me, but someone else must approve
  Kevin Harwell: Looks good to me, approved; Approved for Submit



diff --git a/tests/channels/SIP/rfc2833_dtmf_detect/sipp/broken_dtmf.xml b/tests/channels/SIP/rfc2833_dtmf_detect/sipp/broken_dtmf.xml
old mode 100755
new mode 100644
diff --git a/tests/channels/SIP/rfc2833_dtmf_detect/sipp/dtmf_baseline.xml b/tests/channels/SIP/rfc2833_dtmf_detect/sipp/dtmf_baseline.xml
old mode 100755
new mode 100644
diff --git a/tests/channels/SIP/sip_hold/sipp/phone_A_no_direction.xml b/tests/channels/SIP/sip_hold/sipp/phone_A_no_direction.xml
old mode 100755
new mode 100644
diff --git a/tests/channels/SIP/sip_hold/sipp/phone_B_no_direction.xml b/tests/channels/SIP/sip_hold/sipp/phone_B_no_direction.xml
old mode 100755
new mode 100644
diff --git a/tests/channels/pjsip/subscriptions/rls/rls_test.py b/tests/channels/pjsip/subscriptions/rls/rls_test.py
old mode 100755
new mode 100644
index 4ed9730..9c4051c
--- a/tests/channels/pjsip/subscriptions/rls/rls_test.py
+++ b/tests/channels/pjsip/subscriptions/rls/rls_test.py
@@ -1,4 +1,3 @@
-#/usr/bin/env python
 """
 Copyright (C) 2015, Digium, Inc.
 Jonathan Rose <jrose at digium.com>
diff --git a/tests/channels/pjsip/subscriptions/rls/rls_validation.py b/tests/channels/pjsip/subscriptions/rls/rls_validation.py
old mode 100755
new mode 100644
index 7f30264..3d84f82
--- a/tests/channels/pjsip/subscriptions/rls/rls_validation.py
+++ b/tests/channels/pjsip/subscriptions/rls/rls_validation.py
@@ -1,4 +1,3 @@
-#/usr/bin/env python
 """
 Copyright (C) 2015, Digium, Inc.
 Ashley Sanders <asanders at digium.com>
diff --git a/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_remote/transfer.py b/tests/channels/pjsip/transfers/attended_transfer/nominal/callee_remote/transfer.py
old mode 100755
new mode 100644
diff --git a/tests/manager/bridge_actions/check_bridge.py b/tests/manager/bridge_actions/check_bridge.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/applications/double-subscribe-device-state/subscribe_device_state.py b/tests/rest_api/applications/double-subscribe-device-state/subscribe_device_state.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/bridges/bridge_record/bridges_record.py b/tests/rest_api/bridges/bridge_record/bridges_record.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/bridges/playback/basic/bridges_play.py b/tests/rest_api/bridges/playback/basic/bridges_play.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/bridges/playback/tones/bridges_play.py b/tests/rest_api/bridges/playback/tones/bridges_play.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/channels/originate_then_continue/channel_originate.py b/tests/rest_api/channels/originate_then_continue/channel_originate.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/external_interaction/attended_transfer/attended_transfer.py b/tests/rest_api/external_interaction/attended_transfer/attended_transfer.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/external_interaction/blind_transfer/stasis_bridge_to_non_stasis_app/blind_transfer.py b/tests/rest_api/external_interaction/blind_transfer/stasis_bridge_to_non_stasis_app/blind_transfer.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/mailbox/baseline/mailbox_baseline.py b/tests/rest_api/mailbox/baseline/mailbox_baseline.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/recording/file_conflicts/recording.py b/tests/rest_api/recording/file_conflicts/recording.py
old mode 100755
new mode 100644
diff --git a/tests/rest_api/recording/format_unavailable/recording.py b/tests/rest_api/recording/format_unavailable/recording.py
old mode 100755
new mode 100644

-- 
To view, visit https://gerrit.asterisk.org/8469
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: newmaster
Gerrit-MessageType: merged
Gerrit-Change-Id: I9aac68227e706a8d456110d6a5d46434ed8b1f57
Gerrit-Change-Number: 8469
Gerrit-PatchSet: 1
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180308/4763fe9d/attachment.html>


More information about the asterisk-code-review mailing list