In the file wmgenmenu.c, the macro GRADIENT_BACK is lacking a ')'
which is needed to generate the menus correctly.

The attached patch fixes the problem.

In addition the menu generated to call WPrefs should specify a
complete path, as the file is located in a directory which would not
be expected to be contained in a user's PATH.

The attached patch also addresses this problem.

Thank you!
Cyrus Rahman
--- util/wmgenmenu.c.orig	2015-08-11 12:41:14.000000000 -0600
+++ util/wmgenmenu.c	2017-02-27 12:01:49.157788000 -0700
@@ -217,7 +217,7 @@
 		WMCreatePLString(label),								\
 		WMCreatePLString("EXEC"),								\
 		WMCreatePLString("wdwrite WindowMaker WorkspaceBack '(vgradient, \"" 			\
-		    fcolorspec "\", \"" tcolorspec "\"'"),						\
+		    fcolorspec "\", \"" tcolorspec "\")'"),						\
 		NULL											\
 	);												\
 	WMAddToPLArray(L3Menu, L4Menu)
@@ -334,7 +334,7 @@
 	L1Menu = WMCreatePLArray(
 		WMCreatePLString(_("Configure Window Maker")),
 		WMCreatePLString("EXEC"),
-		WMCreatePLString("WPrefs"),
+		WMCreatePLString("/usr/local/GNUstep/Applications/WPrefs.app/WPrefs"),
 		NULL
 	);
 	WMAddToPLArray(RMenu, L1Menu);

Reply via email to