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.Tasks] _UpdateAndroidResgen is throwing new APT0000 errors in d15-4 #769

Merged
merged 1 commit into from
Aug 23, 2017

Conversation

dellis1972
Copy link
Contributor

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=58889
Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=58890

Commit 0667a2b and ea6b9b4 added support for short path names.
However during the rework we added code to replace

`library_project_imports/`

with an Empty string. Some of the Nuget Support packages seem
to include a __AndroidLibraryProjects__.zip which uses
windows path sepatators rather than unix ones. So the code
to strip off the library_project_imports/ does not work.
We end up with a directory structure like

__library_projects__/[dllname]/library_project_imports/library_project_imports

which a) does not get picked up by our tasks and b) takes up
a bunch of additional characters in our already long path..

The result is the two errors above.. one to do with missing resources
the other a PathToLong error.

Because the niget packages have already shipped and will be in use,
we need to handle BOTH cases.

…T0000 errors in d15-4

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=58889
Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=58890

Commit 0667a2b and ea6b9b4 added support for short path names.
However during the rework we added code to replace

	`library_project_imports/`

with an Empty string. Some of the Nuget Support packages seem
to include a `__AndroidLibraryProjects__.zip` which uses
windows path sepatators rather than unix ones. So the code
to strip off the `library_project_imports/` does not work.
We end up with a directory structure like

	__library_projects__/[dllname]/library_project_imports/library_project_imports

which a) does not get picked up by our tasks and b) takes up
a bunch of additional characters in our already long path..

The result is the two errors above.. one to do with missing resources
the other a PathToLong error.

Because the niget packages have already shipped and will be in use,
we need to handle BOTH cases.
@jonpryor
Copy link
Member

Looks like the macOS+xbuild failure is due to a failure uploading build artifacts to Azure:

MicrosoftAzureStorage - Uploading files to Microsoft Azure
ERROR: MicrosoftAzureStorage - Error occurred while uploading to Azure - xamjenkinsartifact
com.microsoftopentechnologies.windowsazurestorage.exceptions.WAStorageException: remote file operation failed: /Users/builder/jenkins/workspace/xamarin-android-pr-builder at hudson.remoting.Channel@25786a4b:JNLP4-connect connection from 167.220.149.22/167.220.149.22:65511: hudson.remoting.ChannelClosedException: channel is already closed
	at com.microsoftopentechnologies.windowsazurestorage.WAStorageClient.upload(WAStorageClient.java:372)

Merging anyway.

@jonpryor jonpryor merged commit 88936a2 into dotnet:master Aug 23, 2017
jonpryor pushed a commit to jonpryor/xamarin-android that referenced this pull request Sep 6, 2017
…T0000 errors in d15-4 (dotnet#769)

Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=58889
Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=58890

Commit 0667a2b and ea6b9b4 added support for short path names.
However during the rework we added code to replace

	library_project_imports/

with an empty string. Some of the Nuget Support packages seem
to include a `__AndroidLibraryProjects__.zip` which uses
windows path sepatators rather than unix ones. So the code
to strip off the `library_project_imports/` does not work.
We end up with a directory structure like

	__library_projects__/[dllname]/library_project_imports/library_project_imports

which a) does not get picked up by our tasks and b) takes up
a bunch of additional characters in our already long path..

The result is the two errors above... one to do with missing resources
the other a PathToLong error.

Because the nuget packages have already shipped and will be in use,
we need to handle BOTH cases.
jonpryor added a commit to jonpryor/xamarin-android that referenced this pull request Jan 4, 2021
Changes: dotnet/java-interop@2f62ffd...7574f16

  * dotnet/java-interop@7574f166: [generator] Add `generator --with-javadoc-xml=FILE` support. (dotnet#687)
  * dotnet/java-interop@7d197f17: Refactor Crc64 type (dotnet#769)
  * dotnet/java-interop@876442f4: [generator] Fix MSBuild warning/error format for Visual Studio (dotnet#765)
  * dotnet/java-interop@3f6cf72b: [.Localization, .Cecil, .Diagnostics, .Generator] $(Nullable)=enable (dotnet#746)

dotnet/java-interop#687 was merged.
@github-actions github-actions bot locked and limited conversation to collaborators Feb 5, 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.

3 participants