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

Installation error - make command #56

Closed
DomTranch opened this issue Jun 10, 2024 · 3 comments · Fixed by #58
Closed

Installation error - make command #56

DomTranch opened this issue Jun 10, 2024 · 3 comments · Fixed by #58
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@DomTranch
Copy link

Hello,
I am trying to install openhue-cli to try and control a standard Phillips Hue bridge with more control than is given in the standard app. I am currently trying to install on a Kali Linux machine and am running into the following error after following the tutorial:
image
the [Makefile:30: build] Error 127 output was not initally helpful for troubleshooting. Additionally, based on what the help command is giving me, I should expect an executable in the ./dist folder, but that's not in the current directory. Is there anything that I missed in downloading that would alleviate the issue? Is there something else that I'm missing in the instructions? Any guidance would be helpful.

@thibauult
Copy link
Contributor

thibauult commented Jun 13, 2024

Hi @DomTranch, could you please verify if you've installed GoReleaser?


GoReleaser is indeed marked as optional in the Prerequisites section, I should perhaps mark it as required.

@thibauult thibauult added the documentation Improvements or additions to documentation label Jun 13, 2024
@DomTranch
Copy link
Author

DomTranch commented Jun 19, 2024

Yes, that indeed fixed it. Thank you! For anyone else, these are the specific lines that I ran (and apologies for any command line issues, I can't GitHub well):

echo 'deb [trusted=yes] https://repo.goreleaser.com/apt/ /' | sudo tee /etc/apt/sources.list.d/goreleaser.list
sudo apt update
sudo apt install goreleaser

After that,

make build

ran without error. Definitely recommend GoReleaser as a required tool.

@thibauult thibauult linked a pull request Jun 19, 2024 that will close this issue
thibauult added a commit that referenced this issue Jun 19, 2024
If GoReleaser hasn´t been installed, the `make build` command will
produce the following error message:
```
make build
GoReleaser is not installed (https://goreleaser.com/install)
make: *** [Makefile:68: _check-goreleaser-installation] Error 1
``` 

Fixes #56
@thibauult
Copy link
Contributor

Thanks @DomTranch for confirming that it is now working well.

Documentation has been fixed as part of #58, and I also introduced an additional check in the Makefile to verify that GoReleaser in available before running the make build command.

@thibauult thibauult self-assigned this Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants