Re: [Flightgear-devel] Removing PLIB SL dependencies
Martin Spott wrote: > Here you'll find the summary of both: > > ftp://ftp.ihg.uni-duisburg.de/FlightGear/Devel/FGconfigure.ac.diff Now that the source is release is out and new patches arrive, would anyone bother to apply this one in order to eliminate the remaining dependencies on the PLIB sound library ? Thanks, Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] Removing PLIB SL dependencies
Martin Spott wrote: > I've tested successfully on FreeBSD and IRIX, so here's the second one > on top of the first one: Here you'll find the summary of both: ftp://ftp.ihg.uni-duisburg.de/FlightGear/Devel/FGconfigure.ac.diff Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] Removing PLIB SL dependencies
Martin Spott wrote:
> I'd love to hear if it still builds working binaries. If this is the
> case then I'd post a second patch that removes unused declarations,
I've tested successfully on FreeBSD and IRIX, so here's the second one
on top of the first one:
--- configure.ac.1stTue Jan 18 15:14:46 2005
+++ configure.acTue Jan 18 15:17:50 2005
@@ -147,24 +147,7 @@
AC_DEFINE([HAVE_TIMEZONE], 1, [Define if system has timezone variable])
fi
-dnl add correct audio libs and configure for audio support
-dnl LIBS="-lplibsl -lplibsm"
-
-case "${host}" in
-*-*-cygwin* | *-*-mingw32*)
-LIBS="$LIBS -lwinmm"
-;;
-*-apple-darwin*)
-LIBS="$LIBS -framework IOKit -framework CoreFoundation"
-;;
-*-*-irix* )
-LIBS="$LIBS -laudio"
-;;
-
-esac
-audio_LIBS="$LIBS"
LIBS=""
-AC_SUBST(audio_LIBS)
dnl ENABLE_AUDIO_SUPPORT could be depricated at any time in favor of
dnl just assuming we have audio support on all platform. We can
Martin.
--
Unix _IS_ user friendly - it's just selective about who its friends are !
--
___
Flightgear-devel mailing list
Flightgear-devel@flightgear.org
http://mail.flightgear.org/mailman/listinfo/flightgear-devel
2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] Removing PLIB SL dependencies
Martin Spott wrote:
> I'd love to hear if it still builds working binaries. If this is the
> case then I'd post a second patch that removes unused declarations,
O.k., next time I'll add the patch right to the posting :-)
--- configure.ac.original Tue Jan 18 11:56:48 2005
+++ configure.acTue Jan 18 11:58:56 2005
@@ -148,10 +148,7 @@
fi
dnl add correct audio libs and configure for audio support
-LIBS="-lplibsl -lplibsm"
-
-dnl search for FreeBSD library
-AC_SEARCH_LIBS(hid_init, usbhid)
+dnl LIBS="-lplibsl -lplibsm"
case "${host}" in
*-*-cygwin* | *-*-mingw32*)
@@ -213,6 +210,9 @@
AC_SEARCH_LIBS(cos, m)
AC_SEARCH_LIBS(dlclose, dl)
+dnl search for FreeBSD library
+AC_SEARCH_LIBS(hid_init, usbhid)
+
base_LIBS="$LIBS"
dnl Check for SDL if enabled.
@@ -299,9 +299,6 @@
opengl_LIBS="$LIBS"
LIBS="$base_LIBS"
-
-dnl search for FreeBSD library
-AC_SEARCH_LIBS(hid_init, usbhid)
dnl check for OpenAL libraries
case "${host}" in
Martin.
--
Unix _IS_ user friendly - it's just selective about who its friends are !
--
___
Flightgear-devel mailing list
Flightgear-devel@flightgear.org
http://mail.flightgear.org/mailman/listinfo/flightgear-devel
2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] Removing PLIB SL dependencies
Martin Spott wrote: > 1.) src/Input/Makefile still adds '-plibsl -lplibsm' for linking > 'js_demo'. I think they are not used anymore, the whole > $(audio_LIBS) clause could be removed from this subdir. Could you please test the following patch on your favourite platform ? It's just moving some clauses to the "right" place (TM ;-) I'd love to hear if it still builds working binaries. If this is the case then I'd post a second patch that removes unused declarations, Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] Removing PLIB SL dependencies
"Curtis L. Olson" wrote: >>1.) src/Input/Makefile still adds '-plibsl -lplibsm' for linking >>'js_demo'. I think they are not used anymore, the whole >>$(audio_LIBS) clause could be removed from this subdir. > > I have the vague recollection that some platform depends on this for > some odd reason (???) Remind me after the upcoming release and we can > remove it at the start of the next cycle so we have time to fix it if it > breaks some platform. Yes, we had the case that '-lusbhid' for FreeBSD was placed in the audio libraries compartment. I'm currently looking at it in order to find out if audio_LIBS can be removed safely, Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] Removing PLIB SL dependencies
Curtis L. Olson wrote: Martin Spott wrote: Didn't I already post this ? must have gone lost somewhere on my servers :-) I changed my scripts to build PLIB without 'SL' by default which made be encounter some old but apparently unused dependencies: 1.) src/Input/Makefile still adds '-plibsl -lplibsm' for linking 'js_demo'. I think they are not used anymore, the whole $(audio_LIBS) clause could be removed from this subdir. I have the vague recollection that some platform depends on this for some odd reason (???) Remind me after the upcoming release and we can remove it at the start of the next cycle so we have time to fix it if it breaks some platform. I think it's needed for joystick support on certain platforms. Erik ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
Re: [Flightgear-devel] Removing PLIB SL dependencies
Martin Spott wrote: Didn't I already post this ? must have gone lost somewhere on my servers :-) I changed my scripts to build PLIB without 'SL' by default which made be encounter some old but apparently unused dependencies: 1.) src/Input/Makefile still adds '-plibsl -lplibsm' for linking 'js_demo'. I think they are not used anymore, the whole $(audio_LIBS) clause could be removed from this subdir. I have the vague recollection that some platform depends on this for some odd reason (???) Remind me after the upcoming release and we can remove it at the start of the next cycle so we have time to fix it if it breaks some platform. 2.) FlightGear/src/Sound/beacon.hxx lines 36,37 still include PLIB SL headers, these should be removed Good catch, this is now fixed. Curt. -- Curtis Olsonhttp://www.flightgear.org/~curt HumanFIRST Program http://www.humanfirst.umn.edu/ FlightGear Project http://www.flightgear.org Unique text:2f585eeea02e2c79d7b1d8c4963bae2d ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
[Flightgear-devel] Removing PLIB SL dependencies
Didn't I already post this ? must have gone lost somewhere on my servers :-) I changed my scripts to build PLIB without 'SL' by default which made be encounter some old but apparently unused dependencies: 1.) src/Input/Makefile still adds '-plibsl -lplibsm' for linking 'js_demo'. I think they are not used anymore, the whole $(audio_LIBS) clause could be removed from this subdir. 2.) FlightGear/src/Sound/beacon.hxx lines 36,37 still include PLIB SL headers, these should be removed Maybe more to follow, Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org http://mail.flightgear.org/mailman/listinfo/flightgear-devel 2f585eeea02e2c79d7b1d8c4963bae2d
