Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Strong-name the assemblies #72

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

dfederm
Copy link
Member

@dfederm dfederm commented Feb 28, 2022

This strong-names the assemblies so that applications which are themselves strong-named can use these libraries.

The snk file was generated by running: sn -k StrongName.snk

@JamieMagee
Copy link
Member

JamieMagee commented Mar 29, 2022

@dfederm Are we okay to still merge this, even if we're blocked with upstream dependencies?

EDIT: Nope! Moving this to draft for now.

@JamieMagee JamieMagee marked this pull request as draft March 29, 2022 20:53
@cobya cobya added version:major status:blocked Issue is blocked by another issue or external requirement type:feature Feature (new functionality) labels Jun 14, 2022
@JamieMagee
Copy link
Member

I opened a PR upstream dotnet/Docker.DotNet#567

@JamieMagee
Copy link
Member

PR merged upstream. Waiting for a release now.

@JamieMagee
Copy link
Member

Version 3.125.13 was released upstream with the required strong-naming 🎉

@JamieMagee
Copy link
Member

See #453

@JamieMagee
Copy link
Member

@dfederm can you rebase this branch please? I don't have permissions to push to your fork, but I've created a branch you can reset to: strong-naming.

We're getting closer, but the build still fails due to a lack of strong named assemblies for Tomlyn and Valleysoft.DockerfileModel. I've created issues upstream:

I'll also create pull requests when I have time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:blocked Issue is blocked by another issue or external requirement type:feature Feature (new functionality) version:major
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants