Permalink
Browse files

Merge pull request #1 from flavioamieiro/master

Fixes two argument parsing bugs
  • Loading branch information...
2 parents 1210752 + 74e24f0 commit eb4e5660e4726df5f51ab23c69d96487b1238fb7 @pbrisbin committed Aug 29, 2012
Showing with 3 additions and 3 deletions.
  1. +3 −3 dvdcopy
View
@@ -238,7 +238,7 @@ message() { # {{{
--title=<number> set the title, default is longest
- --size=<number> set the desired output size in KB,
+ --size=<number> set the desired output size in KB,
default is 4193404, DVD5
--limit=<number> set the number of times to attempt a
@@ -337,8 +337,8 @@ parse_options() { # {{{
# all other options are handled by a simple eval, we rely on the
# case pattern to do the validation
- --directory=*|--device=*|--title=*|--size=*|--limit=*|--mpeg_dir=*|--dvd_dir=*|--iso_dir=*|--mencoder_option=*)
- eval "${1/--/}" ;;
+ --directory=*|--device=*|--title=*|--size=*|--limit=*|--mpeg_dir=*|--dvd_dir=*|--iso_dir=*|--mencoder_options=*)
+ eval $(echo $1 | sed -r 's/--(.+)=(.+)/\1="\2"/') ;;
*) message ;;
esac

0 comments on commit eb4e566

Please sign in to comment.