bugGNU arch -- a revision control system - Bugs: bug #5788, make test uses the wrong shell

 
 

bug #5788: make test uses the wrong shell

Submitted by:  Ben Burns <benburns>
Submitted on:  Tue 07 Oct 2003 06:36:23 PM UTC  
 
Category: build infrastructureSeverity: 3 - Normal
Item Group: bugStatus: None
Privacy: PublicOpen/Closed: Open
Release: 
Fixed Release: 
Merge Request?: yes -- diffs included
Your Archive Name: 
Your Archive Location: 
Assigned to: None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 03 Feb 2004 07:37:49 PM UTC, comment #2:

This is fixed, it can be closed. thanks.

Ben Burns <benburns>
Sat 08 Nov 2003 10:28:51 PM UTC, comment #1:

Actually you need $(SHELL). I've applied a modified patch - it's in rbtcollins@hotmail.com--barch/package-framework--integration--1.0--patch-1

Robert Collins <robertc>
Tue 07 Oct 2003 06:36:23 PM UTC, original submission:

The following patch makes the tests run using the configured shell.

--- orig/build-tools/Makefiles/unit-tests.mk
+++ mod/build-tools/Makefiles/unit-tests.mk
@@ -125,7 +125,7 @@
tests-timestamp: $(test-progs) $(test-scripts)
set -e ; \
for s in $(test-scripts) ; do \
- sh $$s; \
+ $SHELL $$s; \
done
touch tests-timestamp

Ben Burns <benburns>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

No Changes Have Been Made to This Item

Back to the top


Powered by Savane 3.1-cleanup1