Updates to bin/setup to make installation easier#3229
Conversation
… rubocop fixes. reran Brakeman
|
@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/setupscript so that it:bundle installandyarn install.env,config/database.ymlandconfig/initializers/wicked_pdf.rbfilesvimto allow user to edit the information in the new.envfileconfig/credentials.yml.encandmaster.keyfiles and opens it for editinglicenses,metadata_standards, andresearch_domainstablesrepositoriestable ... didn't include that one since it can take over 10 minutesWill update the wiki installation page once this is merged into
mainin the next release.Should address the issue outlined here: #3216