yeyushengfan258 / We10X-icon-theme

A colorful design icon theme for linux desktops
GNU General Public License v3.0
237 stars 9 forks source link

Improve argument parsing in install.sh #9

Closed suve closed 4 years ago

suve commented 4 years ago
  1. The --dest and --name options removed two arguments using shift 2, but then after case-esac there's another shift. This caused the --dest and --name options to consume 3 command-line arguments, instead of 2, leading to broken argument parsing. Fixed this.

  2. Added a check to test if the argument to --dest and --name actually exists.