Bug 237687

Summary: convmv fails to build from sources
Product: Red Hat Enterprise Linux 5 Reporter: giuseppe bonacci <giuseppe.bonacci>
Component: convmvAssignee: Nils Philippsen <nphilipp>
Status: CLOSED RAWHIDE QA Contact: QE Internationalization Bugs <qe-i18n-bugs>
Severity: low Docs Contact:
Priority: medium    
Version: 5.0CC: jos
Target Milestone: ---   
Target Release: ---   
Hardware: i386   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-09-27 14:56:27 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:
Attachments:
Description Flags
proposed patch for (testsuite.tar)suite/dotests.sh none

Description giuseppe bonacci 2007-04-24 19:27:35 UTC 
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.3) Gecko/20070310 Iceweasel/2.0.0.3 (Debian-2.0.0.3-1)

Description of problem:
when rpmbuild-ing the source rpm, the test suite fails. 

the reason is that the output from a "find" is compared to the output taken on a reference system.  the output lines are fine, except that usually the ordering differs, and that's ok.  before comparing, the test script should sort both the reference output and the observerd results, because find does not guarantee a fixed order. 

Version-Release number of selected component (if applicable):
convmv-1.10-1.el5.src.rpm

How reproducible:
Always


Steps to Reproduce:
1. rpmbuild --rebuild convmv-1.10-1.el5.src.rpm

Actual Results:


Expected Results:


Additional info:
Comment 1 giuseppe bonacci 2007-04-26 13:36:58 UTC 
Created attachment 153510 [details]
proposed patch for (testsuite.tar)suite/dotests.sh
Comment 3 Jens Petersen 2007-08-23 02:09:42 UTC 
I think "LANG=C make test" should fix this.

Also it should be moved to %check.
Comment 4 giuseppe bonacci 2007-08-23 10:16:47 UTC 
Jens, 
I tried to prepend LANG=C.  "make test" was already in '%check'. 

however, it did not work.  (BTW, I already build with LANG=C).

as far as I can tell, the locale does not matter.  the problem is due to the
assumption in dotests.sh that the list of filenames output by find is ordered 
exactly like in testsuite.tar.  generally, this is not the case: the order from 
find is unspecified.

is there any reason against the patch I submitted?  does it break something else?
Comment 5 Nils Philippsen 2007-09-27 14:56:27 UTC 
I've applied your patch and moved "make test" to %check in the current Rawhide
version. I don't think we'd get the buy-in needed to warrant pushing a new
package version through for RHEL5 though.