Re: Window Maker generates menus it cannot parse due to an ommtted ')'

2017-02-28 Thread Doug Torrance

On 02/28/2017 04:23 AM, Cyrus Rahman wrote:

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.


WPrefs isn't guaranteed to be in 
/usr/local/GNUstep/Applications/WPrefs.app either.  Its installation 
path is given by the autoconf variable wprefs_bindir.


I addressed a similar problem with the default menus in the commit below:
http://repo.or.cz/wmaker-crm.git/commitdiff/d48a37060c8ba29eb07e6301497068d2ed0dbddd

Doug


--
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


Window Maker generates menus it cannot parse due to an ommtted ')'

2017-02-28 Thread Cyrus Rahman
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.0 -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);


Re: [PATCH 2/3] wmxss: Add version 0.1 to repository.

2017-02-28 Thread Doug Torrance

On 02/28/2017 09:07 AM, Nerijus Baliunas wrote:

On Thu,  1 Dec 2016 20:38:39 -0500 Doug Torrance  wrote:


diff --git a/wmxss/Src/.nfs3822 b/wmxss/Src/.nfs3822
new file mode 100755
index 
..73eb00946be25add536d30049983ad6f8d98a88a
GIT binary patch
literal 183024
zcmeFa3y@vMc_wxTg!r-?LJ(z9kF|$5lqt*N)nEV)2C}>YFnEwd5C92)04O=+!OX>A
zfSEgknE~<0CgxVzk)1eIuu4`G$Gd?Z_BvILOWBRHsjMxqvfi>bW>ruOBQq*hg)+6W


This file was probably added by accident.


Thanks for noticing this!  I just sent a patch deleting this file.

My initial commit for all of these dockapps is generally just the result 
of "tar -xvzf wmfoo-0.1.tar.gz".  So these sorts of files creep into git 
on occasion.


Doug


--
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


[PATCH] wmxss: Delete NFS "silly rename" file.

2017-02-28 Thread Doug Torrance
These are created by NFS clients to help delete files.  See:
http://nfs.sourceforge.net/#faq_d2

We don't need this in git.

Reported-by: Nerijus Baliunas 
---
 wmxss/Src/.nfs3822 | Bin 183024 -> 0 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)
 delete mode 100755 wmxss/Src/.nfs3822

diff --git a/wmxss/Src/.nfs3822 b/wmxss/Src/.nfs3822
deleted file mode 100755
index 
73eb00946be25add536d30049983ad6f8d98a88a..
GIT binary patch
literal 0
HcmV?d1

literal 183024
zcmeFa3y@vMc_wxTg!r-?LJ(z9kF|$5lqt*N)nEV)2C}>YFnEwd5C92)04O=+!OX>A
zfSEgknE~<0CgxVzk)1eIuu4`G$Gd?Z_BvILOWBRHsjMxqvfi>bW>ruOBQq*hg)+6W
zuxc9rsd|KERipZC23lDISn2XoKq)7^jn{rCU<{W$&j#XbA?4y_w%`G4zL
z>+n3(8hY0kaNpWm_b0cD-`3V}>m98Rx8B!!FWk3^-w^!u-&NfB_YVB=-!1riQJ(bY
zUG90kyK91$?7H8+-r^tQn-AUVJv^P7y6!%5<%-hf7Z$JJX#osCX{m1xw0Dt^
zbe_WB1pd1CJA%Ji{2j+1|2?TU`Q7B6AH)4|{2j*MLHF)`xc`)U-j4g2d*7;U`9XKzco3mlK&?`QFM
z0Dm3)@!t&o4&iT#j_BVn;Q8kwcX%99yaV_{@^Yy4J>KD_J;B@cl2^F#pY9(s_j+~T
z^;^8dQ*Fr5UEgwU+8Ttv>xa%Q7yQF0{?e9
zx25;v>i!+$=RZ>S?^^u3)cprWe}i)qk4#@}-TFVU@EgV7{lNd-IrrT;_j_{gpUAms
zfBiquy9bV;*81NSdAh`Y+{tGJZp!EGmh$rum}8LP*3kPujQbCr|3?x3gK&!{ZhyBm
zH2f0C-tGJkwATNlPe=Y8xK|?g66ik%_j*VFKWVN1XMaO~=YAI9U!$9ebN(Yp@1s%t
zZE$}javy9Bz2`aP@53(qr$C?HzvbL#kiN(dZu}d9e1Dha<>WQ=pCSG0a9aBRMQcd(
zpXCMpAG(6{o{hr)1H>17b@cvw_>0~-_djk8{lxbW?-Ov}j{83c{TCzp&s$v!c>^Cq
z{}<@(i0HozeqW8;e}VW?ep>!R|GKsQpWXxiUvch#4}Zr0Nw|Thb&&rbJPkMVL-#vD
z?>lh2{I1&wet#`;--qze!M$GfV;%JOpI(LgKBd2I4DLu@*6oBl(uZ{q!yT31y5n$1
z@?19qca;Bi&mjGA&^LbmQ@9}~@BSr(r@pxISob-&zZJQ^0{6?2`+oxef*rx<%Vgpa*+*%a65&HeG}_
z`J?+D@B{g-clf;J{?^dDzlU;tNz4D1t*xPZrV-xh<1IT|LmL<2f6@8xMS7HvrFR7U
z?2p{D@c%;OUW7eBdM2M+F2Vl~Bme&j{z%{Zf3`LBQ^?QI2AAG{3;!QP{$GathT(Sl
zbIbpL{E=VkKkWT2{{-P*jl%!0aQ`rJUxWL#$PIn}M_+~8^7DVdy*qOM8S=NmyN@64
ztV}G-&!^q#)>F$X>C#GT{?hX7<<9BZ?#gm&wmZ9W{K9N^df`Ir_~Fy(Vh4dw&UQPG
zrpsqrGo6)PbDi#VXL@2`Zegi)eByLx>gi{jb8*E*m{?f6v~#JGww{_@IN5ShcXt+7PPd-g
zxBuX-o%^3UxOea2Jx888vUAt|Jx_to%F=9iX8-K+%8`X5AarPHVX?Eca;Y^twa`Td
zn9^KlYUSbX;`x=<6zCt{^{6C~u5>QWuC(Mw-XGHU2Nss*)46>UEfDIO49+jlbvlc!
z!X!hcKy0~{~VSabIlC~DX#L83Wm(!U}>+#O?QhGtv_gZXI`dRf+7Jn_HN2qCEt0A&0G9=T}x1x`&oJ%gZXPQwvL-rxs@~
zww~I3;=s;FAD(C(e>7dxbd{k~t*1ne7v^U=-PW#j>g>$Y!uhT%w0)h0`OeA`ONMed
zKC#g4LQt~{-2><6Pj;4CR$%j&F3hV|Jd`f)x!9RHztU+T;d!dUzJ-;&vrEe>6LYgu
zXE$e^qgm&c*7C~I)cj&=`O@-AXP%zB7tWuY>x8(GTOvp9E#PySvVQ8&;|C}9>^}PV
zo~QOdeBhCt`}TNc(6T|%F`>mvt>cH%rDbTY=%{L(6t0%SQypl)O6w_fvz~gUv$Twy
zw_NrN85;l5-TR^WnSyQ2r>a}Xc)Gkyg018GpwpuMJ6E8UCn2Wg)_iB4oScO4rLc}G
zkB4XH7w0;zupVDnSRtmA(P;;mBeV04^`}ljM_S8MOC1#B#ive9w=OX8V~_eGJY`Cz
z)OXFU#M+4r9^br~{m3!&Bo7}waIC!<4|qMLolCxYYNWlLub|?5xwSnmt`LuYke8C#
zO3&qmsk0q%U9eMA&<(<^WyWBI|IuB?&Q`$LgdxnkPYtOHu3un)0i;THv)_aHS`EGPc
z?0Fc~uRVXX)f%*MzPYv}gc
zGwPX>_WYGrYl}U9wbh!pXVf**_KbSvggw95YCUbwFQXe|&!}sD#-346B<`7Y@)Pz9
znJ?J$2hnY_=MT49+w6I3tMxv6-qC99u;=})R%*}3F?3+h)2-Ie$@3p>7;e3M%herU
zy5;Kb-%73yKYOJ$wC>r~`YRjIC~p0$y-*C+-49*axM>>hU3Bk2bBK3@+3?v7t#|##
zM~ByaZr#=OpIg7<&iA&49{BvmZ$J0qrsoi@^-~)DUu|>uao+#IE7z{T?-mRqe20I(
z|DIQ#`~7!3cNfxWUD-Ii{`xzfyZwcA!|R{@($g*Qa>rFXUx!5gyZ;S?KkN1R*)I*Z
zHeUPDUz6V7|C5(*|NGxly5aecM!?@aNaxew@qPL{^nH8g{XMOrk9=<9)r;Wyqo3dS
z%J3I94DZ;{TK9K;6ZZu906%~6!us^<$oG#U|6h0)-d$gp{4vu8Uq5#Bwm-T;89>?p
z;+93&p#6EgyX(rj;i2d5ORqn7-_5EaN58kD)%xElL->6k`MCr1{_M)eWW$DQKf2C*1i4?kg1ihp
z`<2!`Kk;nq9P@iW)Pg&6a{q+AV
z{GR>N4${2i%H3al?aKNc@4HUDm~6EW?tzWelMU(z`v3E1@BZ_b!1H0v!*~AcD;qxf
zqhALPFWkE0cFGH3zItWjy~9X%Pn7O&Bi&bTAl)yxbRTr-UMflVvq*Ob(!D=Q_cKWM
zHKvPve6WYS-f4Mt^1A%ljZ-MA)_YyLpDIar1?BafcO&mtP(Dci*6W+5{upuYba4*w
zJ$Yi;-Foduzriw#%awQljbWGOLr8O+?;y)Pc)!Cv@51vH_xwpbzl`VaA(1bjb+=Zo%n3!cC3o<|_3mV15x&rFNqKZ55)_k2H|zv`av!}E3bd@t~Nk9+<}
zJX_ok;Mvl;M=>ol--Bo4b0c{Eu#0;ao-M6+;`x7Z@9)6#_wf9E@N+Bkm6g-b125b<
zycPCq2<3U#bDQqHe*IR_)Ai8jcV34d?ZowUUw)ot4&0%?BpCkb7l3n=&3!Ly`ttJ(
z2mf`5vw_cf8Z67}>!t)JpaEM${0N=9QMTU#osGED{}%AELBcye;Qn#MeaX>y(eV+}
zxCL=v#@~xB%u6oJZR8cU=K1F~O}(Ttc;!zagB>p1w_G^M6T)3bxNkYXmveqE!tdos
zb|cX1d%n71n6|Qgef`uAV6!%SZo}|9ceGyllh3Q|5_wT)VS8Tt>aD{=QT%6VZ+`Jd
zKl(S<*MITr&m~u<@pqbb_v+nK_gr5;e9sp^`$hPnBl^HM)_?o?>&ewmzp#GzgFnOc
z-jV#^h2-kfFRY(Jz46NLqaJ91UK{kjd42ttUo(27!#FSE9qAdJT>O8?_$9pkNi}b+
zdwS&UGrx)Y%klP4s(EYOQ^DH>#*caXA>{B^$op%M8Opc)|3PjnGwSIJ>!u(#&|3G!
z=NS)mAnKD>PJIscwig-<|GAA*9|WBbvVHL4x@&)<^uGN(=p4!Ma^?E1PowS^T-@~~
zDTCJcui_os00{qy&)$9YY5H$^wzclh(H?-!5SFKv3|dz2;UYzV+avV?Q5@r-*{4M@2_5UNJGxh&Q{vUPm
zKd%1&;=ADEY~(-g{72RQ`QHQmdn5nd&i^Co|H8lj?8bB~@;~JKH>v;sJ&yGMDDwZb
z^M616`5t*X&Aid={N7C&;r)uXS6{gY<$LG#O)roC`KFh*KDX)Rr{C3Tp>7%8fp@?|
z>o(L658?TC$o>6aSoiY!2U&)+uI&ZG9hQe3AUIj9I>@qrc3u{aL2@p{eh<
zFwf?~{3gS|jy-VwL-)RTWj*i$Oyhn#?oZ?XPTZ$)-+=q+E4O{^Rov%3zhM}9Gd#q+
zP%nYeSBd!-*M0G6@cN6u=DWb>j_da!{(Xr1j(cBx;T==oAx*}Cy_{zLUq)IlB2CcR
z@eQPjxXWxu(?(st@5`^fh(M#$ouEz#rnA`X>0JZN7frx4$m*
zVPo-r3bJ??-6+Q&eD1bu-+KY?zj6OH%Ka(ae?UKA`6}Pz-7kFZ?%`k5_}~6I+$ZQp
zo>8BmjdR+rQAb^T33>eH{nwsFyxlL|f9;o8)+p!a5$^+xgEIY~qxDIY8TuQo+lkL_
z-j}e>`ZRFg(s08q7jB&O4E^|?{x7Wm_Sac<-&~(Ce-u}Iedo%$
zPks-$d-YFmyY}iYU%ls-asPY2{Kbbiy#w<8m#r^8k1*f+<*UDPUmWf;2=~iZ-@l1r
zKl8-QPW&s=?fLR#SGUF74N+mQZ-;oC^-PZ0kl#J~Qr
zYkx^zN$(qQzqE}y_3eN2jcwPyOMb8a^s_Cdcm1cI;~l)

Re: [PATCH 2/3] wmxss: Add version 0.1 to repository.

2017-02-28 Thread Nerijus Baliunas
On Thu,  1 Dec 2016 20:38:39 -0500 Doug Torrance  wrote:

> diff --git a/wmxss/Src/.nfs3822 b/wmxss/Src/.nfs3822
> new file mode 100755
> index 
> ..73eb00946be25add536d30049983ad6f8d98a88a
> GIT binary patch
> literal 183024
> zcmeFa3y@vMc_wxTg!r-?LJ(z9kF|$5lqt*N)nEV)2C}>YFnEwd5C92)04O=+!OX>A
> zfSEgknE~<0CgxVzk)1eIuu4`G$Gd?Z_BvILOWBRHsjMxqvfi>bW>ruOBQq*hg)+6W

This file was probably added by accident.

Regards,
Nerijus


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.