Skip to content

Commit

Permalink
Correctly pass Copybara config to container (#1)
Browse files Browse the repository at this point in the history
* Correctly pass Copybara config to container

Copybara Docker images uses a wrapper script instead of directly passing
arguments to Copybara itself, in order to set arguments for the JVM.
The script is receiving its arguments through environment variable,
instead of commandline args.

* Ignore copybara return code

Copybara exits with error code 4 when the workflow would produce no
changes. This means the build fails, so ignore the error code for now.
  • Loading branch information
antoinealb authored Apr 2, 2022
1 parent c7f23cc commit 0979821
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ runs:
- run: docker pull sharelatex/copybara:latest
shell: bash

- run: docker run -v ~/.ssh:/root/.ssh -v ~/.gitconfig:/root/.gitconfig -v "$(pwd)":/usr/src/app -i sharelatex/copybara copybara ${{ inputs.path }}
- run: docker run -v ~/.ssh:/root/.ssh -v ~/.gitconfig:/root/.gitconfig -v "$(pwd)":/usr/src/app -e COPYBARA_CONFIG=${{ inputs.path }} -i sharelatex/copybara copybara || true
shell: bash

branding:
Expand Down

0 comments on commit 0979821

Please sign in to comment.