Bug 522306

Summary: Incorrect sanity check patch used in alexandria-0.6.5-7.fc10.1 build
Product: [Fedora] Fedora Reporter: Aaron Clark <ad2clark>
Component: alexandriaAssignee: Mamoru TASAKA <mtasaka>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 10CC: mtasaka
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
URL: http://rubyforge.org/tracker/?group_id=205&atid=863&func=detail&aid=27000
Whiteboard:
Fixed In Version: 0.6.5-7.fc10.2 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-09-11 23:25:46 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

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.