Bug 607778

"sqlite3-devel" isn't found here. Will see how far it goes with "sqlite-devel".

Yes, that should be sqlite-devel. I added the sqlite dependency after the last local SRPM rebuild, and ought to have tested the final SRPM too. Apologies.

$ sha1sum xnoise-plugins-core-0.1.6.tar.gz 
8eb208d9d3cc263e36904216d17520bc00b89a67  xnoise-plugins-core-0.1.6.tar.gz


* "BuildRequires: intltool" is missing


* MediaWatcher plugin crashes:
  1) start xnoise
  2) open Preferences > Plugins
  3) enable MediaWatcher
  4) disable MediaWatcher
  5) enable MediaWatcher
  6) GLib-GObject warnings on stderr + segfault


* LastfmCovers plugin crashes on exit:
  1) start xnoise
  2) enable LastfmCovers plugin
  3) quit
  4) segfault


* Notifications plugin prints "<b> ... </b>"


* /usr/lib64/xnoise/*.la files are not needed


* xnoise-plugins-core.x86_64: E: incorrect-locale-subdir /usr/share/locale/default/LC_MESSAGES/xnoise-plugins-core.mo


* %{_libdir}/xnoise ownership : currently owned by the -plugins-core package, which is optional, however. Would separate xnoise plugin packages need to own %{_libdir}/xnoise  or do you think it would be better to make the base "xnoise" package own that directory?


* xnoise-plugins-core-devel : What's the purpose of this package? The current %description can't be correct. ;-)


* Rest is fine.

(In reply to comment #3)

Updated SRPM:
http://salimma.fedorapeople.org/specs/gnome/xnoise-plugins-core-0.1.6-2.fc13.src.rpm

> * "BuildRequires: intltool" is missing
Fixed
> 
> 
> * MediaWatcher plugin crashes:
>   1) start xnoise
>   2) open Preferences > Plugins
>   3) enable MediaWatcher
>   4) disable MediaWatcher
>   5) enable MediaWatcher
>   6) GLib-GObject warnings on stderr + segfault
Confirmed; reporting upstream and disabling for now
> 
> 
> * LastfmCovers plugin crashes on exit:
>   1) start xnoise
>   2) enable LastfmCovers plugin
>   3) quit
>   4) segfault
Confirmed. Looks like something broke between the last development version I tried before 0.1.6 (0.1.2 or so), and I did not notice because my albums already have their cover images cached

Disabling for now -- is this the error you get?
Failed to open file '/home/michel/.config/xnoise/album_images/kate_walsh/tim's_house/tim's_house_medium': No such file or directory
Failed to open file '/home/michel/.config/xnoise/album_images/kate_walsh/tim's_house/tim's_house_medium': No such file or directory
*** glibc detected *** xnoise: malloc(): smallbin double linked list corrupted: 0x0000000001006cf0 ***


> * Notifications plugin prints "<b> ... </b>"
Fixed; patch sent upstream

> * /usr/lib64/xnoise/*.la files are not needed
Done
> 
> 
> * xnoise-plugins-core.x86_64: E: incorrect-locale-subdir
> /usr/share/locale/default/LC_MESSAGES/xnoise-plugins-core.mo
> 
> 
> * %{_libdir}/xnoise ownership : currently owned by the -plugins-core package,
> which is optional, however. Would separate xnoise plugin packages need to own
> %{_libdir}/xnoise  or do you think it would be better to make the base "xnoise"
> package own that directory?
That is true. I'll make xnoise owns it, but since xnoise is already in the override tag, the switch could probably happen after the initial xnoise/xnoise-plugins-core combo is released.

> * xnoise-plugins-core-devel : What's the purpose of this package? The current
> %description can't be correct. ;-)
Actually .. I'm not sure. the .pc file does not actually specify where plugins should be installed, but it does at least specify how to find the xnoise package. I have adjust the description.

Okay. The segfaults in the plugins will be left to you anyway. There are more, e.g. when starting from scratch and enabling "Leoslyrics". ABRT doesn't catch them:

$ rm -rf ~/.config/xnoise
$ xnoise
        LastfmCovers plugin failed to activate!
        Mediawatcher plugin failed to activate!
*** glibc detected *** xnoise: malloc(): smallbin double linked list corrupted: 0x00000000015f5f50 ***

$ rm -rf ~/.config/xnoise
$ xnoise
        LastfmCovers plugin failed to activate!
        Mediawatcher plugin failed to activate!
closing...
Segmentation fault (core dumped)


JFTR, the review guidelines say:

| SHOULD: The reviewer should test that the package functions as
| described. A package should not segfault instead of running, for example.

We could go on with even more disabled plugins. Eventually, no plugin may be left. ;) Or old/new segfaults might return with one of the next updates after package approval.

Whether 0.1.6 is ready for Rawhide, I leave this decision to you. 

[...]

About the .pc file: Consider merging this dubious -devel subpackage into the xnoise-plugins-core package. Possibly with a virtual -devel pkg or only when you learn about something that would need such a BuildRequires. Rationale: xnoise-plugins-core builds the plugins with the .pc file in xnoise-devel, so external plugins will likely use that .pc file, too.

[...]

Other issues have been dealt with. As the xnoise-plugins-core-devel pkg may return after the review process or be killed again in future updates, it isn't necessary to insist on its removal during this review.

APPROVED

New Package CVS Request
=======================
Package Name: xnoise-plugins-core
Short Description: Core plugins for xnoise
Owners: salimma
Branches: F-12 F-13
InitialCC: 

(In reply to comment #5)
> JFTR, the review guidelines say:
> 
> | SHOULD: The reviewer should test that the package functions as
> | described. A package should not segfault instead of running, for example.
> 
Apologies there. The plugins were in working order at some point before 0.1.6 came out, but I still should have not assumed that's still the case with the released versions. I guess xnoise is still in flux and the plugins lag a bit behind xnoise itself.

> We could go on with even more disabled plugins. Eventually, no plugin may be
> left. ;) Or old/new segfaults might return with one of the next updates after
> package approval.
> 
> Whether 0.1.6 is ready for Rawhide, I leave this decision to you. 
It's ready for Rawhide, I guess, but I'd probably keep it there and not push it to F-12 and F-13 until they are more stable.

> 
> [...]
> 
> About the .pc file: Consider merging this dubious -devel subpackage into the
> xnoise-plugins-core package. Possibly with a virtual -devel pkg or only when
> you learn about something that would need such a BuildRequires. Rationale:
Good idea; will delete it for now

CVS done (by process-cvs-requests.py).

xnoise-plugins-core-0.1.6-2.fc13,xnoise-0.1.6-2.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/xnoise-plugins-core-0.1.6-2.fc13,xnoise-0.1.6-2.fc13

xnoise-plugins-core-0.1.6-2.fc13, xnoise-0.1.6-2.fc13 has been pushed to the Fedora 13 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 xnoise-plugins-core xnoise'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/xnoise-plugins-core-0.1.6-2.fc13,xnoise-0.1.6-2.fc13

xnoise-plugins-core-0.1.6-2.fc13, xnoise-0.1.6-2.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.