Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow bower to be run as root #776

Merged
merged 1 commit into from
Aug 9, 2021

Commits on Aug 9, 2021

  1. Allow bower to be run as root

    Without this change, we get the following error while running in a docker container
    
    ```
    Updating bower
    
    bower ESUDO         Cannot be run with sudo
    
    Additional error details:
    Since bower is a user command, there is no need to execute it with superuser permissions.
    If you're having permission errors when using bower without sudo, please spend a few minutes learning more about how your system should work and make any necessary repairs.
    
    You can however run a command with sudo using "--allow-root" option
    ```
    
    Since the default user in the container is `root`, this is actually an expected
    use case for us.
    
    I tried to avoid this by calling `bower update --allow-root` in the docker
    setup script e-mission/e-mission-docker#18
    
    But that ran into multiple inconsistencies (
    e-mission/e-mission-docs#657 (comment)
    e-mission/e-mission-docs#657 (comment)
    )
    
    In order to avoid more bitrotted containers, just changing the code in the
    setup script so it works in the container as well
    
    This fixes e-mission/e-mission-docs#657
    shankari committed Aug 9, 2021
    Configuration menu
    Copy the full SHA
    2b342a5 View commit details
    Browse the repository at this point in the history