Skip to content

Commit

Permalink
Drop shebang from upload_doc.sh (#408)
Browse files Browse the repository at this point in the history
This file isn't directly executable in any of its forms:
* When checked into the repository, it is a template
* When installed by gz-cmake, it is still a template
* When expanded in downstream packages, it isn't explicitly given
  executable permission, requiring an explicit interpreter invocation

Additionally, the usage for this script instructs the user to explicitly
provide the interpreter. To avoid misleading users and linters into
believing that this file should be executed directly, drop the
(currently unused) shebang.

Signed-off-by: Scott K Logan <[email protected]>
  • Loading branch information
cottsay authored Jan 24, 2024
1 parent cbdb0cc commit a29a26f
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions cmake/upload_doc.sh.in
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
#!/bin/sh

echo "Usage: sh upload_doc.sh [y/n]"
echo " Optional [y/n] argument indicates whether to upload the docs to S3 automatically."

Expand Down

0 comments on commit a29a26f

Please sign in to comment.