Skip to content

Commit

Permalink
Stop code signing binaries and packages
Browse files Browse the repository at this point in the history
  • Loading branch information
natemcmaster committed Jun 28, 2020
1 parent 59781ae commit 3c9dcee
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 87 deletions.
14 changes: 1 addition & 13 deletions .config/dotnet-tools.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,23 +2,11 @@
"version": 1,
"isRoot": true,
"tools": {
"azuresigntool": {
"version": "2.0.17",
"commands": [
"azuresigntool"
]
},
"nugetkeyvaultsigntool": {
"version": "1.2.28",
"commands": [
"NuGetKeyVaultSignTool"
]
},
"dotnet-reportgenerator-globaltool": {
"version": "4.4.0",
"commands": [
"reportgenerator"
]
}
}
}
}
4 changes: 0 additions & 4 deletions Directory.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,6 @@
<Nullable>annotations</Nullable>
<AssemblyOriginatorKeyFile>$(MSBuildThisFileDirectory)src\StrongName.snk</AssemblyOriginatorKeyFile>
<SignAssembly>true</SignAssembly>
<AzureKeyVaultUrl>https://nmcmaster.vault.azure.net</AzureKeyVaultUrl>
<AzureKeyVaultClientId>6a27a2da-bb78-4baa-bd2b-150fe89ea039</AzureKeyVaultClientId>
<AzureKeyVaultClientSecret>$(KEYVAULT_CLIENT_SECRET)</AzureKeyVaultClientSecret>
<CodeSignCertName>DigiCertCodeSign</CodeSignCertName>

<TreatWarningsAsErrors>true</TreatWarningsAsErrors>
<GenerateFullPaths Condition="'$(TERM_PROGRAM)' == 'vscode'">true</GenerateFullPaths>
Expand Down
3 changes: 0 additions & 3 deletions azure-pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ pr:
- '*'

variables:
- group: AzureKeyVault
- name: BUILD_NUMBER
value: $[counter('buildnumber', 1)]

Expand Down Expand Up @@ -39,8 +38,6 @@ jobs:
packageType: runtime
- powershell: ./build.ps1 -ci
displayName: Invoke build.ps1
env:
KEYVAULT_CLIENT_SECRET: $(kv-access-token)
- task: PublishTestResults@2
displayName: Publish test results
condition: always()
Expand Down
8 changes: 0 additions & 8 deletions build.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@ param(
$Configuration = $null,
[switch]
$ci,
[switch]
$sign,
[Parameter(ValueFromRemainingArguments = $true)]
[string[]]$MSBuildArgs
)
Expand Down Expand Up @@ -39,12 +37,6 @@ if (-not (Test-Path variable:\IsCoreCLR)) {
$IsWindows = $true
}

$CodeSign = $sign -or ($ci -and -not $isPr -and $IsWindows)

if ($CodeSign) {
$MSBuildArgs += '-p:CodeSign=true'
}

$artifacts = "$PSScriptRoot/artifacts/"

Remove-Item -Recurse $artifacts -ErrorAction Ignore
Expand Down
57 changes: 0 additions & 57 deletions src/CodeSign.targets

This file was deleted.

2 changes: 0 additions & 2 deletions src/Directory.Build.targets
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,4 @@
<Message Importance="High" Text="##vso[build.updatebuildnumber]$(PackageVersion)" />
</Target>

<Import Project="CodeSign.targets" />

</Project>

0 comments on commit 3c9dcee

Please sign in to comment.