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

Improve ifup script UX #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Improve ifup script UX #21

wants to merge 2 commits into from

Commits on Jan 21, 2021

  1. scripts/ifup.sh: Set errexit shell option 

    Without the shell option, if there is any error (e.g. brctl not installed), QEMU will boot regardless. In such case, the error may go unnoticed, given the amount of information dumped on the screen by the boot sequence.
    @64kramsystem
    64kramsystem committed Jan 21, 2021
    Configuration menu
    Copy the full SHA
    d5c8d07 View commit details
    Browse the repository at this point in the history

  2. ifup.sh: Add helpful message for when the brctl command is not found 

    In Ubuntu [20.04] for example, the package is not installed by default.
    @64kramsystem
    64kramsystem committed Jan 21, 2021
    Configuration menu
    Copy the full SHA
    76f026b View commit details
    Browse the repository at this point in the history