bin/xbps-create: use process_dict() for alternatives too

Author: classabbyamp

bin/xbps-create/main.c

@@ -233,6 +233,55 @@ process_array(const char *key, const char *val, bool (*validate)(const char *s))
 	xbps_object_release(array);
 }
 
+static void
+process_keyval_array(const char *prop, const char *keyval, const char delim,
+					 bool (*validate_key)(const char *),
+					 bool (*validate_val)(const char *)) {
+	xbps_dictionary_t d;
+	xbps_array_t a;
+	char *key, *valstr;
+	bool alloc = false;
+
+	if ((d = xbps_dictionary_get(pkg_propsd, prop)) == NULL) {
+		d = xbps_dictionary_create();
+		if (d == NULL)
+			die("xbps_dictionary_create");
+		alloc = true;
+	}
+
+	key = strdup(keyval);
+	if (key == NULL)
+		die("strdup");
+	valstr = strchr(key, delim);
+	*valstr = '\0';
+	valstr = valstr + 1;
+	assert(valstr);
+
+	printf("%s: %s = %s with delim %c\n", prop, key, valstr, delim);
+
+	if (validate_key && !validate_key(key)) {
+		diex("%s: invalid key: %s", prop, key);
+	}
+
+	if ((a = xbps_dictionary_get(d, key)) == NULL) {
+		a = xbps_array_create();
+		if (a == NULL)
+			die("xbps_array_create");
+	}
+
+	if (validate_val && !validate_val(valstr)) {
+		diex("%s: invalid value: %s", prop, valstr);
+	}
+
+	xbps_array_add_cstring(a, valstr);
+	xbps_dictionary_set(d, key, a);
+	xbps_dictionary_set(pkg_propsd, prop, d);
+	if (alloc) {
+		xbps_object_release(a);
+		xbps_object_release(d);
+	}
+}
+
 static void
 process_keyval_uint64(const char *prop, const char *keyval, const char delim,
 					  bool (*validate_key)(const char *),
@@ -344,7 +393,6 @@ process_one_alternative(const char *altgrname, const char *val)
 	}
 }
 
-
 static void
 process_dict_of_arrays(const char *key UNUSED, const char *val)
 {
@@ -1162,7 +1210,7 @@ main(int argc, char **argv)
 	process_array("reverts", reverts, NULL);
 	process_array("shlib-provides", shlib_provides, NULL);
 	process_array("shlib-requires", shlib_requires, NULL);
-	process_dict_of_arrays("alternatives", alternatives);
+	process_dict("alternatives", alternatives, ':', process_keyval_array, NULL, NULL);
 
 	/* save cwd */
 	memset(&cwd, 0, sizeof(cwd));