kill contribs

2010-04-08 Thread Tamas TEVESZ
>From 2480d2d8ebebe77a4c9e656fd54576bbc0e56355 Mon Sep 17 00:00:00 2001 From: Tamas TEVESZ Date: Fri, 9 Apr 2010 03:44:23 +0200 Subject: [PATCH] Kill contribs - Patches don't apply - RPM spec refers to stuff long dead, distros probably have better ones anyway --- configure.ac

Re: New autoconf stuff breaks debian build

2010-04-08 Thread Tamas TEVESZ
On Thu, 8 Apr 2010, Tamas TEVESZ wrote: > try this for something that tries to have the debian/ stuff do away > with touching autoconf (apart from creating a configure if needed). it > builds, but i haven't checked the debs either. for some reason the previous one didn't want to apply clenly

Re: pos breaking build

2010-04-08 Thread Tamas TEVESZ
On Thu, 8 Apr 2010, Tamas TEVESZ wrote: seems it's handled properly afterall... -- [-] mkdir /nonexistentFrom 507f99af347d188de864d7f07deab0283cbf7d3e Mon Sep 17 00:00:00 2001 From: Tamas TEVESZ Date: Fri, 9 Apr 2010 02:30:50 +0200 Subject: [PATCH] Fix PO files breaking build --- WPrefs.app/

Re: New autoconf stuff breaks debian build

2010-04-08 Thread Tamas TEVESZ
On Thu, 8 Apr 2010, Brad Jorsch wrote: > You also need to add "-I m4" to the aclocal invocation on the next line, > to tell it to use the files in the m4 directory. After doing that, a > dpkg-buildpackage -b -uc on the 'next' branch successfully completed for > me here, although I haven't chec

pos breaking build

2010-04-08 Thread Tamas TEVESZ
i have no idea how much did i screw ja.po up... begin 644 fixpos M9&EF9B`M+6=I="!A+U=06UB;VQD871E:2`E M6UB;VQD871E:2`E.#O..#B>.!I^.!C>.!ON.!F^."DR(*(",Z("XN+RXN M+U=0

[PATCH] Fix enable-modelock warnings

2010-04-08 Thread Brad Jorsch
Fix warnings that only show up when --enable-modelock is passed to ./configure. --- src/event.c|6 +- src/framewin.c |3 ++- src/startup.c |3 +++ src/window.c |1 - 4 files changed, 10 insertions(+), 3 deletions(-) diff --git a/src/event.c b/src/event.c index 46e2ea9..

compile-time options

2010-04-08 Thread Tamas TEVESZ
let's make some order here. usermenu - gone optional, default off in 0.60. is this of any practical use? (why isn't it on then? what distros turn this on? anyone uses this?) modelock, xinerama - what harm is done if they are on? shape, shm - i can imagine they cause problems on some servers,

Re: New autoconf stuff breaks debian build

2010-04-08 Thread Martin Dietze
On Thu, April 08, 2010, Brad Jorsch wrote: > You also need to add "-I m4" to the aclocal invocation on the next line, > to tell it to use the files in the m4 directory. After doing that, a > dpkg-buildpackage -b -uc on the 'next' branch successfully completed for > me here, although I haven't chec

Re: fix o-o-t builds

2010-04-08 Thread Tamas TEVESZ
On Thu, 8 Apr 2010, Tamas TEVESZ wrote: > > almost forgot. > > carlos, before you apply this, do rename INSTALL to something else, > anything else, as the install-missing stuff (i think because i > switched it to link instead of copy) will overwrite it. which on second thought might not

Re: New autoconf stuff breaks debian build

2010-04-08 Thread Brad Jorsch
On Thu, Apr 08, 2010 at 01:33:12PM +0200, Martin Dietze wrote: > > just tried to produce the debian build of the current 'crm' and > 'next' branches and found that the build is currently broken. > First, debian/rules contains a dependency to acinclude.m4 which > has disappeared. I fixed this, and

plug leak

2010-04-08 Thread Tamas TEVESZ
you are not paying attention :) >From a522299ba882afb89143e4343429dca7b818836a Mon Sep 17 00:00:00 2001 From: Tamas TEVESZ Date: Thu, 8 Apr 2010 21:50:04 +0200 Subject: [PATCH] Plug leak --- WINGs/proplist.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/WINGs/propli

Re: fix o-o-t builds

2010-04-08 Thread Tamas TEVESZ
almost forgot. carlos, before you apply this, do rename INSTALL to something else, anything else, as the install-missing stuff (i think because i switched it to link instead of copy) will overwrite it. since autoshit seems to like INSTALL, and i'm hazarding a guess people are very accustomed

fix o-o-t builds

2010-04-08 Thread Tamas TEVESZ
>From 24e3759f852b4a39190ec17ead4998095a88cf15 Mon Sep 17 00:00:00 2001 From: Tamas TEVESZ Date: Thu, 8 Apr 2010 21:38:03 +0200 Subject: [PATCH] Fix out-of-tree builds - The throwing out of autogen.sh in favor of autoreconf comes from http://www.gnu.org/software/automake/manual/autoconf/autor

Re: Please moderate message to wmaker-...@lists.windowmaker.info

2010-04-08 Thread John H. Robinson, IV
Window Maker Lists wrote: > Hello, > Sorry about the noise. That was me, approving some queued up messages in entirely the wrong way. -- John H. Robinson, IV jaq...@sbih.org http WARNING: I cannot be held responsib

[PATCH 2/4] Fix internationalized format string warnings

2010-04-08 Thread Brad Jorsch
A few internationalized format strings have only one %s in the default while two strings are being passed. Change those defaults to have two %s's, which means we also have to update the .po files to match. May as well throw in the extra %s in the translated version while we're at it. --- WPrefs.ap

[PATCH 3/4] XShapeEvent strict aliasing violation

2010-04-08 Thread Brad Jorsch
C99 defines new strict aliasing rules to allow compilers to make certain optimizations. These rules prohibit converting an XEvent to an event struct (e.g. XShapeEvent) that is not already in the XEvent union using pointer type punning (e.g. "(XShapeEvent *)&ev"), and vice versa. The canonical fix s

[PATCH 1/4] Fix trivial warnings

2010-04-08 Thread Brad Jorsch
* Format string integer size warnings * Missing or non-value returns in non-void functions * A missing #include in WPrefs.app/MouseSettings.c --- WINGs/dragcommon.c |2 +- WINGs/proplist.c |1 + WPrefs.app/MouseSettings.c |1 + src/appmenu.c | 22 +++

[PATCH 4/4] Fix uninitialized vars

2010-04-08 Thread Brad Jorsch
Initialized variables that are conditionally set. In particular, this construct is dangerous: void *p; if (something) p = couldReturnNull(); if (!p) p = fallbackFunction(); --- WPrefs.app/Appearance.c |2 +- WPrefs.app/Paths.c |5 - src/misc.c |2 +- src/mot

Re: Error cleanup

2010-04-08 Thread Brad Jorsch
On Thu, Apr 08, 2010 at 11:14:14AM +0200, Carlos R. Mafra wrote: > > I would like to apply this, but the patch mixes the warning fixes > with other things (like the piece in src/event.c which you had a > second thought later in this thread). They're all warning fixes, just for different warnings.

Please moderate message to wmaker-...@lists.windowmaker.info

2010-04-08 Thread Window Maker Lists
Hello, This mail has been sent by the mailing list manager operating the wmaker-...@lists.windowmaker.info mailing list to the human moderators of the list. Should the message at the bottom be allowed to be sent to the list? If so, reply to this mail or send mail to wmaker-dev-appro

Please moderate message to wmaker-...@lists.windowmaker.info

2010-04-08 Thread Window Maker Lists
Hello, This mail has been sent by the mailing list manager operating the wmaker-...@lists.windowmaker.info mailing list to the human moderators of the list. Should the message at the bottom be allowed to be sent to the list? If so, reply to this mail or send mail to wmaker-dev-appro

Please moderate message to wmaker-...@lists.windowmaker.info

2010-04-08 Thread Window Maker Lists
Hello, This mail has been sent by the mailing list manager operating the wmaker-...@lists.windowmaker.info mailing list to the human moderators of the list. Should the message at the bottom be allowed to be sent to the list? If so, reply to this mail or send mail to wmaker-dev-appro

Please moderate message to wmaker-...@lists.windowmaker.info

2010-04-08 Thread Window Maker Lists
Hello, This mail has been sent by the mailing list manager operating the wmaker-...@lists.windowmaker.info mailing list to the human moderators of the list. Should the message at the bottom be allowed to be sent to the list? If so, reply to this mail or send mail to wmaker-dev-appro

Re: poke wmgenmenu some more

2010-04-08 Thread Tamas TEVESZ
On Thu, 8 Apr 2010, Carlos R. Mafra wrote: > The terminal stuff is OK but now it chooses 'mrxvt' while I want it to be > 'xterm'. > So I also applied a patch changing the order in which the apps appear in the > terminal list, with xterm being the first one. duh, i might have forgotten to men

Re: New autoconf stuff breaks debian build

2010-04-08 Thread Martin Dietze
On Thu, April 08, 2010, Carlos R. Mafra wrote: > I've run into something similar here too, but this problem went away after I > run > autogen.sh, iirc. OK, I did a quick check on what's wrong with the build. The debian build uses a build directory into which the Makefiles are generated, so that

New gcc warning

2010-04-08 Thread Carlos R. Mafra
On Wed, 31 Mar 2010 at 4:39:46 +0200, Tamas TEVESZ wrote: > Subject: [PATCH] Style stuff up > diff --git a/util/fontconv.c b/util/fontconv.c > index e470998..e3263b4 100644 > --- a/util/fontconv.c > +++ b/util/fontconv.c [...] > typedef struct str { > char *str; > - int len; > +

Re: New autoconf stuff breaks debian build

2010-04-08 Thread Carlos R. Mafra
On Thu, 8 Apr 2010 at 13:33:12 +0200, Martin Dietze wrote: > > [..] > checking whether ln -s works... yes > checking whether gcc needs -traditional... no > /var/src/windowmaker/wmaker-git/configure: line 11220: syntax error near > unexpected token `-Wall' > /var/src/windowmaker/wmaker-git/config

New autoconf stuff breaks debian build

2010-04-08 Thread Martin Dietze
Hi, just tried to produce the debian build of the current 'crm' and 'next' branches and found that the build is currently broken. First, debian/rules contains a dependency to acinclude.m4 which has disappeared. I fixed this, and now I am running into this here: [..] checking whether ln -s works.

Re: general tidying

2010-04-08 Thread Carlos R. Mafra
On Sat, 3 Apr 2010 at 18:24:16 +0200, Tamas TEVESZ wrote: > Subject: [PATCH] Tidy > > - Kill unused vars in wrlib > - Add missing initializers to defaults.c:staticOptionList > - Re-format it slightly > - Kill some dead code > - Kill stupid "if (const op var)" Nice, it reduced a bit the compilati

Re: poke wmgenmenu some more

2010-04-08 Thread Carlos R. Mafra
On Sat, 3 Apr 2010 at 13:31:58 +0200, Tamas TEVESZ wrote: > Subject: [PATCH] Poke wmgenmenu some more > > - change some variable names to better reflect their purpose, do a little > write-up how a menu is built > - auto-detect what terminal to use for apps in need of a terminal > - fix the chun