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

Container commands #60

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

Commits on Sep 1, 2020

  1. Add TTY tests

    Adds test to validate that the TTY is correctly passed through to the
    container when stdout is a TTY
    JoshuaWatt committed Sep 1, 2020
    Configuration menu
    Copy the full SHA
    47e43d0 View commit details
    Browse the repository at this point in the history

Commits on Sep 22, 2020

  1. Rework user commands to add to container commands

    Reworks the way that commands specified by the user are wrapped by
    Pyrex. Instead of completely replacing the container commands, the user
    commands override and extend the container command defintions, taking
    precedence over them (i.e. if the user excludes a command it will be
    excluded even if the container includes it, and if the user includes it,
    it will be included even if the container excludes it).
    
    In order for this to work, the user command shims must be individually
    written to execute the full path to the command inside the container,
    since we don't know if it will be in $PATH.
    JoshuaWatt committed Sep 22, 2020
    Configuration menu
    Copy the full SHA
    6fa9ff4 View commit details
    Browse the repository at this point in the history