Skip to content

Loading…

Better signing #72

Closed
wants to merge 1 commit into from

1 participant

@vchekan

Hi,
It is not very convenient to sign assembly as it is implemented now. User has to define PrivateKeyPath variable and pass it to the Msbuild or hack CommonProperties.target xml file.

The proposed patch looks up a file with predefined name "private.snk" and if it exists, use it.
It makes compilation with your own signature as simple as putting a signature file into root folder and renaming it to "private.snk".

@vchekan

Will refactor into a separate branch and re-issue pull request.

@vchekan vchekan closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jul 22, 2011
  1. @vchekan
Showing with 6 additions and 0 deletions.
  1. +6 −0 build/CommonProperties.targets
View
6 build/CommonProperties.targets
@@ -41,4 +41,10 @@
<AssemblyOriginatorKeyFile></AssemblyOriginatorKeyFile>
<DelaySign>false</DelaySign>
</PropertyGroup>
+
+ <!-- if private signature is supplied, use it regarless other settings -->
+ <PropertyGroup Condition="Exists('..\private.snk')">
+ <AssemblyOriginatorKeyFile>..\private.snk</AssemblyOriginatorKeyFile>
+ <DelaySign>false</DelaySign>
+ </PropertyGroup>
</Project>
Something went wrong with that request. Please try again.