Fixed issue with paremeter evaluation and updated README

README.md

@@ -6,16 +6,16 @@ cURL for downloading the setup file and package logo.
 
 ```
 Usage:
-  ./new-package.sh <product-id>
+  new-package.sh <product-id>
 
 Optional parameters:
   -d <Product Description>    Adds a description to the package
   -n <Product Name>           Adds a product name instead of autogenerating one
-  -g yes                        Adds the Gruelag prefix to package ID and name
+  -g yes                      Adds the Gruelag prefix to package ID and name, do not manually add Gruelag when using this!
   -u <Program URL>            Downloads an installer file from the specified URL
   -i <Logo URL>               Applies a logo to the package from the specified URL
+  -t <Package Type>           Sets the type of package to create, see below
   -v <Program version>        Sets the version of the program (not the package)
-  -t <Package Type>             Sets the type of package to create, see below:
 
 Package types:
   winbatch                    Creates a blank WinBatch package
@@ -25,9 +25,17 @@ Package types:
   execps                      Creates a blank ExecWith Powershell package
   zip                         Creates a blank ZIP based package
   msi                         Creates a blank MSI installer package
+  weblinks                    Creates a weblink creator package
 
-Optional parameters for "zip" type package:
-  -e <filename.exe>     Specifies the launcher executable inside the ZIP file
+Parameters for 'zip' type package:
+  -e <filename.exe>           Optional, specifies the launcher executable inside the ZIP file
+
+Parameters for 'weblinks' type package:
+  -l <link name>;<target url>;[<where>]    Can be used multiple times, specifies the URL and display name for one web link.
+                                           <where> is optional and can hold one of the following values:
+                                               'desktop' to create only desktop links on the public user desktop
+                                               'startmenu' to create links only in the public user start menu
+                                               'both' to create links at both places, this is the default
 ```
 
 Example for Kdenlive:

new-package.sh

@@ -67,12 +67,15 @@ do
 		i) imageUrl="${OPTARG}";;
 		v) programVersion="${OPTARG}";;
 		l) weblinks+=("${OPTARG}");;
-		*)
-			echo "Unrecognized option '${flag}'"
 		h)
 			usage
 			exit 0
 			;;
+		*)
+			echo "Unrecognized option '${flag}'"
+			usage
+			exit 1
+			;;
 	esac
 done