You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have opened a github PR #700 for configure to support --without-vm-sound-Sun so that the plugin can be disabled on the configure command line.
The plugin may still be useful on some Solaris platforms and compiles/builds fine, so I am not proposing to remove vm-sound-Sun, just to be able to disable it through "configure".
For this to become active, can somebody please regenerate the configure script from configure.in after accepting (testing?) and approving my PR #700 .
Thanks,
David Stes
The text was updated successfully, but these errors were encountered:
Thanks for approving the PR. I am currently using the "pulseaudio" audio plugin but it may be useful to keep the older audio plugins such as the vm-sound-Sun and vm-sound-OSS available. vm-sound-Sun builds/compiles ok but so does vm-sound-pulse. So by using configure, this can be chosen at compile time which plugins to build.
The vm-sound-OSS (for opensound) plugin has a similar option to disable it (or enable it by default if it finds the required header file). Would it be possible please that the maintainer of OpenSmalltalk "configure" (I think this is Tobias Pape) please update the configure script now that the vm-sound-Sun plugin has a similar option as the one for vm-sound-OSS ? Thanks to Tobias Pape (or other maintainer(s) of OpenSmalltalk configure) for the excellent work on this. Thank you.
I have opened a github PR #700 for configure to support --without-vm-sound-Sun so that the plugin can be disabled on the configure command line.
The plugin may still be useful on some Solaris platforms and compiles/builds fine, so I am not proposing to remove vm-sound-Sun, just to be able to disable it through "configure".
For this to become active, can somebody please regenerate the configure script from configure.in after accepting (testing?) and approving my PR #700 .
Thanks,
David Stes
The text was updated successfully, but these errors were encountered: