Skip to content

fix(doctor): add an alternative path for the Dart plugin #163215

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.

Sign up for GitHub

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

Jump to bottom
Merged
auto-submit merged 1 commit into from
Mar 18, 2025
Merged

fix(doctor): add an alternative path for the Dart plugin #163215

auto-submit merged 1 commit into from
Mar 18, 2025

Conversation

@kszczek
Copy link
Contributor

@kszczek kszczek commented Feb 13, 2025

It seems like with some recent plugin/IDE version, the path for the Dart plugin has changed from <base path>/Dart to <base path>/dart, causing flutter doctor to incorrectly assume that the plugin is not installed.

Add dart as an alternative to the existing Dart package name.

Fixes #163214.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Feb 13, 2025
@kszczek kszczek force-pushed the fix-dart-plugin-detection branch from 619d669 to 471dd27 Compare February 13, 2025 18:42
@bkonyi
Copy link
Contributor

bkonyi commented Feb 13, 2025

@jwren can you take a look at this? It seems strange that we'd be seeing lowercase dart in some situations.

@jwren
Copy link
Member

jwren commented Feb 14, 2025

This is surprising, the Dart plugin for IntelliJ has always been "Dart", see the bottom of https://plugins.jetbrains.com/plugin/6351-dart. @alexander-dorosko -- any idea why this might be different for some users?

@kszczek
Copy link
Contributor Author

kszczek commented Feb 14, 2025

see the bottom of https://plugins.jetbrains.com/plugin/6351-dart

I don't think that's relevant. For example, the plugin ID for Flutter is "io.flutter", yet the directory containing it is called "flutter-intellij"

For additional context, this happened after I wiped my desktop, so after a clean OS install, clean Android Studio install and clean plugin install. I don't know if an update would cause this directory to be renamed, but a clean install sure did create it lowercased.

@jwren
Copy link
Member

jwren commented Feb 18, 2025 •
edited
Loading

CC @bkonyi
Thanks @kszczek. I don't have any additional comments or concerns around this PR landing.

@kszczek
Copy link
Contributor Author

kszczek commented Mar 17, 2025

Hi @bkonyi, just checking in to see if there are any updates or if anything else is needed from my side to move this forward. Thanks!

@bkonyi
Copy link
Contributor

bkonyi commented Mar 17, 2025

Sorry for the delay! This LGTM if @jwren has no concerns.

@bkonyi bkonyi self-requested a review March 17, 2025 19:02
@bkonyi bkonyi added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 17, 2025
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Mar 17, 2025
@auto-submit
Copy link
Contributor

auto-submit bot commented Mar 17, 2025

autosubmit label was removed for flutter/flutter/163215, because This PR has not met approval requirements for merging. The PR author is not a member of flutter-hackers and needs 1 more review(s) in order to merge this PR.

  • Merge guidelines: A PR needs at least one approved review if the author is already part of flutter-hackers or two member reviews if the author is not a flutter-hacker before re-applying the autosubmit label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

@bkonyi bkonyi requested a review from jwren March 17, 2025 19:11
@bkonyi
Copy link
Contributor

bkonyi commented Mar 17, 2025

@jwren this needs a second reviewer. Can I get your rubber stamp?

@bkonyi bkonyi added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 17, 2025
@kszczek
fix(doctor): add an alternative path for the Dart plugin
cf7a2a5 
It seems like with some recent plugin/IDE version, the path for the
Dart plugin has changed from `<base path>/Dart` to `<base path>/dart`,
causing `flutter doctor` to incorrectly assume that the plugin is not
installed.

Add `dart` as an alternative to the existing `Dart` package name.

Fixes: flutter#163214
@kszczek kszczek force-pushed the fix-dart-plugin-detection branch from 471dd27 to cf7a2a5 Compare March 18, 2025 08:45
@auto-submit auto-submit bot added this pull request to the merge queue Mar 18, 2025
Merged via the queue into flutter:master with commit 4c35308 Mar 18, 2025
132 checks passed
@flutter-dashboard flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Mar 18, 2025
@kszczek kszczek deleted the fix-dart-plugin-detection branch March 18, 2025 10:09
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 25, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
803f8b3 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 25, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
682c032 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
c78b1a6 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
71c8e49 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
a79a553 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
277ebc3 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
0844559 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
46875bf 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
7bbb109 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
238f018 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
d765d7b 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
a9ededb 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
23f7710 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 28, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
b0b4655 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 28, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
1c75f26 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 28, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
49f3c15 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 28, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
8154cd5 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 29, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
a50d04d 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 29, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
5572a0d 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
c994485 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
3c75e18 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
c5baead 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
88090a0 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
ef09d7e 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
d669293 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
53e28c2 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
e247b3a 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
18fed31 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
fc4a1ec 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
4edf9b0 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
76ec07e 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
b8baf31 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 20, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
43d546f 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 20, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
b1e65a3 
…tter/flutter#163215)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 21, 2025
@engine-flutter-autoroll
4c35308 fix(doctor): add an alternative path for the Dart plugin (flu…
97ab9e2 
…tter/flutter#163215)
romanejaquez pushed a commit to romanejaquez/flutter that referenced this pull request Aug 14, 2025
@kszczek @romanejaquez
fix(doctor): add an alternative path for the Dart plugin (flutter#163215
965b21e 
)

It seems like with some recent plugin/IDE version, the path for the Dart
plugin has changed from `<base path>/Dart` to `<base path>/dart`,
causing `flutter doctor` to incorrectly assume that the plugin is not
installed.

Add `dart` as an alternative to the existing `Dart` package name.

Fixes flutter#163214.

## 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.
- [ ] I updated/added relevant documentation (doc comments with `///`).
- [ ] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [ ] I followed the [breaking change policy] and added [Data Driven
Fixes] where supported.
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview
[Tree Hygiene]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
[test-exempt]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
[Discord]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
[Data Driven Fixes]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jwren jwren jwren approved these changes

@bkonyi bkonyi bkonyi approved these changes

Assignees

No one assigned

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

flutter doctor incorrectly reports the Dart plugin as missing

3 participants

@kszczek @bkonyi @jwren