> On 7/26/06, Brendan McCane <mccane@...> wrote:
> > G'day Maintainers,
> >
> > I notice that there are a bunch of build errors on the lems.=ADbrown.=
=ADedu
> > Linux-2.=AD6_=ADem64t_=ADgcc-4.=AD0.=AD3_=AD-Wall build. Are these crit=
ical errors
> > or something specific to that setup? Should I ignore them and make a
> > release anyway? Or is someone looking into them?
>
On 7/28/06, Matt Leotta <matt.leotta@...> wrote:
> I run that dashboard build. The errors are all coming from the
> recently added wxWidgets support in vgui by Miguel. The errors will
> all go away if I turn off wxWidgets in CMake. Miguel said this might
> be related to unicode support compiled into wxWidgets. I'm using a
> stock wxWidgets package from Debian, so I would imagine this problem
> is more wide spread. I think most Dashboard builds either don't have
> wxWidgets or it is disabled in the CMake cache, so it is hard to tell
> for sure.
>
> In my opinion these are not critical errors, and you should probably
> go ahead with the release. However, you might want to get feedback
> from Miguel first. He said he was going to look into the problem.
The problem is that the wxWidgets lib that Matt's build is using has
unicode support built in. These incompatibilities should be pretty
easy to fix, but I can't work on this right now as I am in the middle
of reallocating. Probably by the end of august I will have a setup to
work on this. I don't know to what extent Amitha is working on it. He
submitted a patch related to this yesterday, but there is more work to
be done.
These are all compile time errors, which I am able to pick up and work
on because of the dashboard. However, I expect that there are some
run-time issues with other platforms than Windows, since I have only
tested in the windows platform up to now. This is also something I
will put some effort into after I settle down (debugging in a Linux
platform).
That said, I agree with Matt that these are not critical errors and
that the release should be made. There is an option to turn vgui_wx
off in the cmake GUI and the default toolkit is not wxWidgets, but
usually gtk or mfc. The only thing that might be appropriate is to
turn the VGUI_USE_WX OFF by default on this release.
--Miguel
|