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

[Xamarin.Android.Build.Tests] Fix ActionBarSherlock URL #8926

Merged

Conversation

jonpryor
Copy link
Member

@jonpryor jonpryor commented May 6, 2024

On 2024-May-01, xamarin/monodroid-samples was updated so that the main branch no longer contained samples for Classic Xamarin.Android. This in turn meant the removal of the ActionBarSherlock sample.

This change in turn broke xamarin-android unit tests, which attempt to download this URL, which no longer exists:

Oops.

As part of the removal of Classic Xamarin.Android samples, a new monodroid-samples/archived-xamarin branch was created, which is the state of the main branch before all old samples were removed.

Update the BindingBuildTest.RemoveEventHandlerResolution() integration test to use the correct URL for ActionBarSherlock.

On 2024-May-01, xamarin/monodroid-samples was updated so that the
`main` branch no longer contained samples for Classic Xamarin.Android.
This in turn meant the removal of the ActionBarSherlock sample.

This change in turn broke xamarin-android unit tests, which attempt
to download this URL, which no longer exists:

  * <https://github.com/xamarin/monodroid-samples/blob/main/ActionBarSherlock/ActionBarSherlock/Jars/ActionBarSherlock-4.3.1.zip?raw=true>

Oops.

As part of the removal of Classic Xamarin.Android samples, a new
`monodroid-samples/archived-xamarin` branch was created, which is
the state of the `main` branch before all old samples were removed.

Update the `BindingBuildTest.RemoveEventHandlerResolution()`
integration test to use the correct URL for ActionBarSherlock.
Copy link
Member

@jonathanpeppers jonathanpeppers left a comment

Choose a reason for hiding this comment

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

The new URL works for me in a browser, so should fix the test.

@jonathanpeppers
Copy link
Member

The new test is green, going to merge to see if our various release branches can be greener:

image

There are other failures, I think were caused by #8921. I think if a PR is merged between build & test lanes it can cause failures on PRs.

@jonathanpeppers jonathanpeppers merged commit 50419e1 into main May 7, 2024
40 of 48 checks passed
@jonathanpeppers jonathanpeppers deleted the dev/jonp/jonp-fix-RemoveEventHandlerResolution branch May 7, 2024 16:01
jonathanpeppers pushed a commit that referenced this pull request May 7, 2024
On 2024-May-01, xamarin/monodroid-samples was updated so that the
`main` branch no longer contained samples for Classic Xamarin.Android.
This in turn meant the removal of the ActionBarSherlock sample.

This change in turn broke xamarin-android unit tests, which attempt
to download this URL, which no longer exists:

  * <https://github.com/xamarin/monodroid-samples/blob/main/ActionBarSherlock/ActionBarSherlock/Jars/ActionBarSherlock-4.3.1.zip?raw=true>

Oops.

As part of the removal of Classic Xamarin.Android samples, a new
`monodroid-samples/archived-xamarin` branch was created, which is
the state of the `main` branch before all old samples were removed.

Update the `BindingBuildTest.RemoveEventHandlerResolution()`
integration test to use the correct URL for ActionBarSherlock.
jonathanpeppers pushed a commit that referenced this pull request May 7, 2024
On 2024-May-01, xamarin/monodroid-samples was updated so that the
`main` branch no longer contained samples for Classic Xamarin.Android.
This in turn meant the removal of the ActionBarSherlock sample.

This change in turn broke xamarin-android unit tests, which attempt
to download this URL, which no longer exists:

  * <https://github.com/xamarin/monodroid-samples/blob/main/ActionBarSherlock/ActionBarSherlock/Jars/ActionBarSherlock-4.3.1.zip?raw=true>

Oops.

As part of the removal of Classic Xamarin.Android samples, a new
`monodroid-samples/archived-xamarin` branch was created, which is
the state of the `main` branch before all old samples were removed.

Update the `BindingBuildTest.RemoveEventHandlerResolution()`
integration test to use the correct URL for ActionBarSherlock.
grendello added a commit that referenced this pull request May 7, 2024
* release/8.0.2xx:
  [build] use new Mono archive URL (#8931)
  [Xamarin.Android.Build.Tests] Fix ActionBarSherlock URL (#8926)
grendello added a commit that referenced this pull request May 9, 2024
* main:
  [docs] Add "Getting Started" docs (#8934)
  [Xamarin.Android.Build.Tests] Fix ActionBarSherlock URL (#8926)
grendello added a commit that referenced this pull request May 13, 2024
* main:
  Bump to xamarin/java.interop@78d5937 (#8935)
  [docs] Add "Getting Started" docs (#8934)
  [Xamarin.Android.Build.Tests] Fix ActionBarSherlock URL (#8926)
  Update README (#8913)
  Bumps to xamarin/java.interop@4e893bf (#8924)
  Bump to dotnet/installer@fa261b952d 9.0.100-preview.5.24253.16 (#8921)
  [Mono.Android] Dispose of the `RunnableImplementor` on error (#8907)
  Bump NDK to r26d (#8868)
grendello added a commit that referenced this pull request May 13, 2024
* main:
  It's ".NET for Android", not ".NET Android" (#8933)
  Bump to xamarin/java.interop@78d5937 (#8935)
  [docs] Add "Getting Started" docs (#8934)
  [Xamarin.Android.Build.Tests] Fix ActionBarSherlock URL (#8926)
@github-actions github-actions bot locked and limited conversation to collaborators Jun 8, 2024
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.

None yet

2 participants