-
Notifications
You must be signed in to change notification settings - Fork 109
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
Updates to bin/setup to make installation easier #3229
Conversation
… rubocop fixes. reran Brakeman
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good :)
@briri @benjaminfaure @pengyin-shan @martaribeiro Not sure why the setup fails for me. As always it must be something on my PC. I am on an Ubuntu PC and I am using RVM to set the Ruby to version 2.7.2. Note I am using a newly cloned version of roadmap. I guess others are using their current. So don't see error as I won't have old recaptcha properties.
I have run the setup several times and it fails at db setup:
|
did the I haven't used a PC though in over 15 years and don't a machine available to test with. @johnpinto1 I just pushed a change that attempts to set the If it doesn't work, can you use that
|
@briri That sorted issue of credentials. I just need to sort issue with duplicate tables after running db:migrate.
|
excellent! You may need to delete the DB and start over from scratch. Sometimes a stalled installation can leave the DB in a state of limbo. |
@briri I am testing the setup with Postgres. It is failing because the annotations table is never created in the migrations. Am I misunderstanding something. I don't see a migration file for creating this table. Also with Postgres one has to run migrations twice at least because of the following error within a db:migrate because you can't change a type within a single transaction several times:
|
Thanks for trying it out @johnpinto1. I did some more reading on the new There are some very old migrations in the |
@briri Phew that was the quickest fix. Yippee! up and running with Postgres using setup. Thanks. |
Updated the
ruby bin/setup
script so that it:bundle install
andyarn install
.env
,config/database.yml
andconfig/initializers/wicked_pdf.rb
filesvim
to allow user to edit the information in the new.env
fileconfig/credentials.yml.enc
andmaster.key
files and opens it for editinglicenses
,metadata_standards
, andresearch_domains
tablesrepositories
table ... didn't include that one since it can take over 10 minutesWill update the wiki installation page once this is merged into
main
in the next release.Should address the issue outlined here: #3216