-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
soundtouch 1.5.0 upgrade #5381
Comments
Commented by: kwhat |
Commented by: kwhat current sed patch for externalizing.
|
Commented by: kwhat
|
Commented by: rryan Hey KWhat, Thanks for the patch. The version of soundtouch in Ubuntu is 1.3.1 still, and will be for 10.10. Also, we would then have to build SoundTouch for our OSX and Windows packages, so there is more work to be done for getting a patch that would remove SoundTouch from our source tree ready. I do think we should upgrade the version of SoundTouch in Mixxx to 1.4.1 now that the 1.8.0 release is basically over. Regards, |
Commented by: Pegasus-RPG I quickly tested ST v1.5.0 in the 1.7 Mixxx branch a long time ago and ran into no issues on Windows. Building a new version for Windows is not a problem. As for Ubuntu, can't we add it to our PPA? So I'm in favor of doing this in Trunk in time for 1.9.0 as well. |
Commented by: kwhat I have attached a new approach for getting libsoundtouch playing nicely across platforms. Basically it adds an option to externalize it which should address some of the problems with ubuntu and other platforms where a more recent version of soundtouch is not available. I have tested this on linux and it appears to work in both cases. |
Commented by: Pegasus-RPG RJ: if we'd continue to use our own version anyway, why not go with the latest, 1.5.0? |
Commented by: quadrispro Hi, soundtouch 1.5.0 has joined Debian experimental, so please provide us a clean way to avoid building against the local copy of the library. Thanks in advance. |
Commented by: kwhat Hi Alessio, Take a look at the gentoo bug report ( http://bugs.gentoo.org/show_bug.cgi?id=310821 ). The libsoundtouch patches are all up to date (1.9.0 beta 1). Hope that helps. |
Commented by: kwhat I have updated the internal version of sound touch to 1.5.0 in my branch and built successfully. Everything appears to be working fine, more testing is needed. The 1.5.0 external lib has been rock solid for me for the last 4 months and I dont expect the internal to cause any issues. It should be noted that there are some changes to the way optimization is done in the scons script. There is no point in using mmx, sse and 3dnow together. The code will only build with one in the order I listed them in. Because SSE is faster and because you optimize for it at every level > 1 I have added only that version. Apparently this lib can also do int samples... I don't know if that is something that maybe beneficial to mixxx so its disabled in favor of float samples. If int samples are desired, please change #define FLOAT_SAMPLES 1 to 0 and build mmx_optimized.cpp instead of the current sse_optimized.cpp. Additional tweaking maybe required. On a separate note, fidlib-0.9.9 could also use and update to 0.9.10. |
Issue closed with status Fix Released. |
Reported by: kwhat
Date: 2010-04-13T04:14:01Z
Status: Fix Released
Importance: Wishlist
Launchpad Issue: lp562007
Attachments: mixxx-1.7.2.patch, mixxx-1.8.0-libsoundtouch.patch, mixxx-9999-libsoundtouch.patch
looks like sound touch 1.5.0 incorporated in the 1.4.1 patched coded included in mixxx.
I have attached a build patch I adapted from the Debian branch to use the external library. This also includes a small patch to remove invalid qt linker paths. You may wish to remove these.
The text was updated successfully, but these errors were encountered: