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

Change PATH configuration line on 'macOS install' page #9351

Closed
IvoSorokins opened this issue Sep 5, 2023 · 0 comments · Fixed by #10031
Closed

Change PATH configuration line on 'macOS install' page #9351

IvoSorokins opened this issue Sep 5, 2023 · 0 comments · Fixed by #10031
Assignees
Labels
a.get-started Relates to the Getting Started section of docs.flutter.dev devos.macOS Relates to developing apps on macOS platform e1-hours Effort: < 8 hrs from.page-issue Reported in a reader-filed concern p2-medium Necessary but not urgent concern. Resolve when possible.

Comments

@IvoSorokins
Copy link

Page URL

https://docs.flutter.dev/get-started/install/macos/

Page source

https://github.com/flutter/website/tree/main/src/get-started/install/macos.md

Describe the problem

The use of pwd in the PATH configuration line (export PATH="$PATH:pwd/flutter/bin") is incorrect and unnecessary. The correct way to add a directory to the PATH is to specify the absolute path to that directory directly. The pwd command, which stands for "print working directory," is used to display the current directory, but it's not needed in the PATH configuration.

Expected fix

Rename to: export PATH="$PATH:[PATH_OF_FLUTTER_GIT_DIRECTORY]/bin" as it is under "Update your path" also step 3, using which did not bring any confusion

Additional context

Step: "3. Add the flutter tool to your path:" is under section Get the Flutter SDK

@atsansone atsansone added a.get-started Relates to the Getting Started section of docs.flutter.dev p2-medium Necessary but not urgent concern. Resolve when possible. e1-hours Effort: < 8 hrs devos.macOS Relates to developing apps on macOS platform st.triage.ltw labels Sep 5, 2023
@atsansone atsansone changed the title [PAGE ISSUE]: 'macOS install' Change PATH configuration line on 'macOS install' page Sep 5, 2023
@atsansone atsansone added the from.page-issue Reported in a reader-filed concern label Sep 5, 2023
@atsansone atsansone self-assigned this Jan 11, 2024
atsansone added a commit that referenced this issue Jan 22, 2024
Adds PATH instructions for macOS.
Fixes #9351
Fixes #10015
Fixes #10040 
Fixes #10036

---------

Co-authored-by: John Ryan <ryjohn@google.com>
atsansone added a commit to atsansone/website that referenced this issue Jan 24, 2024
Adds PATH instructions for macOS.
Fixes flutter#9351
Fixes flutter#10015
Fixes flutter#10040 
Fixes flutter#10036

---------

Co-authored-by: John Ryan <ryjohn@google.com>
atsansone added a commit to atsansone/website that referenced this issue Apr 5, 2024
Adds PATH instructions for macOS.
Fixes flutter#9351
Fixes flutter#10015
Fixes flutter#10040 
Fixes flutter#10036

---------

Co-authored-by: John Ryan <ryjohn@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a.get-started Relates to the Getting Started section of docs.flutter.dev devos.macOS Relates to developing apps on macOS platform e1-hours Effort: < 8 hrs from.page-issue Reported in a reader-filed concern p2-medium Necessary but not urgent concern. Resolve when possible.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants