`start-ts.sh`: several issues

joeflack4 commented 1 year ago

Overview

I haven't worked with this locally in 2-3 weeks. I see that #4 is still open, but I decided to try again anyway. However this time, I got a new error.

Issue list

[ ] 1. No such option: --hapi-port
[ ] 2. yq: command not found
[ ] 3. setup-servers: command not found

Issue details

1. `No such option: --hapi-port`

This happened on Sunday, 01/08.

(tims-ts) sh bin/start-ts.sh
Usage: setup-servers hapi-jpa-starter [OPTIONS]
Try 'setup-servers hapi-jpa-starter --help' for help.

Error: No such option: --hapi-port

2. `yq: command not found`

This happened on Thursday, 01/12 after I had followed these reset steps.

(tims-ts) bin/start-ts.sh
bin/start-ts.sh: line 37: yq: command not found

3. `setup-servers: command not found`

This happened on Thursday, 01/12 after I had followed these reset steps. I was able to get past the yq on line 37 and get to this point on line 41 by commenting out the yq lines temporarily to see how far I'd get.

(tims-ts) bin/start-ts.sh
bin/start-ts.sh: line 41: setup-servers: command not found

ShahimEssaid commented 1 year ago

@joeflack4 try this:

git reset --hard
git pull --ff-only
rm -rf .venv
bin/reset-ts.sh
bin/start-ts.sh

Let me know if that works. With any git pull to get an updated tims-ts version, the .venv needs to be deleted manually. I can write a script to do all the steps needed for an "upgrade" of a tims-ts instance. I just hadn't gotten to that yet.

The /timsui/ part needs some additional local configuration but I'll document that next. Just see if HAPI comes up for now.

joeflack4 commented 1 year ago

@ShahimEssaid Thanks for the steps; I shoulda tried reset-ts.sh when this initially. But I did ensure the other steps were complete as well.

I tried following these steps, but I got two new issues. Issue (2) above, is because yq isn't installed I imagine, but idk how you would go about packaging it in the project. If you can't, we can add it as a manual setup step. For issue (1) and (3) above, I'm kinda surprised that this is happening now when they were working previously.

ShahimEssaid commented 1 year ago

yq gets installed with the scripts if you delete the .venv. But, it might require jq installed by the OS as well. Let's have a Zoom next week to figure out why things are not working on your machine as well.

Sent from mobile

On Thu, Jan 12, 2023, 5:44 PM Joe Flack @.***> wrote:

@ShahimEssaid https://github.com/ShahimEssaid Thanks for the steps; I shoulda tried reset-ts.sh when this initially. But I did ensure the other steps were complete as well.

I tried following these steps, but I got two new issues. Issue (2) above, is because yq isn't installed I imagine, but idk how you would go about packaging it in the project. If you can't, we can add it as a manual setup step. For issue (1) and (3) above, I'm kinda surprised that this is happening now when they were working previously.

— Reply to this email directly, view it on GitHub https://github.com/HOT-Ecosystem/tims-ts/issues/6#issuecomment-1381204584, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABELXUK3DR3GTD7SI6OJ2UDWSCXPFANCNFSM6AAAAAATU3BXSY . You are receiving this because you were mentioned.Message ID: @.***>

ShahimEssaid commented 1 year ago

I make the pip install silent in the scripts but maybe I shouldn't to help notice any issues.

Sent from mobile

On Thu, Jan 12, 2023, 5:44 PM Joe Flack @.***> wrote:

@ShahimEssaid https://github.com/ShahimEssaid Thanks for the steps; I shoulda tried reset-ts.sh when this initially. But I did ensure the other steps were complete as well.

I tried following these steps, but I got two new issues. Issue (2) above, is because yq isn't installed I imagine, but idk how you would go about packaging it in the project. If you can't, we can add it as a manual setup step. For issue (1) and (3) above, I'm kinda surprised that this is happening now when they were working previously.

— Reply to this email directly, view it on GitHub https://github.com/HOT-Ecosystem/tims-ts/issues/6#issuecomment-1381204584, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABELXUK3DR3GTD7SI6OJ2UDWSCXPFANCNFSM6AAAAAATU3BXSY . You are receiving this because you were mentioned.Message ID: @.***>

timsbiomed / tims-ts