Skip to content

Commit

Permalink
Remove unused variables
Browse files Browse the repository at this point in the history
Discovered this when running the product construction build for core-setup.  REPO_PASS wasn't able to be set because the backing secret had been disabled in KV.  Looked through its usage in core-setup and saw no uses.  I went through the same environment block in the docker configuration run and eliminated anything without a use.


Commit migrated from dotnet/core-setup@3807ee7
  • Loading branch information
mmitche committed Mar 21, 2018
1 parent 7c98286 commit ce6785a
Show file tree
Hide file tree
Showing 7 changed files with 0 additions and 43 deletions.
7 changes: 0 additions & 7 deletions src/installer/buildpipeline/Core-Setup-Linux-Arm-BT.json
Original file line number Diff line number Diff line change
Expand Up @@ -292,9 +292,6 @@
"value": "-ConfigurationGroup=$(BuildConfiguration) $(PB_AdditionalBuildArguments)",
"allowOverride": true
},
"CONNECTION_STRING": {
"value": "PassedViaPipeBuild"
},
"DEB_REPO_PASSWORD": {
"value": null,
"isSecret": true
Expand All @@ -315,10 +312,6 @@
"value": null,
"isSecret": true
},
"PUBLISH_TO_AZURE_BLOB": {
"value": "true",
"allowOverride": true
},
"DOTNET_BUILD_CONTAINER_TAG": {
"value": "core-setup-$(PB_DockerOS)-$(Build.BuildId)"
},
Expand Down
4 changes: 0 additions & 4 deletions src/installer/buildpipeline/Core-Setup-Linux-BT.json
Original file line number Diff line number Diff line change
Expand Up @@ -995,10 +995,6 @@
"value": null,
"isSecret": true
},
"PUBLISH_TO_AZURE_BLOB": {
"value": "true",
"allowOverride": true
},
"DOTNET_BUILD_CONTAINER_TAG": {
"value": "core-setup-$(PB_DockerOS)-$(Build.BuildId)"
},
Expand Down
7 changes: 0 additions & 7 deletions src/installer/buildpipeline/Core-Setup-OSX-BT.json
Original file line number Diff line number Diff line change
Expand Up @@ -163,13 +163,6 @@
"value": "-ConfigurationGroup=$(BuildConfiguration) $(PB_AdditionalBuildArguments)",
"allowOverride": true
},
"CONNECTION_STRING": {
"value": "PassedViaPipeBuild"
},
"PUBLISH_TO_AZURE_BLOB": {
"value": "true",
"allowOverride": true
},
"GITHUB_PASSWORD": {
"value": "PassedViaPipeBuild"
},
Expand Down
6 changes: 0 additions & 6 deletions src/installer/buildpipeline/Core-Setup-Windows-Arm-BT.json
Original file line number Diff line number Diff line change
Expand Up @@ -408,12 +408,6 @@
"STORAGE_CONTAINER": {
"value": "dotnet"
},
"CONNECTION_STRING": {
"value": "PassedViaPipeBuild"
},
"PUBLISH_TO_AZURE_BLOB": {
"value": "true"
},
"GITHUB_PASSWORD": {
"value": "PassedViaPipeBuild"
},
Expand Down
6 changes: 0 additions & 6 deletions src/installer/buildpipeline/Core-Setup-Windows-BT.json
Original file line number Diff line number Diff line change
Expand Up @@ -608,9 +608,6 @@
"CertificateId": {
"value": "400"
},
"CONNECTION_STRING": {
"value": "PassedViaPipeBuild"
},
"COREHOST_TRACE": {
"value": "0",
"allowOverride": true
Expand Down Expand Up @@ -700,9 +697,6 @@
"PUBLISH_RID_AGNOSTIC_PACKAGES": {
"value": "true"
},
"PUBLISH_TO_AZURE_BLOB": {
"value": "true"
},
"SourceVersion": {
"value": "HEAD"
},
Expand Down
3 changes: 0 additions & 3 deletions tools-local/scripts/common/_common.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,6 @@ COMMONDIR="$( cd -P "$( dirname "$COMMONSOURCE" )" && pwd )"

source "$COMMONDIR/_prettyprint.sh"

# Other variables are set by the outer build script
export CHANNEL=$RELEASE_SUFFIX

[ -z "$DOTNET_INSTALL_DIR" ] && export DOTNET_INSTALL_DIR=$REPOROOT/.dotnet_stage0/$RID
[ -z "$DOTNET_CLI_VERSION" ] && export DOTNET_CLI_VERSION=0.1.0.0
[ -z "$DOTNET_ON_PATH" ] && export DOTNET_ON_PATH=$STAGE2_DIR && export PATH=$STAGE2_DIR/bin:$PATH
Expand Down
10 changes: 0 additions & 10 deletions tools-local/scripts/dockerrun.sh
Original file line number Diff line number Diff line change
Expand Up @@ -99,16 +99,6 @@ echo "Using code from: $DOCKER_HOST_SHARE_DIR"
docker run $INTERACTIVE -t --rm --sig-proxy=true \
--name $DOTNET_BUILD_CONTAINER_NAME \
-v $DOCKER_HOST_SHARE_DIR:/opt/code \
-e CHANNEL \
-e CONNECTION_STRING \
-e REPO_ID \
-e REPO_USER \
-e REPO_PASS \
-e REPO_SERVER \
-e DOTNET_BUILD_SKIP_CROSSGEN \
-e PUBLISH_TO_AZURE_BLOB \
-e DOCKER_HUB_REPO \
-e DOCKER_HUB_TRIGGER_TOKEN \
-e NUGET_FEED_URL \
-e NUGET_SYMBOLS_FEED_URL \
-e NUGET_API_KEY \
Expand Down

0 comments on commit ce6785a

Please sign in to comment.