OpenNeo/impress-2020
OpenNeo/impress-2020
1
0
Fork 0
Code Pull requests Activity

Oops, fix YAML + bash mistake

Browse source 
Okay so you know how in 3f07933f7a I switched the newline stuff here?

Yeah, right, I forgot, newlines are significant in bash :p I forgot this because I've never used them inside a `bash -c` invocation, but like, of course

Now, I'm still using `|` for clarity and reduced dependence on magic, but getting my lines right :p
This commit is contained in:
Emi Matchu 2021-11-03 17:34:07 -07:00
parent 32ae1accce
commit 1e3e8391b4
1 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
deploy/playbooks/deploy.yml
View file

@ -59,7 +59,7 @@
command: |
bash -c "
cmp '{{ remote_app_root }}/yarn.lock' '/srv/impress-2020/current/yarn.lock' &&
test -e /srv/impress-2020/current/node_modules"
test -e /srv/impress-2020/current/node_modules"
failed_when: no
register: can_reuse_node_modules_when_successful
@ -70,9 +70,7 @@
# running. So, these two copies of node_modules *should* just stay
# permanently the way they are forever, so linking shoouulld be safe 🤞
command: |
bash -c "
ln -s $(realpath /srv/impress-2020/current/node_modules)
{{ remote_app_root }}/node_modules"
bash -c "ln -s $(realpath /srv/impress-2020/current/node_modules) {{ remote_app_root }}/node_modules"
when: "can_reuse_node_modules_when_successful.rc == 0"
- name: Run `yarn install` to install dependencies in new remote folder