-
Notifications
You must be signed in to change notification settings - Fork 163
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
Remove postinstall step #1419
Remove postinstall step #1419
Conversation
The postinstall script created `./data` and `./narratives` directories, but is now obsolete. The `get-data` script will create its own `./data` directory, and the docs identify when this should be run. There is no script to automatically fetch narratives. Removing this from the postinstall step means that `env` is not necessary for an auspice install, which should facilitate installation on windows.
Thanks, @jameshadfield! I think this will eventually work, but the installation recommended above errors with a missing dependency and lack of pre-built binaries. Node thinks it needs a C++ compiler to build the missing binaries. Here is the full error message:
Looking at the release page for the iltorb package that 404s, there is an older version available for Windows here: I don't know enough about npm's bundling logic, but presumably the pre-built binaries for these packages get included in a full Auspice release as opposed to a GitHub branch installation? |
Thanks @huddlej. Regardless of whether this fixes the windows installation issues, I think it is a beneficial change so I'm going to merge for the next release, then we can re-test and see if this helped with windows 🤞 . |
The postinstall script created
./data
and./narratives
directories, but is now obsolete. Theget-data
script will create its own./data
directory, and the docs identify when this should be run. There is no script to automatically fetch narratives. Removing this from the postinstall step means thatenv
is not necessary for an auspice install, which should facilitate installation on windows.This should be npm installable via
npm install --global nextstrain/auspice#windows
Testing: