Skip to content

Remove sudo from build_and_check_circle_config#104

Open
andheiberg-babylonhealth wants to merge 1 commit into
roopakv:masterfrom
andheiberg-babylonhealth:patch-1
Open

Remove sudo from build_and_check_circle_config#104
andheiberg-babylonhealth wants to merge 1 commit into
roopakv:masterfrom
andheiberg-babylonhealth:patch-1

Conversation

@andheiberg-babylonhealth

Copy link
Copy Markdown

Interested to get your thoughts. I have a docker image that doesn't have sudo installed. So I can't rely on sudo. I wonder why this was setup in the first place, but it was presumably because the file permissions of your image were more sensibly locked down.

For now I will install sudo on the CI image to get this to pass. Just curious to hear your thoughts.

Interested to get your thoughts. I have a docker image that doesn't have sudo installed. So I can't rely on sudo. I wonder why this was setup in the first place, but it was presumably because the file permissions of your image were more sensibly locked down.

For now I will install sudo on the CI image to get this to pass. Just curious to hear your thoughts.
@andheiberg-babylonhealth

Copy link
Copy Markdown
Author

The other option for me is to install circleci cli my self and use install-circle-cli: false

@roopakv

roopakv commented Jul 12, 2022

Copy link
Copy Markdown
Owner

what if you make sudo a param?

use-sudo defaults to true, but if false then it installs without sudo?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants