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

[BUG] Local-user install instruction doesn't work #53

Closed
mks-h opened this issue Sep 10, 2024 · 3 comments · Fixed by #47
Closed

[BUG] Local-user install instruction doesn't work #53

mks-h opened this issue Sep 10, 2024 · 3 comments · Fixed by #47
Labels
bug Something isn't working

Comments

@mks-h
Copy link
Member

mks-h commented Sep 10, 2024

The "local-user install" new instruction that just got merged in amber-lang/amber#473 doesn't actually work.

The previous installation instruction with curl -s "URL" | /bin/bash -s -- --user worked and still works for me on Fedora. The new bash -- --user <(curl -s "URL") doesn't work, and gives the following error:

bash: --user: No such file or directory

This only applies to the local-user install, not the system-wide install.

@mks-h mks-h added the bug Something isn't working label Sep 10, 2024
@Mte90
Copy link
Member

Mte90 commented Sep 10, 2024

It is needed that section?
I mean is enough that the script is executed by the specific user, also --user isn't a bash parameter.

@Mte90
Copy link
Member

Mte90 commented Oct 2, 2024

So the right command is:

bash -- <(curl -s "https://raw.githubusercontent.com/amber-lang/amber/master/setup/install.sh") --user 

I added that in this pr for the doc website #47

@Mte90 Mte90 linked a pull request Oct 2, 2024 that will close this issue
@Ph0enixKM Ph0enixKM transferred this issue from amber-lang/amber Oct 4, 2024
@Ph0enixKM
Copy link
Member

I moved this issue to the correct repository

@Mte90 Mte90 closed this as completed in #47 Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants