relaxing the constraint of non-empty pattern (#1640)

fixes #1626

Author: tulinkry

src/org/opensolaris/opengrok/configuration/Groups.java

@@ -168,7 +168,10 @@ public static void main(String[] argv) {
             deleteGroup(cfg.getGroups(), groupname);
             out = prepareOutput(outFile);
             printOut(list, cfg, out);
-        } else if (groupname != null && grouppattern != null) {
+        } else if (groupname != null) {
+            if (grouppattern == null) {
+                grouppattern = "";
+            }
             // perform insert/update. parent may be null
             if (!modifyGroup(cfg.getGroups(), groupname, grouppattern, parent)) {
                 System.err.println("Parent group does not exist \"" + parent + "\"");

tools/Groups

@@ -449,7 +449,6 @@ Add ()
     done
 
     [ -z "$NAME" ] && echo "name must be specified" && LocalUsage && exit 5
-    [ -z "$PATTERN" ] && echo "pattern must be specified" && LocalUsage && exit 5
 
     CheckInputFile
     Info