Skip to content

Conversation

@jonpryor
Copy link
Contributor

@jonpryor jonpryor commented Dec 5, 2024

Context: 356485e

Java.Interop.dll assembly version is now 10.0.

With 356485e significantly changing generator output, without a corresponding generator --lang-features or equivalent switch to revert behavior, it feels "safer" to bump the assembly version now to reduce the likelihood of inadvertent assembly mixing.

(Not that I'm arguing that we should have such a flag. We absolutely need to move away from requiring System.Reflection.Emit.)

Context: 356485e

`Java.Interop.dll` assembly version is now 10.0.

With 356485e significantly changing `generator` output, without a
corresponding `generator --lang-features` or equivalent switch to
revert behavior, it feels "safer" to bump the assembly version *now*
to reduce the likelihood of inadvertent assembly mixing.

(Not that I'm arguing that we *should* have such a flag.
We absolutely need to move away from requiring System.Reflection.Emit.)
@jonpryor
Copy link
Contributor Author

jonpryor commented Dec 5, 2024

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jonpryor jonpryor merged commit 2d48efe into main Dec 5, 2024
4 checks passed
@jonpryor jonpryor deleted the dev/jonp/v10 branch December 5, 2024 15:08
jonpryor pushed a commit to dotnet/android that referenced this pull request Dec 6, 2024
Fixes: dotnet/java-interop#1258

Changes: dotnet/java-interop@2440416...2d48efe

  * dotnet/java-interop@2d48efe7: [build] `main` *conceptually* targets .NET 10 (dotnet/java-interop#1283)
  * dotnet/java-interop@356485ee: [generator] Remove `JNINativeWrapper.CreateDelegate` from bindings (dotnet/java-interop#1275)
  * dotnet/java-interop@14f94a57: [Java.Interop.BootstrapTasks] Filter out invalid JDKs (dotnet/java-interop#1278)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@github-actions github-actions bot locked and limited conversation to collaborators Jan 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants