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

Add support for aarch64 #522

Closed
wants to merge 29 commits into from
Closed

Add support for aarch64 #522

wants to merge 29 commits into from

Conversation

mglazer
Copy link

@mglazer mglazer commented Aug 11, 2022

Before this PR

After this PR

==COMMIT_MSG==
Add support for aarch64

This causes a breaking change for any direct consumers as the
coordinates for the osx bundle will be changing such that there
is now a:

osx-x86_64 package

and

osx-aarch64 package

This adds a bit more safety to ensure that consumers don't inadvertently
grab the incorrect package.

Also bump the rust version to 1.63.0 which is the latest version, just for good measure.
==COMMIT_MSG==

Possible downsides?

This causes a breaking change for any direct consumers as the
coordinates for the osx bundle will be changing such that there
is now a:

osx-x86_64 package

and

osx-aarch64 package

This adds a bit more safety to ensure that consumers don't inadvertently
grab the incorrect package.
@changelog-app
Copy link

changelog-app bot commented Aug 11, 2022

Generate changelog in changelog/@unreleased

Type
See change types. Select one:

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Add support for aarch64

This causes a breaking change for any direct consumers as the
coordinates for the osx bundle will be changing such that there
is now a:

osx-x86_64 package

and

osx-aarch64 package

This adds a bit more safety to ensure that consumers don't inadvertently
grab the incorrect package.

Also bump the rust version to 1.54.0. Versions prior to 1.49.0 do not have support for aarch64. I arbitrarily picked 1.54.0 even though 1.63.0 was just published since I didn't want to go too bleeding edge.

Check the box to generate changelog(s)

  • Generate changelog entry

@policy-bot policy-bot bot requested a review from xRuiAlves August 11, 2022 21:03
@mglazer mglazer requested a review from carterkozak August 11, 2022 22:47
futures = "0.1.15"
hyper = "0.12.2"
hyper-openssl = "0.6"
http-zipkin = "0.1"
lazy_static = "1.0"
log = "0.4"
mime = "0.3"
openssl = "0.10"
openssl = { version = "0.10", features = ["vendored"] }
Copy link
Author

Choose a reason for hiding this comment

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

This little gem is necessary for cross compilation.

@@ -9,15 +9,15 @@ license-file = "../LICENSE"
base64 = "0.9"
bytes = "0.4"
crossbeam = "0.3"
flate2 = "1.0"
flate2 = "1.0.24"
Copy link
Author

Choose a reason for hiding this comment

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

This bumps us to a version which has a native miniz implementation rather than the C implementation which fails to link.

@@ -1,1863 +1,1893 @@
# This file is automatically @generated by Cargo.
# It is not intended for manual editing.
version = 3
Copy link
Author

Choose a reason for hiding this comment

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

I'll be honest...I don't know enough Rust to know if it's fine that this changed as much as it did.

Mike Glazer added 8 commits August 12, 2022 07:14
…rification into feature/add-aarch-64

* 'feature/add-aarch-64' of github.com:palantir/conjure-verification:
  Add generated changelog entries
@xRuiAlves
Copy link

Hi friends! Do you think you could remove me as reviewer from this one? Don't have the permission to do so :D
(cc @carterkozak )

@carterkozak carterkozak removed the request for review from xRuiAlves November 25, 2022 14:29
@mglazer mglazer closed this Nov 26, 2022
@mglazer
Copy link
Author

mglazer commented Nov 26, 2022

This is pretty beyond repair, I spent way too much time trying to fix it, rust diverged too much from when this repo was originally crafted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants