Skip to content

Enable arm64 build for Arrow#21467

Merged
vicroms merged 3 commits intomicrosoft:masterfrom
jgiannuzzi:arrow-arm64
Nov 19, 2021
Merged

Enable arm64 build for Arrow#21467
vicroms merged 3 commits intomicrosoft:masterfrom
jgiannuzzi:arrow-arm64

Conversation

@jgiannuzzi
Copy link
Copy Markdown
Contributor

@jgiannuzzi jgiannuzzi commented Nov 16, 2021

Describe the pull request

  • What does your PR fix?

    This PR adds support for building Arrow for arm64, and for cross-compilation to work (e.g. building arm64-osx from a x64-osx host). Building on arm64-windows is still not supported and would require a patch to Arrow (maybe for a future PR?).

  • Which triplets are supported/not supported? Have you updated the CI baseline?

    Adding non-Windows arm64 triplets to the current list (all x64 triplets).
    No change to the CI baseline (The non-Windows arm64 triplets are community supported anyway).

  • Does your PR follow the maintainer guide?

    Yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    Yes

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout ba383ed2332f42593a30dd3f35b09f4a59d31d91 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/a-/arrow.json b/versions/a-/arrow.json
index af8c5e2..deb3acd 100644
--- a/versions/a-/arrow.json
+++ b/versions/a-/arrow.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "5e55365e8c85eb0d1a0aeb230c88179cf6c15141",
+      "git-tree": "bb1f4493a97f3e06a50b33d1669c3730095ab7a4",
       "version": "5.0.0",
       "port-version": 2
     },

You have modified or added at least one portfile where deprecated functions are used.

Details

If you feel able to do so, please consider migrating them to the new functions:
vcpkg_install_cmake -> vcpkg_cmake_install (from port vcpkg-cmake)
vcpkg_build_cmake -> vcpkg_cmake_build (from port vcpkg-cmake)
vcpkg_configure_cmake -> vcpkg_cmake_configure (Please remove the option PREFER_NINJA) (from port vcpkg-cmake)
vcpkg_fixup_cmake_targets -> vcpkg_cmake_config_fixup (from port vcpkg-cmake-config)

In the ports that use the new function, you have to add the corresponding dependencies:

{
  "name": "vcpkg-cmake",
  "host": true
},
{
  "name": "vcpkg-cmake-config",
  "host": true
}

The following files are affected:

  • ports/arrow/portfile.cmake

@jgiannuzzi
Copy link
Copy Markdown
Contributor Author

How can I tell the CI to build the arrow port on arm64-windows?

@jgiannuzzi jgiannuzzi marked this pull request as ready for review November 16, 2021 17:35
@jgiannuzzi jgiannuzzi marked this pull request as draft November 16, 2021 19:03
@jgiannuzzi jgiannuzzi force-pushed the arrow-arm64 branch 2 times, most recently from 5ce11ec to 779fe15 Compare November 16, 2021 19:36
@jgiannuzzi jgiannuzzi marked this pull request as ready for review November 16, 2021 19:58
@JackBoosY JackBoosY added category:community-triplet A PR or issue related to community triplets not officially validated by the vcpkg team. category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist labels Nov 17, 2021
@JackBoosY
Copy link
Copy Markdown
Contributor

LGTM.

@JackBoosY JackBoosY added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Nov 18, 2021
@vicroms vicroms merged commit 77c4e0c into microsoft:master Nov 19, 2021
@jgiannuzzi jgiannuzzi deleted the arrow-arm64 branch November 19, 2021 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:community-triplet A PR or issue related to community triplets not officially validated by the vcpkg team. category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants