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.

Sign up for GitHub

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

Jump to bottom

install: Add prominent warning+timeout when targeting host root #505

Merged
merged 2 commits into from
May 3, 2024
Merged

install: Add prominent warning+timeout when targeting host root #505

merged 2 commits into from
May 3, 2024

Commits on May 3, 2024

  1. install: Add prominent warning+timeout when targeting host root 

    It was raised in a discussion that it's rather easy to copy-paste
this command you may find in a document and not fully understand
what it does.

Let's prominently warn by default, with a timeout.

Signed-off-by: Colin Walters <walters@verbum.org>
    @cgwalters
    cgwalters committed May 3, 2024
    Configuration menu
    Copy the full SHA
    dcd42f7 View commit details
    Browse the repository at this point in the history

  2. ci: Drop unnecessary -ti from podman run invocations 

    Especially the `-t` forces as pty allocation which causes
bad rendering for things like progress bars in the github actions
flow.

Signed-off-by: Colin Walters <walters@verbum.org>
    @cgwalters
    cgwalters committed May 3, 2024
    Configuration menu
    Copy the full SHA
    2094895 View commit details
    Browse the repository at this point in the history