Skip to content
This repository has been archived by the owner on Oct 20, 2024. It is now read-only.

huffup broken symbolic link #227

Closed
MathisGD opened this issue Nov 4, 2022 · 0 comments · Fixed by #228
Closed

huffup broken symbolic link #227

MathisGD opened this issue Nov 4, 2022 · 0 comments · Fixed by #228

Comments

@MathisGD
Copy link
Contributor

MathisGD commented Nov 4, 2022

huffup, when used with --path to install a local repo does not work for me. The error is:

huffup: command failed: ln -s PWD/target/release/huffc HUFF_BIN_DIR/huffc

because HUFF_BIN_DIR/huffc already exists.

I think that we want to link PWD/target/release/huffc with HUFF_BIN_DIR/huffc and not HUFF_BIN_DIR/huff.

huff-rs/huffup/huffup

Line 78 in e0f640f

ensure ln -s "$PWD/target/release/huffc" "$HUFF_BIN_DIR/huff"

MathisGD added a commit to MathisGD/huff-rs that referenced this issue Nov 4, 2022
@MathisGD
fix: huffup symlink (huff-language#227)
c7f859d
@MathisGD MathisGD mentioned this issue Nov 4, 2022
Merged
Maddiaa0 pushed a commit that referenced this issue Jan 25, 2023
@MathisGD
fix: huffup symlink (#227) (#228)
3883e51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(huffup): symlink MathisGD/huff-rs
2 participants
@Maddiaa0 @MathisGD