Bug 522306 - Incorrect sanity check patch used in alexandria-0.6.5-7.fc10.1 build
Summary: Incorrect sanity check patch used in alexandria-0.6.5-7.fc10.1 build
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: alexandria
Version: 10
Hardware: x86_64
OS: Linux
low
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL: http://rubyforge.org/tracker/?group_i...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-09-10 02:56 UTC by Aaron Clark
Modified: 2009-09-11 23:25 UTC (History)
1 user (show)

Fixed In Version: 0.6.5-7.fc10.2
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-09-11 23:25:46 UTC


Attachments (Terms of Use)

Description Aaron Clark 2009-09-10 02:56:24 UTC
Description of problem:
The modified sanity check patch was only added to the alexandria-0.6.5-7.fc11.1 build, so ISBN search remains broken on F10.

Version-Release number of selected component (if applicable):
alexandria-0.6.5-7.fc10.1

How reproducible:
Always

Steps to Reproduce:
1. Compare the contents of alexandria-0.6.5-search-result-sanity-check.patch for the F11 and F10 builds
  
Actual results:
F10 build still adds the following line:
results.delete_if { |book, cover| ((!book) || (!cover)) }

Expected results:
F10 build correctly adds the following line instead (use the patch from the F11 build):
results.delete_if { |book, cover| book.nil? }

Additional info:
See link for original upstream bug and comments

Comment 1 Mamoru TASAKA 2009-09-10 06:17:07 UTC
Ah, sorry...

Comment 2 Fedora Update System 2009-09-10 06:19:02 UTC
alexandria-0.6.5-7.fc10.2 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/alexandria-0.6.5-7.fc10.2

Comment 3 Aaron Clark 2009-09-11 15:36:50 UTC
The new build looks good, you can mark it as fixed.

Comment 4 Fedora Update System 2009-09-11 23:25:41 UTC
alexandria-0.6.5-7.fc10.2 has been pushed to the Fedora 10 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.