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

Remove fuchsia support from tools. #123040

Open
chinmaygarde opened this issue Mar 20, 2023 · 3 comments
Open

Remove fuchsia support from tools. #123040

chinmaygarde opened this issue Mar 20, 2023 · 3 comments
Labels
engine flutter/engine repository. See also e: labels. P2 Important issues not at the top of the work list platform-fuchsia Fuchsia code specifically team-engine Owned by Engine team triaged-engine Triaged by Engine team

Comments

@chinmaygarde
Copy link
Member

Just a stub for issue discussed in triage. Importantly, the device discovery tools.

@chinmaygarde chinmaygarde added engine flutter/engine repository. See also e: labels. platform-fuchsia Fuchsia code specifically labels Mar 20, 2023
@chinmaygarde
Copy link
Member Author

cc @jonahwilliams

@jonahwilliams jonahwilliams self-assigned this Mar 21, 2023
@chinmaygarde chinmaygarde added waiting for PR to land (fixed) A fix is in flight P2 Important issues not at the top of the work list labels Mar 27, 2023
@ChristianKleineidam
Copy link

What's the motivation behind removing the fuchsia support from tools?

@chinmaygarde
Copy link
Member Author

There are already bespoke processes in place for existing Fuchsia workflows and this tools support isn't used all that often and isn't well tested on all host platforms. Specifically stuff like the device discovery tools were causing issues for iOS and Android developers.

Past the device discovery, attaching to any remote Flutter process is now better supported because we are better at custom embedder workflows.

So, a combination of lack of use due to supported alternatives and maintenance burden.

Is this going to break your workflow? If so, chime in because we were not aware of this being a breaking change for anyone. But we could be mistaken and may need to hold and consider alternatives.

@flutter-triage-bot flutter-triage-bot bot added team-engine Owned by Engine team triaged-engine Triaged by Engine team labels Jul 8, 2023
@jonahwilliams jonahwilliams removed the waiting for PR to land (fixed) A fix is in flight label Aug 1, 2023
@jonahwilliams jonahwilliams removed their assignment Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engine flutter/engine repository. See also e: labels. P2 Important issues not at the top of the work list platform-fuchsia Fuchsia code specifically team-engine Owned by Engine team triaged-engine Triaged by Engine team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants