279375 – [patch] easytag fails during cddb search

Bug 279375 - [patch] easytag fails during cddb search

Summary: [patch] easytag fails during cddb search

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	Gentoo Sound Team

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2009-07-27 19:10 UTC by GregS
Modified:	2009-12-27 14:05 UTC (History)
CC List:	5 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Patch for easytag that is fixing the problem (easytag-2.1.6-cddb_search.patch,1.91 KB, patch) 2009-07-27 19:12 UTC, GregS	Details \| Diff
New ebuild that contains previously attached patch (easytag-2.1.6-r2.ebuild,1.47 KB, text/plain) 2009-07-27 19:13 UTC, GregS	Details
View All Add an attachment (proposed patch, testcase, etc.)

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

Description GregS 2009-07-27 19:10:45 UTC

While trying to search cddb manually, search fails showing '.HTTP/1.1 301 Moved Permanentl' error.

Reproducible: Always

Steps to Reproduce:
1. Search cddb withing easytag manually
2.
3.



Expected Results:  
Search should return a result from search query.

Comment 1 GregS 2009-07-27 19:12:28 UTC

Created attachment 199373 [details, diff]
Patch for easytag that is fixing the problem

Comment 2 GregS 2009-07-27 19:13:22 UTC

Created attachment 199375 [details]
New ebuild that contains previously attached patch

Comment 3 Wormo (RETIRED) gentoo-dev

2009-07-28 22:23:09 UTC

Thanks for the report, and for submitting your solution! Assigning to easytag maintainers.

Comment 4 Sebastian Pipping gentoo-dev

2009-09-23 15:02:28 UTC

Greg, have you proposed the patch upstream?  What do they say?

Comment 5 Bruno 'brubru' Tarquini 2009-12-19 22:37:07 UTC

Upstream has posted a patch for this: http://sourceforge.net/projects/easytag/files/, patch_easytag_216_cddb_manual_search_fix.diff.

debian (and ubuntu) have applied it: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=527216 & https://bugs.launchpad.net/ubuntu/+source/easytag/+bug/384929

Worked fine for me.

Comment 6 George Kargiotakis 2009-12-21 15:04:35 UTC

this patch works fine, please apply it to the tree

Comment 7 Samuli Suominen (RETIRED) gentoo-dev

2009-12-21 15:18:45 UTC

(In reply to comment #5)
> Upstream has posted a patch for this:
> http://sourceforge.net/projects/easytag/files/,
> patch_easytag_216_cddb_manual_search_fix.diff.

Applied the upstream patch to -r3

+*easytag-2.1.6-r3 (21 Dec 2009)
+
+  21 Dec 2009; Samuli Suominen <ssuominen@gentoo.org>
+  +easytag-2.1.6-r3.ebuild, +files/easytag-2.1.6-cddb.patch:
+  Upstream patch for CDDB searching wrt #279375.

Comment 8 GregS 2009-12-27 14:00:37 UTC

The upstream patch for easytag 2.1.6-r3 doesn't work for me.
I have an error when searching manually:

HTTP/1.0 301 Moved Permanently
Date: Sun, 27 Dec 2009 13:58:09 GMT
Server: Apache
X-Powered-By: PHP/5.2.4-2ubuntu5.7
location: http://www.gnudb.org/search/gnudb.org?page=0
Vary: Accept-Encoding
Content-Length: 0
Connection: close
Content-Type: text/html

Comment 9 GregS 2009-12-27 14:05:04 UTC

removing the ~/.easytag directory solved the problem.