fourkitchens / fire

FIRE is a commands standardization tool
0 stars 0 forks source link

fire requires `wget`, but it's not commonly installed on OSX #18

Closed dalin- closed 3 months ago

dalin- commented 6 months ago

To reproduce:

$ fire build
...
[Exec] Running fire local:get-db
 [Exec] Running wget `terminus backup:get aft-leadernet.github-dev --element=db` -O /Users/dave/Sites/aft-leadernet/reference/site-db.sql.gz
sh: wget: command not found
 [Exec]  Exit code 127  Time 5.68s
 [Exec]  Exit code 127  Time 5.802s
dalin- commented 6 months ago

Fire:

wget `terminus backup:get aft-leadernet.github-dev --element=db` -O /Users/dave/Sites/aft-leadernet/reference/site-db.sql.gz

Sous does

terminus backup:get aft-leadernet.github-dev --element=db --to=reference/db.sql.gz

It's not obvious to me why we need wget to be involved. I think the direct call to terminus should be just as good.

dalin- commented 4 months ago

Latest PR fixed it. But now a new issue:

$ fire get-db
 [Exec] Running terminus backup:get aft-leadernet.github-dev --element=db --to=/Users/dave/Sites/aft-leadernet/reference/site-db.sql.gz
 [notice] Downloading aft-leadernet_github-dev_2024-04-24T16-28-14_UTC_database.sql.gz to /Users/dave/Sites/aft-leadernet/reference/site-db.sql.gz
 [error]  Target file /Users/dave/Sites/aft-leadernet/reference/site-db.sql.gz already exists.
 [Exec]  Exit code 1  Time 3.954s
rigoucr commented 3 months ago

Thank you Dave!!