-
-
Notifications
You must be signed in to change notification settings - Fork 329
Add initial Linux desktop support #2114
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
Conversation
…cated literal operator error
veloce
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes are indeed contained, so we can have that linux platform for dev purposes.
|
Thanks for your review 🙂 |
|
I don't have a linux computer right now to test the platform so I cannot add it. Will do later or feel free to open a pr with the platform files. |
|
Ok! Thanks for merging @veloce |
PR related to #2
Hey @veloce
Concerning your comment on the issue above, I created a small PR to add initial Linux desktop support. In case you have any comments or suggested changes to make merging more likely, feel free to reach out.
Info
The Linux platform must be created before. I thought you might want to do that by yourself via:
flutter create --platforms=linux .flutter_secure_storage_linuxplugin I forked the exact version which is used in Lichess and only added three necessary changes to let it compile. This is an open upstream issue (see Linux deprecated literal operator juliansteenbakker/flutter_secure_storage#920)