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 macOS arm64 binary #186

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jansedivy
Copy link

Hello!

I noticed facebook/flow has been publishing macOS arm64 binaries since February 17th based on this commit facebook/flow@0c8fcd9.

This PR adds macOS arm64 binary to the flow-bin export.

@jansedivy jansedivy force-pushed the jansedivy-add-macos-arch64 branch from fa59719 to 51efb43 Compare May 11, 2023 10:29
@mroch
Copy link
Contributor

mroch commented May 16, 2023

thanks for the PR. the reason we haven't done this yet is concern over the ever-growing size of this package... already 143MB before this change!

my current thinking is that we should publish per-platform packages like [email protected] using os and cpu package.json fields, and have flow-bin depend on them all as optionalDependencies. only the one that supports your machine will get installed.

or is it fine and we should just accept the nearly 200mb packages? :)

@nifgraup
Copy link

Some ideas on how to reduce size:

  1. strip debug symbols
  2. build Flow with https://github.com/dinosaure/esperanto
  3. use optionalDependencies for each binary like like the esbuild package

(mentioning facebook/flow#8664 to link issues together)

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