GNU bug report logs -
#25501
sed-4.3: testsuite/runtest does not check for a program extension
Previous Next
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 25501 in the body.
You can then email your comments to 25501 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
bug-sed <at> gnu.org
:
bug#25501
; Package
sed
.
(Sat, 21 Jan 2017 14:42:01 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Juan Manuel Guerrero <juan.guerrero <at> gmx.de>
:
New bug report received and forwarded. Copy sent to
bug-sed <at> gnu.org
.
(Sat, 21 Jan 2017 14:42:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
OFYI, the testsuite/runtest shell script makes the assumption that programs
like get-mb-cur-max and test-mbrtowc have no extension but this is not true
for non-posix systems like Windows (aka MinGW) or MSDOS (aka DJGPP). Thus
the check fails when runtest is invoked. In some way the possibility for
an extension should be provided. The patch below is not intended as fix,
it shall only demonstrate the issue. It is clear that this problem cannot
be reproduced on a posix system like linux.
This means that the test fails to find programs if they have an extension
like ".exe" as used to be in DOS/Windows.
Regards,
Juan M. Guerrero
diff -aprNU5 sed-4.3.orig/testsuite/runtest sed-4.3/testsuite/runtest
--- sed-4.3.orig/testsuite/runtest 2016-07-25 03:03:56 +0000
+++ sed-4.3/testsuite/runtest 2017-01-13 17:49:42 +0000
@@ -9,11 +9,11 @@
: ${SED="./sed/sed"}
makefile="${abs_top_srcdir}/testsuite/Makefile.tests"
# Strip the directory './testsuite/' from the test name.
-test=`echo "$@"| sed 's,.*/,,'`
+test=`echo "$@"| sed 's,.*/,,;s,\.exe$,,'`
# Skip the utility programs, they are not standalone tests
test "$test" = get-mb-cur-max \
|| test "$test" = test-mbrtowc \
&& exit 77
Added tag(s) fixed.
Request was from
Assaf Gordon <assafgordon <at> gmail.com>
to
control <at> debbugs.gnu.org
.
(Tue, 09 Oct 2018 12:54:01 GMT)
Full text and
rfc822 format available.
Reply sent
to
Assaf Gordon <assafgordon <at> gmail.com>
:
You have taken responsibility.
(Tue, 09 Oct 2018 12:54:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Juan Manuel Guerrero <juan.guerrero <at> gmx.de>
:
bug acknowledged by developer.
(Tue, 09 Oct 2018 12:54:02 GMT)
Full text and
rfc822 format available.
Message #12 received at 25501-done <at> debbugs.gnu.org (full text, mbox):
tags 25501 fixed
stop
Hello,
On 21/01/17 07:42 AM, Juan Manuel Guerrero wrote:
> OFYI, the testsuite/runtest shell script makes the assumption that programs
> like get-mb-cur-max and test-mbrtowc have no extension but this is not true
> for non-posix systems like Windows (aka MinGW) or MSDOS (aka DJGPP).
Thank you for the report.
In the time since your bug report, sed's testsuite has
been greatly improved, and the problematic file (runtest)
has been completely removed:
https://git.savannah.gnu.org/cgit/sed.git/commit/?id=7daa26d068e5e84eed1212e203c2ce79c074604c
I've just tested the latest git revision on cygwin
and the auxiliary test files (get-mb-cur-max.exe and test-mbrtowc.exe)
are executed correctly.
As such I'm closing this bug report.
regards,
- assaf
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Wed, 07 Nov 2018 12:24:04 GMT)
Full text and
rfc822 format available.
This bug report was last modified 5 years and 170 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.