This is sources Bugzilla
Bugzilla Version 2.17.5
Bugzilla Bug 9901
  Testsuite assumes SHELL is set and fails if not Last modified: 2009-03-14 20:17:28
Bug List: First Last (This bug is not in your list)   Show list      Query page      Enter new bug
Bug#: 9901   Hardware:   Reporter: Loïc Minier <lool+redhat@via.ecp.fr>
Host: Target: Build:
Product:     Add CC:
Component:   Version:   CC:
Remove selected CCs
Status: RESOLVED   Priority:  
Resolution: WONTFIX   Severity:  
Assigned To: Ulrich Drepper <drepper@redhat.com>   Target Milestone:  
Flags: Requestee:
  backport ()
  examined ()
  testsuite ()
Summary:
Keywords:

Attachment Description Type Created Actions
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 9901 depends on: Show dependency tree
Show dependency graph
Bug 9901 blocks:

Additional Comments:


Leave as RESOLVED WONTFIX
Reopen bug
Mark bug as VERIFIED

View Bug Activity   |   Format For Printing


Description:   Last confirmed: 0000-00-00 00:00 Opened: 2009-02-25 16:57
Hi,

env -i make check fails with:
Testing ASCIItst-table.sh:38: command not found: tst-table-charmap.sh

while make check works.

This is due to the assumption SHELL is set:
    <original env>: SHELL unset
        $(MAKE) check: SHELL defined by make
            $(SHELL) tst-tables.sh: SHELL is a regular var, not env
                $(SHELL) tst-table.sh: SHELL unset (new shell)
                    $(SHELL) tst-table-charmap.sh: fails

This patch against CVS fixes it:
Index: iconvdata/tst-tables.sh
===================================================================
RCS file: /cvs/glibc/libc/iconvdata/tst-tables.sh,v
retrieving revision 1.34
diff -u -r1.34 tst-tables.sh
--- iconvdata/tst-tables.sh     12 Dec 2007 07:58:54 -0000      1.34
+++ iconvdata/tst-tables.sh     25 Feb 2009 16:57:03 -0000
@@ -25,6 +25,9 @@
 common_objpfx=$1
 objpfx=$2
 
+if test "$SHELL" = ""; then SHELL=/bin/sh; fi
+export SHELL
+
 status=0
 
 cat <<EOF |


BTW, This is with /bin/sh == bash, but my shell in passwd (getent passwd `id
-u`) is set to zsh which results in bash -c 'echo $SHELL' => /bin/zsh which
might be one reason it fails.

Bye

------- Additional Comment #1 From Ulrich Drepper 2009-03-14 20:17 -------
There are countless assumptions that are made while running the test suite.  I
don't start changing the test suite so that people with strange/wrong
environments can run it.  Fix up your environment.

Bug List: First Last (This bug is not in your list)   Show list      Query page      Enter new bug
Actions: New | Query | bug # | Reports | Requests   New Account | Log In