Skip to content

[release/10.0.1xx] Complete Maven feed backport - #12469

Open
jonathanpeppers wants to merge 5 commits into
release/10.0.1xxfrom
jonathanpeppers-backport-public-maven-feeds
Open

[release/10.0.1xx] Complete Maven feed backport#12469
jonathanpeppers wants to merge 5 commits into
release/10.0.1xxfrom
jonathanpeppers-backport-public-maven-feeds

Conversation

@jonathanpeppers

Copy link
Copy Markdown
Member

Summary

  • backport 1c4d41c09eec so direct Maven mirroring fails when any requested coordinate produces no payloads
  • update external/Java.Interop from 33992194b9373e9322244612c94ed9941b9bc2fd to [release/10.0.1xx] Backport Maven hardening and feed routing java-interop#1498's single release backport commit 7c110bfad3ee0a87f303609426167722ccb4ba77
  • include the release-applicable Java.Interop Maven test and java-source-utils routing that could not be carried by the main-branch subtree patch
  • seed the release JavaParser 3.18.0 dependency graph into dotnet-public-maven for anonymous CI resolution

Backport mapping

Main change Release equivalent
ce636392a229 / #11711, 3c62389651c2 / #12055, Android portions of 466da4a84dc2 / #12199, and adffb38fd731 / #12368 659cd98e7fb9 / #12397
8df1e87799f1 / #12198, 7ace3137a4fa / #12336, c09c89aa1f2e / #12338, 13727e665929 / #12335, and d57b8c42f131 / #12367 352831edcba6 / #12396
approved .NET tool feed portion of fa00357649bb / #11701 dc9a8524138e / #12420
1c4d41c09eec / #12412 24252012aaf6 in this PR
Java.Interop hardening plus release-applicable routing from 466da4a84dc2 dotnet/java-interop#1498, pinned here at 7c110bfad3ee0a87f303609426167722ccb4ba77

The later Dependabot-only Google Maven exposure and unrelated Gradle/dependency version churn are intentionally excluded. The final Java.Interop pin supersedes the earlier hardening-only candidate without amending or force-pushing commits.

Validation

  • Java.Interop.Tools.Maven tests: 106 passed, 0 failed
  • CI-mode java-source-utils jar build resolved through dotnet-public-maven and succeeded after the mirror helper seeded the full transitive graph
  • mirror-dependencies.ps1 parsed successfully with the PowerShell parser
  • Java.Interop final SHA is exactly one commit on top of the release pin
  • git diff --check origin/release/10.0.1xx..HEAD

jonathanpeppers and others added 3 commits August 20, 2026 15:57
## Summary

- return explicit success and failure counts from `Invoke-Mirror`
- evaluate direct Maven mirroring per requested coordinate
- fail `-MavenArtifact` when any coordinate mirrors no payloads

Addresses the `eng/gradle/mirror-dependencies.ps1` feedback from #12397 without modifying the release backport.

## Validation

- parsed `mirror-dependencies.ps1` with the PowerShell parser
- verified a coordinate succeeds when at least one payload returns HTTP 200
- verified all-failed payloads exit with code 1
- verified multiple coordinates exit with code 1 when any coordinate has no successful payload
- `git diff --check`

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
(cherry picked from commit 1c4d41c)
Pin dotnet/java-interop#1495's single release backport commit, which validates Maven coordinates and keeps resolved cache paths contained.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Replace the superseded hardening-only pin with dotnet/java-interop#1498's single release commit, adding approved-feed routing for Java.Interop Maven tests and java-source-utils.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings August 20, 2026 21:12
@jonathanpeppers

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Completes the release-branch Maven feed backport, including mirror completeness validation and related Java.Interop updates.

Changes:

  • Tracks per-coordinate mirror success and failure results.
  • Fails direct mirroring when requested coordinates produce no payloads.
  • Preserves Gradle retry handling with structured results.
  • Updates the Java.Interop release pin and Maven dependency graph.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Backport the Android-side parent POM coordinate fallback from 7da7db7 and cover POMs that inherit groupId and version.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 30d6319e-30d5-4311-831c-58c4a0571403
@jonathanpeppers

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.

Update the gitlink to the current dotnet/java-interop release/10.0.1xx tip containing merged PR #1498.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 30d6319e-30d5-4311-831c-58c4a0571403
@jonathanpeppers
jonathanpeppers marked this pull request as ready for review August 21, 2026 17:08
@jonathanpeppers

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.

@jonathanpeppers jonathanpeppers added the ready-to-review This PR is ready to review/merge, I think any CI failures are just flaky (ignorable). label Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-review This PR is ready to review/merge, I think any CI failures are just flaky (ignorable).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants