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

Fix make install command #2

Merged
merged 1 commit into from
Apr 20, 2018
Merged

Fix make install command #2

merged 1 commit into from
Apr 20, 2018

Conversation

pchaigno
Copy link
Contributor

I had to change the following to be able to run make install. Not sure whether this was working with a previous version of npm or if it's an oversight, so I'm opening this pull request (sorry, I really like pull requests :p) for confirmation.

The npm command to install packages was missing an install subcommand.
@Alhadis
Copy link
Owner

Alhadis commented Apr 20, 2018

Oops. 😅Think it's just an oversight. I remember writing npm i -g (i being its lazy shorthand for install), but I was juggling syntax around to get it to shut the fuck up, and probably erased the subcommand without noticing. 😓

(NPM is everything about Unix done wrong. Why the hell does it log to both standard output AND standard error at the same time? 😕)

@Alhadis Alhadis merged commit f3e962b into master Apr 20, 2018
@pchaigno pchaigno deleted the fix-makefile branch April 20, 2018 15:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants