Skip to content

[ci] Don't use the short url for install script #28591

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

Merged
merged 1 commit into from
Mar 24, 2025
Merged

Conversation

rmarinho
Copy link
Member

Description of Change

We are hitting some issues with the url for downloading the dotnet install script . Trying to see if this helps.

@rmarinho rmarinho added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Mar 24, 2025
@rmarinho rmarinho requested a review from mattleibow March 24, 2025 15:19
@rmarinho rmarinho requested a review from a team as a code owner March 24, 2025 15:19
@rmarinho rmarinho requested a review from PureWeen March 24, 2025 15:19
@github-project-automation github-project-automation bot moved this from Todo to Approved in MAUI SDK Ongoing Mar 24, 2025
@rmarinho rmarinho merged commit a0183dd into main Mar 24, 2025
10 of 98 checks passed
@rmarinho rmarinho deleted the fix-script-url branch March 24, 2025 15:36
@github-project-automation github-project-automation bot moved this from Approved to Done in MAUI SDK Ongoing Mar 24, 2025
@rmarinho
Copy link
Member Author

/backport to inflight/current

Copy link
Contributor

Started backporting to inflight/current: https://github.com/dotnet/maui/actions/runs/14039450256

@rmarinho
Copy link
Member Author

/backport to inflight/candidate

Copy link
Contributor

Started backporting to inflight/candidate: https://github.com/dotnet/maui/actions/runs/14039462744

@rmarinho
Copy link
Member Author

/backport to release/9.0.1xx-sr5

@rmarinho
Copy link
Member Author

/backport to release/9.0.1xx-sr6

Copy link
Contributor

Started backporting to release/9.0.1xx-sr5: https://github.com/dotnet/maui/actions/runs/14062625477

Copy link
Contributor

Started backporting to release/9.0.1xx-sr6: https://github.com/dotnet/maui/actions/runs/14062629602

Copy link
Contributor

@rmarinho an error occurred while backporting to "release/9.0.1xx-sr6", please check the run log for details!

Error: The specified backport target branch "release/9.0.1xx-sr6" wasn't found in the repo.

@rmarinho
Copy link
Member Author

/backport to release/9.0.1xx-sr4

Copy link
Contributor

Started backporting to release/9.0.1xx-sr4: https://github.com/dotnet/maui/actions/runs/14062691188

Copy link
Contributor

@rmarinho backporting to "release/9.0.1xx-sr4" failed, the patch most likely resulted in conflicts:

$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch

Applying: [ci] Don't use the short url for install script
Using index info to reconstruct a base tree...
M	src/DotNet/DotNet.csproj
Falling back to patching base and 3-way merge...
Auto-merging src/DotNet/DotNet.csproj
CONFLICT (content): Merge conflict in src/DotNet/DotNet.csproj
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 [ci] Don't use the short url for install script
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

@jonathanpeppers
Copy link
Member

@rmarinho can we get this one on the net10.0 branch? It is causing our Xamarin.Android-PR (MAUI Tests MAUI Integration) lane to fail on all PRs:

@github-actions github-actions bot locked and limited conversation to collaborators Apr 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants