[Bug gold/12598] New: Two of gold testsuite scripts require bash

[binutils at marino dot st]

http://sourceware.org/bugzilla/show_bug.cgi?id=12598

           Summary: Two of gold testsuite scripts require bash
           Product: binutils
           Version: 2.22 (HEAD)
            Status: NEW
          Severity: normal
          Priority: P2
         Component: gold
        AssignedTo: address@hidden
        ReportedBy: address@hidden


Created attachment 5320
  --> http://sourceware.org/bugzilla/attachment.cgi?id=5320
Suggested method to remove bash-specific notation from gold testsuite scripts

This was originally reported to the binutils mail list: 
http://sourceware.org/ml/binutils/2011-03/msg00366.html
========================================================

Most of the gold testsuite scripts start with "#!/bin/sh" and run fine on my
platform.

The following test scripts don't specify a shell unlike the rest:
* icf_safe_so_test.sh
* icf_safe_test.sh
  icf_preemptible_functions_test.sh
  icf_sht_rel_addend_test.sh


Two of those scripts (asterisk) fail on my platform because they use notation
specific to bash.  I believe the intent is that all testsuite scripts only
require the Bourne shell and therefore these two scripts have a bug.  I have
attached the patch I used to fix the two broken scripts.  Hopefully somebody
agrees and incorporates the patches or provides and alternative solution.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.