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

Automatically choose the correct architecture #59

Closed
nex3 opened this issue Aug 3, 2022 · 4 comments · Fixed by #77
Closed

Automatically choose the correct architecture #59

nex3 opened this issue Aug 3, 2022 · 4 comments · Fixed by #77
Labels
type-enhancement A request for a change that isn't a bug

Comments

@nex3
Copy link
Member

nex3 commented Aug 3, 2022

Currently, this action always defaults to installing the x64 Dart binaries, even when running on architectures that aren't compatible with x64 (such as MacOS ARM). It would be nice if it instead defaulted to checking the current architecture and installing the matching binary.

nex3 added a commit to sass/dart-sass-embedded that referenced this issue Aug 3, 2022
nex3 added a commit to sass/dart-sass-embedded that referenced this issue Aug 3, 2022
joshpeterson30489 added a commit to joshpeterson30489/embedded-dart-sass-development that referenced this issue Sep 30, 2022
@devoncarew devoncarew added the type-enhancement A request for a change that isn't a bug label Nov 4, 2022
nex3 added a commit to sass/dart-sass that referenced this issue Mar 6, 2023
nex3 added a commit to sass/dart-sass-embedded that referenced this issue Mar 6, 2023
nex3 added a commit to sass/dart-sass that referenced this issue Mar 6, 2023
nex3 added a commit to sass/dart-sass-embedded that referenced this issue Mar 6, 2023
@nex3
Copy link
Member Author

nex3 commented Mar 10, 2023

It looks like this is still an issue, at least for Mac OS ARM machines: https://github.com/sass/dart-sass/actions/runs/4388953095/jobs/7686128912

jathak added a commit to sass/dart-sass that referenced this issue Mar 10, 2023
@devoncarew
Copy link
Member

Are you using the published version of setup-dart or the latest from head?

@devoncarew
Copy link
Member

We plan to publish ~next week (after a bit of burn-in time). #79 has info on how to use the version from head.

@nex3
Copy link
Member Author

nex3 commented Mar 10, 2023

We're using the @v1 tag. We'll probably wait to update until the stable version is published.

jathak added a commit to sass/dart-sass that referenced this issue Mar 10, 2023
* Revert "Remove workaround for dart-lang/setup-dart#59 (#1904)"

This reverts commit 434f2b9.

* Bump version
jathak added a commit to sass/dart-sass-embedded that referenced this issue Mar 11, 2023
jathak added a commit to sass/dart-sass-embedded that referenced this issue Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-enhancement A request for a change that isn't a bug
Projects
None yet
2 participants