From: Miguel A. Figueroa-V. <mi...@ie...> - 2006-07-28 14:51:21
|
> On 7/26/06, Brendan McCane <mc...@cs...> 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 <mat...@gm...> 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 |