Fix Travis CI PRs builds and add copyrights (#52)

* Fix Travis CI PRs builds and add copyrights

* Update env-vars-check

* Update env-vars-check

* Update env-vars-check

scripts/build

@@ -1,4 +1,7 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 # Usage:
 # Build the project in the local environment:  ./scripts/build

scripts/build.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 :: Usage:

scripts/clean-up

@@ -1,4 +1,7 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 APP_HOME="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && cd .. && pwd )/"
 source "$APP_HOME/scripts/.functions.sh"

scripts/clean-up.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 :: strlen("\scripts\") => 9

scripts/compile

@@ -1,4 +1,7 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 # Usage:
 # Compile the project in the local environment:  ./scripts/compile

scripts/compile.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 :: Usage:

scripts/docker/build

@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 # Copyright (c) Microsoft. All rights reserved.
-# Note: Bash doesn't parse very well shebang arguments (e.g "-e") since version 4.
+# Note: Windows Bash doesn't support shebang extra params
 set -e
 
 # Note: use lowercase names for the Docker images

scripts/docker/build.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 :: Note: use lowercase names for the Docker images

scripts/docker/publish

@@ -1,4 +1,7 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 # Note: use lowercase names for the Docker images
 DOCKER_IMAGE="azureiotpcs/pcs-storage-adapter-dotnet"

scripts/docker/publish.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 :: Note: use lowercase names for the Docker images

scripts/docker/run

@@ -1,4 +1,7 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 # Note: use lowercase names for the Docker images
 DOCKER_IMAGE="azureiotpcs/pcs-storage-adapter-dotnet"

scripts/docker/run.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 :: Note: use lowercase names for the Docker images

scripts/env-vars-check

@@ -1,4 +1,13 @@
 #!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
+
+# Do not run during Travis CI builds, to avoid failures in case of third party PRs
+if [[ "$TRAVIS_SECURE_ENV_VARS" == "false" ]]; then
+    echo "Info: skipping env vars check during Travis CI build"
+    exit 0
+fi
 
 if [[ -z "$PCS_STORAGEADAPTER_DOCUMENTDB_CONNSTRING" ]]; then
     echo "Error: the PCS_STORAGEADAPTER_DOCUMENTDB_CONNSTRING environment variable is not defined."

scripts/env-vars-check.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 IF "%PCS_STORAGEADAPTER_DOCUMENTDB_CONNSTRING%" == "" (

scripts/env-vars-setup

@@ -1,4 +1,7 @@
 #!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 # Prepare the environment variables used by the application.
 

scripts/env-vars-setup.cmd

@@ -1,3 +1,7 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
+@ECHO off & setlocal enableextensions enabledelayedexpansion
+
 :: Prepare the environment variables used by the application.
 
 :: DocumentDb connection string

scripts/git/fix-perms.sh

@@ -1,4 +1,7 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 # Sometimes when creating bash scripts in Windows, bash scripts will not have
 # the +x flag carried over to Linux/MacOS. This script should help setting the

scripts/git/pre-commit-runner-no-sandbox.sh

@@ -1,5 +1,6 @@
 #!/usr/bin/env bash
-
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
 set -e
 
 # Path relative to .git/hooks/

scripts/git/pre-commit-runner-with-sandbox.sh

@@ -1,5 +1,6 @@
 #!/usr/bin/env bash
-
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
 set -e
 
 # Path relative to .git/hooks/

scripts/git/pre-commit.sh

@@ -1,6 +1,8 @@
 #!/usr/bin/env bash
-
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
 set -e
+
 APP_HOME="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && cd .. && cd .. && pwd )/"
 source "$APP_HOME/scripts/git/.functions.sh"
 

scripts/git/setup

@@ -1,4 +1,7 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 APP_HOME="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && cd .. && cd .. && pwd )/"
 

scripts/git/setup.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 :: strlen("\scripts\git\") => 13

scripts/run

@@ -1,4 +1,7 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 # Usage:
 # Run the service in the local environment:  ./scripts/run

scripts/run.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 :: Usage:

scripts/travis

@@ -1,4 +1,7 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+# Copyright (c) Microsoft. All rights reserved.
+# Note: Windows Bash doesn't support shebang extra params
+set -e
 
 # Example usage (see "travis help" for more information):
 #  travis help

scripts/travis.cmd

@@ -1,3 +1,5 @@
+:: Copyright (c) Microsoft. All rights reserved.
+
 @ECHO off & setlocal enableextensions enabledelayedexpansion
 
 :: Example usage (see "travis help" for more information: