-
Notifications
You must be signed in to change notification settings - Fork 0
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 non-poetry dependencies #133
Comments
I think we may also need to add |
Yes, now that nginx is gone I think it may make sense to move the contents of the pywb directory in git up into the main folder? Then a Fortunately it looks like the systemd command IS using the Poetry environment. So other than making the Docker dev environment work this is mainly an issue for was_robot_suite I think? |
My bad, it looks like poetry IS being used in was_robot_suite after all. I think it would be good to remove the pip installed version in all environments to prevent future confusion. |
I've updated the title and description of this ticket to be less alarmist :-) |
In order to reindex using the Docker development environmemt the `poetry run` command will need to be used. Refs #133
To prevent confusion we should remove the previously installed pywb and cdxj-indexer (those that were not installed with Poetry) that are in
/opt/apt/was/.local
. This can be done by sshing into each environment and doing a:The Docker instructions also need to be modified to use
poetry run
withcdxj-indexer
. Now that nginx is no longer being used here we can move the contents of the pywb directory in this repo up one level into the base directory. This will mean you won't need tocd pywb
prior to running commands withpoetry run
. However this will likely require changing the systemd process to not use the subdirectory when referencing theuwsgi.conf
.The text was updated successfully, but these errors were encountered: