-
Notifications
You must be signed in to change notification settings - Fork 28.5k
Building Flutter projects on MacOS fails if FlutterMacOS.framwork is read-only (ie. in a Nix environment) #147142
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
Comments
Thanks for the report and potential fix. I'll keep the issue open since this is related to Nix environment / setup which is external to the framework. |
cc @vashworth |
@asyncmeow You can try if this workaround works for you: #148354 (comment) |
…erMacOS.framework (#148580) Fixes #148354 Fixes #147142 Closes #147144 ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] I updated/added relevant documentation (doc comments with `///`). - [x] I added new tests to check the change I am making, or this PR is [test-exempt]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing.
…erMacOS.framework (flutter#148580) Fixes flutter#148354 Fixes flutter#147142 Closes flutter#147144 ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] I updated/added relevant documentation (doc comments with `///`). - [x] I added new tests to check the change I am making, or this PR is [test-exempt]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing.
…k and FlutterMacOS.framework (#148717) This pull request is created by [automatic cherry pick workflow](https://github.com/flutter/flutter/wiki/Flutter-Cherrypick-Process#automatically-creates-a-cherry-pick-request) Please fill in the form below, and a flutter domain expert will evaluate this cherry pick request. ### Issue Link: What is the link to the issue this cherry-pick is addressing? #148354 #147142 ### Changelog Description: Correct read/write permisions for FlutterMacOS.framework to allow MacOS developers to publish to the app store ### Impact Description: MacOS developers. ### Workaround: Is there a workaround for this issue? Run chmod locally after building #148354 (comment) ### Risk: What is the risk level of this cherry-pick? ### Test Coverage: Are you confident that your fix is well-tested by automated tests? ### Validation Steps: What are the steps to validate that this fix works? 1. `flutter build macos --release` any macos project 2. `ls -la macos/Build/Products/Release/test_create.app/Contents/Frameworks/` 3. Note that the permissions for FlutterMacOS.framework is `drwxr-xr-x`, NOT `dr-------` (note that this is validated to work here #148354 (comment))
This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of |
Steps to reproduce
nix
is installed on your MacOS machinenix-shell -p flutter
to create a shell with Flutter in your environmentflutter build macos
and notice how it failsnix
environment you created in step 2 (exit
)flutter build macos
and notice how it succeedsExpected results
The build should succeed under the nix environment.
Actual results
The build fails on the nix environment due to
FlutterMacOS.framework
being read-only in the Nix store. The Nix store is intentionally read-only, so changingFlutterMacOS.framework
torw
within the Nix store would ruin the point of using Nix here.Code sample
Code sample
As this issue occurs on the brand-new project created by `dart create bug`, I attached that file unmodified:Screenshots or Video
No response
Logs
Logs
Flutter Doctor output
Doctor output
The text was updated successfully, but these errors were encountered: