Re: [ANNOUNCE] Releasing ctwm 3.7 beta3.
On Thu 05 May 2005 at 07:53:04 +0200, Richard Levitte - VMS Whacker wrote: rhialto I think many of the assignments in the Imakefile should use rhialto ?= instead of =, so that they can usefully be set from the rhialto Imakefile.local. I've a problem with that. As far as I know, that's highly dependent on GNU make. Unless we can clearly say that such an assignment operator is supported by all makes out there, I'm not sure I want to go for it. It is also supported by Berkeley Make. Otherwise I would never have proposed it, since I don't use GNU Make if I can avoid it at all. The ?= construct is used a lot in NetBSD system makefiles. But your alternative idea is of course even more portable. That should work even on V7-style Makes. -Olaf. -- ___ Olaf 'Rhialto' Seibert-- rhialto/at/falu.nl \X/ Hi! I'm a signature virus! Copy me to your .signature to help me spread!
Re: [ANNOUNCE] Releasing ctwm 3.7 beta4.
On Thu 05 May 2005 at 13:54:41 +0200, Rhialto wrote: tmp_win-iconify_by_unmapping |= - (short)(int) LookInList(Scr-IconifyByUn, tmp_win-full_name, - tmp_win-class); + LookInList(Scr-IconifyByUn, tmp_win-full_name, + tmp_win-class) != NULL; Perhaps this third one should even better be tmp_win-iconify_by_unmapping = tmp_win-iconify_by_unmapping || LookInList(Scr-IconifyByUn, tmp_win-full_name, tmp_win-class); i.e. foo |= bar != NULL becomes foo = foo || bar, for lack of a ||= operator. . It's more efficient too in case foo is set already. -Olaf. -- ___ Olaf 'Rhialto' Seibert-- rhialto/at/falu.nl \X/ Hi! I'm a signature virus! Copy me to your .signature to help me spread!
Re: [ANNOUNCE] Releasing ctwm 3.7 beta4.
Quoting Richard Levitte - VMS Whacker ([EMAIL PROTECTED]):
Release of ctwm 3.7 beta 4.
As usual, patches are welcome!
The perror(select) command loops in events.c:634 if AnimationSpeed is 0
and USE_SIGNALS is not set.
Here is a patch:
--- events.c-orig Wed May 4 12:23:52 2005
+++ events.cThu May 5 14:39:15 2005
@@ -620,6 +620,7 @@
nextEvent (event);
return;
}
+tout = NULL;
if (animate) tout = (AnimationSpeed 0) ? timeout : NULL;
while (1) {
FD_ZERO (mask);
-Derik.
Re: [ANNOUNCE] Releasing ctwm 3.7 beta4.
In message [EMAIL PROTECTED] on Thu, 5 May 2005 14:52:38 +0200, Derik van
Zuetphen [EMAIL PROTECTED] said:
dz The perror(select) command loops in events.c:634 if AnimationSpeed is 0
dz and USE_SIGNALS is not set.
Ah, that must be the reason why there were those mysterious messages
coming sporadically...
dz Here is a patch:
dz
dz --- events.c-orig Wed May 4 12:23:52 2005
dz +++ events.cThu May 5 14:39:15 2005
dz @@ -620,6 +620,7 @@
dz nextEvent (event);
dz return;
dz }
dz +tout = NULL;
dz if (animate) tout = (AnimationSpeed 0) ? timeout : NULL;
dz while (1) {
dz FD_ZERO (mask);
Almost applied. I did it a little higher up, by initialising tout
where it's defined.
Thanks for the notification. I think it's time I do a round with
GCC_PEDANTIC...
Cheers,
Richard
-
Please consider sponsoring my work on free software.
See http://www.free.lp.se/sponsoring.html for details.
--
Richard Levitte [EMAIL PROTECTED]
http://richard.levitte.org/
When I became a man I put away childish things, including
the fear of childishness and the desire to be very grown up.
-- C.S. Lewis
Re: [ANNOUNCE] Releasing ctwm 3.7 beta4.
On Thu 05 May 2005 at 15:10:31 +0200, Richard Levitte - VMS Whacker wrote: Which gcc and what are your default options? I didn't get any warnings about that. Either way, thanks for the patch, I just applied it. gcc -v tells me: Using built-in specs. Configured with: /home/nick/work/netbsd/src/tools/gcc/../../gnu/dist/gcc/configure --enable-long-long --disable-multilib --enable-threads --disable-symvers --build=i386-unknown-netbsdelf --host=alpha--netbsd --target=alpha--netbsd Thread model: posix gcc version 3.3.3 (NetBSD nb3 20040520) These are the options (and the warnings): cc -O2-I/usr/pkg/include -I/usr/pkg/include -I/usr/X11R6/include -DCSRG_BASED -DFUNCPROTO=15 -DNARROWPROTO-DUSEM4 -DGNOME -DXPM -DJPEG -DX11R6 -DI18N -I/usr/pkg/include -I/usr/X11R6/include -c add_window.c add_window.c: In function `AddWindow': add_window.c:399: warning: cast from pointer to integer of different size add_window.c:416: warning: cast from pointer to integer of different size add_window.c:420: warning: cast from pointer to integer of different size I think I see the warnings because pointer and integer are actually of different size on the Alpha (64 bits). I don't see the warnings on my laptop, which is a 32 bit processor. It's probably old history, and might even come all the way from twm... I expect so. I browsed through cvs and these casts were already present in the first version of the file (1.1). However some of them were removed in revision 1.2, among lots of other changes (the log message says that those are all changes Claude had done since version 3.6). These were the last of those casts remaining. I didn't look into it in more detail. Richard -Olaf. -- ___ Olaf 'Rhialto' Seibert-- rhialto/at/falu.nl \X/ Hi! I'm a signature virus! Copy me to your .signature to help me spread!
Re: [ANNOUNCE] Releasing ctwm 3.7 beta4.
In message [EMAIL PROTECTED] on Thu, 5 May 2005 14:52:38 +0200, Derik van Zuetphen [EMAIL PROTECTED] said: dz The perror(select) command loops in events.c:634 if dz AnimationSpeed is 0 and USE_SIGNALS is not set. This made me look further, using the following in Imakefile.local (I'm not going to care about shadowed variables for 3.7): #undef GCC_PEDANTIC CCOPTIONS = -pedantic -ansi -Wall \ -Wstrict-prototypes -Wmissing-prototypes -Wundef \ -aux-info [EMAIL PROTECTED] Worked like a charm, and I corrected a whole bunch of warnings, mostly having to do with signed vs. unsigned conflist, badly declared functions, uninitialized variables, stuff like that. Thanks, Derik, that was a good push :-). Cheers, Richard - Please consider sponsoring my work on free software. See http://www.free.lp.se/sponsoring.html for details. -- Richard Levitte [EMAIL PROTECTED] http://richard.levitte.org/ When I became a man I put away childish things, including the fear of childishness and the desire to be very grown up. -- C.S. Lewis
Re: [ANNOUNCE] Releasing ctwm 3.7 beta4.
In message [EMAIL PROTECTED] on Thu, 5 May 2005 15:26:21 +0200, Rhialto [EMAIL PROTECTED] said: rhialto I think I see the warnings because pointer and integer are rhialto actually of different size on the Alpha (64 bits). I don't rhialto see the warnings on my laptop, which is a 32 bit processor. Ah, good point. Hmm, I've an AlphaStation lying around, maybe I should install something on it and play. Now, the choice stands between Linux (Debian), NetBSD and OpenBSD, I wonder which to choose. rhialto It's probably old history, and might even come all the way rhialto from twm... rhialto rhialto I expect so. I browsed through cvs and these casts were rhialto already present in the first version of the file (1.1). rhialto However some of them were removed in revision 1.2, among lots rhialto of other changes (the log message says that those are all rhialto changes Claude had done since version 3.6). (Hooray for Claude!) rhialto These were the last of those casts remaining. I didn't look rhialto into it in more detail. OK, I trust your build system to tell us otherwise :-). Thanks! Cheers, Richard - Please consider sponsoring my work on free software. See http://www.free.lp.se/sponsoring.html for details. -- Richard Levitte [EMAIL PROTECTED] http://richard.levitte.org/ When I became a man I put away childish things, including the fear of childishness and the desire to be very grown up. -- C.S. Lewis
Re: minor spec file updates (against 3.7beta3)
Richard Levitte - VMS Whacker [EMAIL PROTECTED] writes: Actually, I want to question this. Should ctwm use /etc/X11/ctwm and /usr/X11R6/lib/X11/ctwm? That way, twm and ctwm can easily live side by side. Sounds like a good idea to me. -- Johan
Re: [ANNOUNCE] Releasing ctwm 3.7 beta3.
In message [EMAIL PROTECTED] on Thu, 05 May 2005 17:05:19 +0200, [EMAIL PROTECTED] said: ai26 In a message of Wed, 04 May 2005 13:04:28 +0200 (CEST) ai26 Received on Wed, 04 May 2005 13:07:32 +0200 ai26 ai26 Richard Levitte - VMS Whacker [EMAIL PROTECTED] wrote to lp-free-ctwm@lp.se ai26 ai26 Ok, I tried (beta4). It's quite a bit better but on this oldish ai26 VMS 7.2 there just isn't a smlib.h. I've searched in all of ai26 sys$sysroot. Maybe it's hidden in some .tlb but all of ai26 DECW$INCLUDE is in single files. OK ai26 I then undefined the USE_SESSION in the CFLAGS but it still ai26 tries to include session support (whatever that is): And that was an error on my part. What you should undefine is X11R6. Unfortunately, it isn't checked very well, so session.h gets included no matter what. I'll work on it. I've done all my tests so far on an Alpha running VMS 7.3-2, but I do have a VAX running 7.2, so I'll do some tests there as well. ai26 A400 cc/vers ai26 DEC C V5.7-004 on OpenVMS Alpha V7.2-1 WOW! That's an OLD compiler! This is what I have (can't find older): $ cc/vers Compaq C V6.2-003 on OpenVMS VAX V7.2 Cheers, Richard - Please consider sponsoring my work on free software. See http://www.free.lp.se/sponsoring.html for details. -- Richard Levitte [EMAIL PROTECTED] http://richard.levitte.org/ When I became a man I put away childish things, including the fear of childishness and the desire to be very grown up. -- C.S. Lewis
Re: [ANNOUNCE] Releasing ctwm 3.7 beta4.
On Thu, May 05, 2005 at 09:58:30AM +0200 I heard the voice of Richard Levitte - VMS Whacker, and lo! it spake thus: Release of ctwm 3.7 beta 4. A little slower this time :| FreeBSD port updated to b4. http://www.over-yonder.net/~fullermd/dl/ctwm-b4-port.tar.gz -- Matthew Fuller (MF4839) | [EMAIL PROTECTED] Systems/Network Administrator | http://www.over-yonder.net/~fullermd/ On the Internet, nobody can hear you scream.
Re: [ANNOUNCE] Releasing ctwm 3.7 beta4.
On Thu 05 May 2005 at 17:46:25 -0500, Matthew D. Fuller wrote: A little slower this time :| FreeBSD port updated to b4. http://www.over-yonder.net/~fullermd/dl/ctwm-b4-port.tar.gz And for NetBSD: http://www.falu.nl/~rhialto/ctwm-current.shar -Olaf. -- ___ Olaf 'Rhialto' Seibert-- rhialto/at/falu.nl \X/ Hi! I'm a signature virus! Copy me to your .signature to help me spread!
