Skip to content

Conversation

@directhex
Copy link
Contributor

The build system dies if xabuild is a symlink pointing to the real xabuild with a relative path instead of an absolute one. The -f flag to readlink handles this.

The build system dies if xabuild is a symlink pointing to the real xabuild with a *relative* path instead of an absolute one. The `-f` flag to readlink handles this.
@jonpryor
Copy link
Contributor

macOS doesn't support readlink -f, and it's moot for use on @directhex's Debian PR, as xabuild will be installed as /usr/bin/xabuild directly.

@jonpryor jonpryor closed this Aug 17, 2017
@jonpryor jonpryor deleted the directhex-patch-1 branch June 11, 2019 21:42
jonpryor added a commit to jonpryor/xamarin-android that referenced this pull request Dec 14, 2020
Changes: dotnet/java-interop@99897b2...2f62ffd

  * dotnet/java-interop@2f62ffd4: [Xamarin.Android.Tools.Bytecode] @not-null androidx.annotation.NonNull (dotnet#759)
  * dotnet/java-interop@7c1ad51c: [java-source-utils] Properly represent unresolved types (dotnet#758)
  * dotnet/java-interop@c1bc5c83: [java-source-utils] Better support orphaned Javadocs (dotnet#757)
  * dotnet/java-interop@0cb8e2d3: [Xamarin.Android.Tools.ApiXmlAdjuster] Use Dictionaries for perf (dotnet#756)
  * dotnet/java-interop@99f89905: [build] Fix for MSBuild 16.8 (dotnet#755)
jonpryor added a commit that referenced this pull request Dec 15, 2020
Changes: dotnet/java-interop@99897b2...2f62ffd

  * dotnet/java-interop@2f62ffd4: [Xamarin.Android.Tools.Bytecode] @not-null androidx.annotation.NonNull (#759)
  * dotnet/java-interop@7c1ad51c: [java-source-utils] Properly represent unresolved types (#758)
  * dotnet/java-interop@c1bc5c83: [java-source-utils] Better support orphaned Javadocs (#757)
  * dotnet/java-interop@0cb8e2d3: [Xamarin.Android.Tools.ApiXmlAdjuster] Use Dictionaries for perf (#756)
  * dotnet/java-interop@99f89905: [build] Fix for MSBuild 16.8 (#755)
@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.

4 participants