Skip to content

[Dockerfile] [Fix] add missing leading slash in path #3195

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
merged 1 commit into from
Sep 18, 2023
Merged

[Dockerfile] [Fix] add missing leading slash in path #3195

merged 1 commit into from
Sep 18, 2023

Conversation

namcxn
Copy link
Contributor

@namcxn namcxn commented Sep 18, 2023

Bug: missing /home

> [18/24] RUN chown nvm:nvm -R "home/nvm/.nvm":
0.467 chown: cannot access 'home/nvm/.nvm': No such file or directory
------
dockerfile:90

ljharb
ljharb approved these changes Sep 18, 2023
View reviewed changes
Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch

@ljharb
Copy link
Member

ljharb commented Sep 18, 2023

cc @PeterDaveHello

@ljharb ljharb changed the title fix: missing "/" no such file or directory [Dockerfile] [Fix] add missing leading slash in path Sep 18, 2023
@ljharb ljharb added the bugs Oh no, something's broken :-( label Sep 18, 2023
@ljharb ljharb merged commit 1de3333 into nvm-sh:master Sep 18, 2023
@PeterDaveHello
Copy link
Collaborator

Thanks!

@namcxn namcxn deleted the patch-1 branch January 18, 2024 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ljharb ljharb ljharb approved these changes

Assignees
No one assigned
Labels
bugs Oh no, something's broken :-(
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@namcxn @ljharb @PeterDaveHello