Bug 561635 - LADSPA audacious-plugin broken
Summary: LADSPA audacious-plugin broken
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: audacious-plugins
Version: 12
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Michael Schwendt
QA Contact: Fedora Extras Quality Assurance
URL: http://jira.atheme.org/browse/AUDPLUG...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-02-04 03:25 UTC by moony
Modified: 2010-02-09 05:04 UTC (History)
2 users (show)

Fixed In Version: 2.2-17.fc12
Clone Of: 533641
Environment:
Last Closed: 2010-02-09 05:04:55 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description moony 2010-02-04 03:25:39 UTC
+++ This bug was initially created as a clone of Bug #533641 +++

Description of problem:
Audacious does not remember the settings of its LADSPA plugin after a restart of audacious. 

Version-Release number of selected component (if applicable):
audacious-plugins-2.1-12

How reproducible:
Always, AFAIK.

Steps to Reproduce:
1.  Install LADSPA packages and audacious
2.  Configure LADSPA plugin within audacious by adding a LADSPA plugin and editing its settings.
3.  Restart audacious.
  
Actual results:
Settings not remembered.

Expected results:
Settings remembered.

Additional info:
See http://jira.atheme.org/browse/AUDPLUG-2 .  I have successfully built the ladpsa plugin by using the audcious 2.0 version of ladspa.c and ladspa.h.  I can post them if necessary.

--- Additional comment from moony on 2009-11-07 22:13:44 EST ---

Fixed source RPM available here:
http://www.infinality.net/fedora/linux/12/SRPMS/audacious-plugins-2.1-13.2.fc12.src.rpm

--- Additional comment from mschwendt on 2009-11-08 07:08:37 EST ---

Applied in F-12 (not yet in devel). I've used the 2.0.1 version, though.

--- Additional comment from updates on 2009-11-08 07:13:37 EST ---

audacious-plugins-2.1-13.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/audacious-plugins-2.1-13.fc12

--- Additional comment from moony on 2009-11-08 10:34:10 EST ---

(In reply to comment #2)
> Applied in F-12 (not yet in devel). I've used the 2.0.1 version, though.  

Ah yes... I should have been more accurate.  That's the same version I used as well.

--- Additional comment from moony on 2009-11-09 19:40:39 EST ---

Verified fixed in your posted build.  (tested x86_64 version)

--- Additional comment from updates on 2009-11-10 12:42:11 EST ---

audacious-plugins-2.1-13.fc12 has been pushed to the Fedora 12 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update audacious-plugins'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2009-11158

--- Additional comment from updates on 2009-11-16 02:33:34 EST ---

audacious-plugins-2.1-16.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 1 moony 2010-02-04 03:26:15 UTC
This bug is present again in the recent 2.2 update.

Comment 2 Michael Schwendt 2010-02-04 09:25:12 UTC
Where have you been when Audacious 2.2 was in updates-testing for over a month?
https://admin.fedoraproject.org/updates/F12/FEDORA-2010-0028

Comment 3 Michael Schwendt 2010-02-04 11:14:15 UTC
Please tell me whether this fix would be sufficient:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1962578

Comment 4 Fedora Update System 2010-02-05 01:49:45 UTC
audacious-plugins-2.2-17.fc12 has been pushed to the Fedora 12 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update audacious-plugins'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2010-1508

Comment 5 moony 2010-02-05 03:02:33 UTC
Well, the updated audacious-plugins helped, but it didn't fix it completely.  In the ~/.config/audacious/config file, under [ladspa], for some reason it had "plugins=0".  Perhaps this happened somehow with the 2.2 update?

But, I'm running your latest audacious-plugins now and changed it to "plugins=5" (which is how many I previously had running).  Now, it works, but it required that config change, so not sure how to classify the status of the bug at this point.

Thanks!

Comment 6 moony 2010-02-05 03:03:24 UTC
(In reply to comment #2)
> Where have you been when Audacious 2.2 was in updates-testing for over a month?
> https://admin.fedoraproject.org/updates/F12/FEDORA-2010-0028    

Busy at work!

Comment 7 Michael Schwendt 2010-02-05 10:08:17 UTC
> In the ~/.config/audacious/config file, under [ladspa], for
> some reason it had "plugins=0".  Perhaps this happened somehow
> with the 2.2 update?

Yes, 2.2 saved crap.

What had happened prior to the 2.2 release was that the ladspa plugin's restore() function was still commented out in the source code. Apparently because it didn't build anymore due to broken/work-in-progress changes in other parts of the same file since 2.1.  So the plugin did not even try to load'n'restore from the config file. Additionally, it saved values to the config file, which were broken. plugins=0 was just one. It saved .ini file section names, which were incompatible with what the broken restore() function tried to implement.

My fix mimics the config files from 2.0.1 and older. But there is no guarantee that a future version upgrade won't be incompatible. It might be that 2.3 or a later release will switch to a more complex .ini file section naming scheme for configured ladspa plugins. For example, I could imagine how the broken/incomplete implementation in 2.2 would work if finished properly, but I don't like that solution (as it would be more work to iterate over the configured plugins and their config file sections).

Comment 8 Fedora Update System 2010-02-09 05:04:47 UTC
audacious-plugins-2.2-17.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.