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

Improved script argument parsing (any order), plus couple tiny tweaks #2

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

Commits on Aug 6, 2021

  1. Tiny typo in README.md

    lgorenstein authored Aug 6, 2021
    Configuration menu
    Copy the full SHA
    497511c View commit details
    Browse the repository at this point in the history
  2. Merge pull request #1 from lgorenstein/lgorenstein-patch-1

    Tiny typo in README.md
    lgorenstein authored Aug 6, 2021
    Configuration menu
    Copy the full SHA
    0f9c9cf View commit details
    Browse the repository at this point in the history

Commits on Aug 7, 2021

  1. Typo in error message

    lgorenstein authored Aug 7, 2021
    Configuration menu
    Copy the full SHA
    9fb3e94 View commit details
    Browse the repository at this point in the history
  2. Quote command arguments as separate words

    Minor, but feels cleaner (if we are in bash, might as well use bash-isms)
    lgorenstein authored Aug 7, 2021
    Configuration menu
    Copy the full SHA
    aaef41e View commit details
    Browse the repository at this point in the history
  3. Improve parsing of script arguments

    Original version required -D, -I and -O options to be listed in exactly this order. While doable, this is somewhat counter-intuitive with typical Unix option processing expectations (e.g. `ls -l -a` is expected to behave the same as `ls -a -l`). This change introduces rudimentary options processing that allows specifying them in an arbitrary order (as long as they are before any consecutive command arguments).
    lgorenstein authored Aug 7, 2021
    Configuration menu
    Copy the full SHA
    34be333 View commit details
    Browse the repository at this point in the history
  4. Merge pull request #2 from lgorenstein/lgorenstein-patch-2

    Better arguments parsing, plus couple tiny tweaks
    lgorenstein authored Aug 7, 2021
    Configuration menu
    Copy the full SHA
    672d796 View commit details
    Browse the repository at this point in the history