Closed Bug 1721477 Opened 3 years ago Closed 2 years ago

Linux build instructions suggest bad modification to .bashrc

Categories

(Firefox Build System :: Build Documentation, defect)

All
Linux
defect

Tracking

(firefox111 fixed)

RESOLVED FIXED
111 Branch
Tracking Status
firefox111 --- fixed

People

(Reporter: mstange, Assigned: glandium)

Details

Attachments

(1 file)

This bug is about the suggestion in the "Note" box in the "Install Mercurial" section:

https://firefox-source-docs.mozilla.org/setup/linux_build.html?highlight=linux%20build#install-mercurial

Once pip has installed mercurial, on a fresh Ubuntu install, hg version doesn't work because the Python packages aren't in the path. The Note box handles this case, however, this suggestion is no good:

# If you're using bash
echo "export PATH=\"$(python3 -m site --user-base)/bin:$PATH\"" >> ~/.bashrc

The $PATH variable is resolved, and what ends up in the .bashrc file no longer has the dollar sign. It has the fully resolved PATH at the time the echo command was invoked.

I think this needs a backslash before the dollar sign.

I'm not sure if the same is needed for zsh.

Assignee: nobody → mh+mozilla
Status: NEW → ASSIGNED
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/5937b6071540
Fix documentation suggestion for PATH augmentation. r=firefox-build-system-reviewers,andi
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: