Skip to content

Conversation

@andy5995
Copy link

(fixes #2)

After I made the change to package.json for this commit, I still got an
error when I ran `npm install`. I had to run `make clean` in
`node_modules/pocketsphinx/`. Then I ran `npm install` again and it
worked.
@andy5995
Copy link
Author

I forgot to ping #2 when I made the 2nd commit. This is done.

@ajbogh
Copy link
Owner

ajbogh commented Aug 31, 2018

Can you help me understand your change a bit, because I want to weigh it with my change which also fixes the problem.

My change was to add the -i flag to make install. This ignores install errors and allows the installation to continue. My thought was because /usr/bin/install was complaining about existing files, I could simply ignore the problem and it would continue. In this case my hunch was right and pocketsphinx still worked.

The portion of the package.json with the change:

./configure --prefix=`pwd` && make && make install -i

Pros to my change:

  • minor change, only required a single install option

Cons:

  • Might ignore other catastrophic errors that would break the installer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to complete build on Debian 9

2 participants