----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/45/#review53 -----------------------------------------------------------
I'd prefer the following behavior: Let the user specify whether to use libndofdev or not. (Of course we'd have to agree on a default if nothing was specified). Then ... ... if libndofdev use has been enabled and libndofdev can be found, --> build with libndofdev support, thus with the joystick/spacenavigator features. ... if libndofdev use has been enabled but libndofdev can not be found, --> error out. (With some meaningful message) ... if libndofdev use has been disabled, --> don't build with libndofdev support, even if libndofdev is present. Rationale: If I *want* spacenavigator or joystick support, but libndofdev cannot be found (e.g. because I forgot to install it or because it was installed to a location that isn't searched), I'll get a heads-up at build time rather than having to figure out why stuff doesn't behave the way I want at runtime. Note: This would be analogous to how we currently handle FMOD, even although disabling FMOD -- other than disabling libndofdev -- doesn't result in unavailable viewer features. - Boroondas On 2010-12-19 07:54:19, Aleric Inglewood wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/45/ > ----------------------------------------------------------- > > (Updated 2010-12-19 07:54:19) > > > Review request for Viewer. > > > Summary > ------- > > Only define -DLIB_NDOF=1 when NDOF is actually found, on standalone. While > FindNDOF.cmake was added to the repository, it wasn't used. > > > This addresses bug VWR-10579. > http://jira.secondlife.com/browse/VWR-10579 > > > Diffs > ----- > > indra/cmake/NDOF.cmake b0689af42a71 > > Diff: http://codereview.secondlife.com/r/45/diff > > > Testing > ------- > > I can't remember when I started using this, but it was long ago. I don't have > libndofdev installed. > I just installed Michelle's debian package libndofdev-dev and then it found > it: > -- Found NDOF: Library in '/usr/lib/libndofdev.so' and header in > '/usr/include' > > > Thanks, > > Aleric > >
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges