M4A plugin fails to load, unable to resolve symbol

Bug #610746 reported by William Good
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Undecided
William Good

Bug Description

When trying to load the M4A plugin, trunk head and 1.8 head say:
Debug: [Main]: Failed to dynamically load "./lin32_build/plugins/lin32_build/libsoundsourcem4a.so" "Cannot load library ./lin32_build/plugins/lin32_build/libsoundsourcem4a.so: (./lin32_build/plugins/lin32_build/libsoundsourcem4a.so: undefined symbol: _ZN11SoundSource12getCuePointsEv)"

Think I found the problem, working on a patch (researching this has only taken approximately 1 million ~6 minute total rebuilds of mixxx).

William Good (bkgood)
Changed in mixxx:
assignee: nobody → Bill Good (bkgood)
status: New → In Progress
Revision history for this message
William Good (bkgood) wrote :

This patch fixes it. Basically, in rev 2467 (1.8 branch, same change happened at 2432 in trunk), the SConscript for the m4a plugin lost instruction to build soundsource.cpp with soundsourcem4a.cpp, and it's apparently needed to subclass (makes since, since the getCuePoint method the error talks about would be public on soundsourcem4a too). In order for the m4a sconscript to get at the soundsource.cpp file in src directory, I had to add a VariantDir SCons instruction right before the SConscript call to the plugins SConscript, which basically tells scons that, if it can't find a file while looking in plugins/soundsourcem4a, look in src/. This would have to be repeated for any other plugins that came along -- sucks, but I really couldn't find a way around it. SConscript() does take an argument variant_dir, but it was making everything go fubar for me.

Revision history for this message
William Good (bkgood) wrote :

Also (OT), according to http://www.scons.org/doc/HTML/scons-user/a8524.html#b-SharedLibrary , LoadableModule might be a better idea for what we're doing -- no idea if the plugin works under OSX now.

Revision history for this message
ironstorm (ironstorm-gmail) wrote :

I have a trunk build from last week on osx and the plugin works

Revision history for this message
William Good (bkgood) wrote :

Patch applied in 1.8 r2502, trunk pending merge.

Changed in mixxx:
status: In Progress → Fix Committed
RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/5464

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.