Re: [Freeglut-developer] Concerning Release Candidate 3

[Fay, J. F Dr C. U. A. 46 S. <joh...@eg...>]

Martin,

	While there may be a way to fix it, I don't think I would like
to hold up the release for it.  As General Patton was fond of saying, "a
good plan today is better than a perfect plan tomorrow."  I would like
to get a release out the door--it's about two years overdue as it
is--and deal with the more complicated issues later.

	I plan to start making noises about 2.8.0 some time around the
next equinox.

John F. Fay
Technical Fellow
Jacobs Technology TEAS Group
850-883-1294 

-----Original Message-----
From: Martin Payne [mailto:li...@ma...] 
Sent: Saturday, November 21, 2009 2:18 PM
To: FreeGLUT developers list
Subject: Re: [Freeglut-developer] Concerning Release Candidate 3

Hi John, all,

John Tsiombikas wrote:
> That's no good for me, can't really test it. This patch raises the
> required autoconf version to 2.64 which is not even in Debian unstable
> at the moment.
>
> >From inspection of the patch I can't see what's relevant
> to the mac os x build. It seems to focus on fixing some ifdef blocks
> checking things such as the presence of errno.h and various standard
> UNIX headers.

Yep, same for me. I had to modify the "autoconfigure.ac" to build it 
successfully on Fedora 11 (x64 version), but I couldn't really see what 
the patch fixes. I did test your full screen fix at the same time 
though, and confirm that full screen under Gnome is now truly full
screen.

I have also just noted a problem with the errno.h ifdef blocks under 
Windows. When compiling under VS 2008, I'm getting a warning about an 
unused variable:

Warning    9    warning C4101: 'sLen' : unreferenced local variable    
c:\Users\mpayne\Documents\Visual Studio 2008\Projects\freeglut 
Trunk\freeglut\freeglut\src\freeglut_init.c    689    freeglut

The problem is that "HAVE_ERRNO" is not defined when building under VS 
2008, so the code that uses the variable is conditionally not compiled. 
The patch changes some of these ifdef blocks (not all of them) to check 
"HAVE_ERRNO_H" instead of "HAVE_ERRNO", but still that is not something 
which is defined when building under VS 2008. My understanding was that 
this was put in place to avoid errors when building for WinCE (see 
http://sourceforge.net/mailarchive/forum.php?thread_name=4af805db3dd89fa
490675f1edd6dc613%40mail.smartmobili.com&forum_name=freeglut-developer).

Is there something better we can replace this with, so the code compiles

under Windows, but not WinCE?

Regards,
Martin

------------------------------------------------------------------------
------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008
30-Day 
trial. Simplify your report design, integration and deployment - and
focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Freeglut-developer mailing list
Fre...@li...
https://lists.sourceforge.net/lists/listinfo/freeglut-developer