[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2011-01-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

Terje Røsten terje...@phys.ntnu.no changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-01-06 14:22:12

--- Comment #9 from Terje Røsten terje...@phys.ntnu.no 2011-01-06 14:22:12 
EST ---
Imported, built and pushed to testing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2011-01-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

--- Comment #8 from Kevin Fenzi ke...@tummy.com 2011-01-02 15:11:40 EST ---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2010-12-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

--- Comment #6 from Michael Schwendt mschwe...@gmail.com 2010-12-28 04:39:42 
EST ---
Created attachment 470930
  -- https://bugzilla.redhat.com/attachment.cgi?id=470930
puddletag exception freeze with non-initialized quodlibet

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2010-12-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

Michael Schwendt mschwe...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mschwe...@gmail.com

--- Comment #5 from Michael Schwendt mschwe...@gmail.com 2010-12-28 04:38:51 
EST ---
Good.

[...]

I've played a bit more with the run-time aspects of the app and have run into
two problems:

First, trouble testing the Music Library feature. A yum install quodlibet was
necessary to make sense of the Tools  Import Music Library menu, which only
said:

 | No supported music libraries were found. Most likely the
 | required dependencies aren't installed. Visit the puddletag
 | website, puddletag.sourceforge.net for more details.

Then reselecting the menu and loading QuodLibet made puddletag freeze (will
attach the Python exception). Turned out one has to actually run the quodlibet
app first to init some files. To reproduce:

 1. rm -rf $HOME/.quodlibet
 2. puddletag
 3. open menu Tools  Import Music Library
 4. select QuodLibet and load it

[...]

Second issue was that the preferences default to using amarok -p as audio
player. On systems where Amarok is not installed (all Fedora non-KDE default
installs!), no error/warning dialog is displayed when trying to play a file. If
one enters the preferences and removes the default amarok -p setting, there
is a warning, at least.

I wonder whether there could be a default better than amarok -p? Maybe use
xdg-open (and a dependency on it)?

[...]

Apart from that: APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2010-12-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

Michael Schwendt mschwe...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-review+

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2010-12-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

Terje Røsten terje...@phys.ntnu.no changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from Terje Røsten terje...@phys.ntnu.no 2010-12-28 16:13:07 
EST ---
Thanks! I will have a look at these issues before pushing to stable.


New Package SCM Request
===
Package Name: puddletag
Short Description: Feature rich, easy to use tag editor
Owners: terjeros
Branches: f13 f14
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2010-12-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

--- Comment #3 from Terje Røsten terje...@phys.ntnu.no 2010-12-27 14:58:44 
EST ---
Thanks, updated package:
- 0.9.11
- fix license
- add comment about py reqs
- add slash to dir in files
- fix typo in description
- remove buildroot tag
- remove define of python macro
- remove python req
- fix sed expression

spec: http://terjeros.fedorapeople.org/puddletag/puddletag.spec
srpm:
http://terjeros.fedorapeople.org/puddletag/puddletag-0.9.11-1.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2010-12-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

--- Comment #4 from Terje Røsten terje...@phys.ntnu.no 2010-12-27 17:42:07 
EST ---
koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2690582

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2010-12-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

Bug 645277 depends on bug 495324, which changed state.

Bug 495324 Summary: New version of pyparsing is available (1.5.1)
https://bugzilla.redhat.com/show_bug.cgi?id=495324

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|MODIFIED
 Status|MODIFIED|ON_QA
 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #1 from Terje Røsten terje...@phys.ntnu.no 2010-12-06 04:44:00 
EST ---
Update to latest release 0.9.7:

spec: http://terjeros.fedorapeople.org/puddletag/puddletag.spec
srpm: http://terjeros.fedorapeople.org/puddletag/puddletag-0.9.7-1.fc14.src.rpm
koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2646294

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

2010-10-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=645277

Terje Røsten terje...@phys.ntnu.no changed:

   What|Removed |Added

 Depends on||495324

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review