Re: [PATCH] Add new program: psub
On Sun, Apr 27, 2008 at 9:02 PM, Bo Borgerson <[EMAIL PROTECTED]> wrote: > Hi, > > This program uses the temporary fifo management system that I built > for zargs to provide generic process substitution for arguments to a > sub-command. > > This program has some advantages over the process substitution built > into some shells (bash, zsh, ksh, ???): > > 1. It doesn't rely on having a shell that supports built-in process > substitution. > 2. By using descriptively named temporary fifos it allows programs > that include filenames in output or diagnostic messages to provide > more useful information than with '/dev/fd/*' inputs. > 3. It supports `--files0-from=F' style argument passing, as well. > Hi, I'd like to point out that this is also true of `xargs' invocation. $ echo "http://lists.gnu.org/mailman/listinfo/bug-coreutils
FYI, 11 mostly-test-related patches
[I hesitated to post so many small test-only patches all together,
but it's better than not posting at all. Besides, maybe posting
at all will evoke some review feedback. So here goes... ]
>From 6bbbd65d895505c506105ea12e44922433e63574 Mon Sep 17 00:00:00 2001
From: Jim Meyering <[EMAIL PROTECTED]>
Date: Thu, 24 Apr 2008 12:31:13 +0200
Subject: [PATCH] tests: do define built_programs
* tests/check.mk (built_programs): Define here, where it's used.
This definition was in now-removed tests/misc/Makefile.am.
Signed-off-by: Jim Meyering <[EMAIL PROTECTED]>
---
tests/check.mk |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/tests/check.mk b/tests/check.mk
index cb4373a..978313b 100644
--- a/tests/check.mk
+++ b/tests/check.mk
@@ -36,6 +36,9 @@ vc_exe_in_TESTS: Makefile
check: vc_exe_in_TESTS
.PHONY: vc_exe_in_TESTS
+built_programs = \
+ (cd $(top_builddir)/src && MAKEFLAGS= $(MAKE) -s built_programs.list)
+
# Append this, because automake does the same.
TESTS_ENVIRONMENT =\
abs_top_builddir='$(abs_top_builddir)' \
--
1.5.5.1.68.gbdcd8
>From cca6a8d7e525edb232160046d074fff16a76bbb5 Mon Sep 17 00:00:00 2001
From: Jim Meyering <[EMAIL PROTECTED]>
Date: Sun, 27 Apr 2008 21:31:08 +0200
Subject: [PATCH] tweak HACKING advice
Signed-off-by: Jim Meyering <[EMAIL PROTECTED]>
---
HACKING |9 +++--
1 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/HACKING b/HACKING
index d616443..b40ff00 100644
--- a/HACKING
+++ b/HACKING
@@ -26,11 +26,16 @@ That downloads the entire repository, including revision
control history
dating back to 1991. The repository (the part you download, and which
resides in coreutils/.git) currently weighs in at about 30MB. So you
don't want to download it more often than necessary. Once downloaded,
-you can get incremental updates by running this command from inside
-your new coreutils/ directory:
+you can get incremental updates by running one of these commands from
+inside your new coreutils/ directory:
+If you have made *no* changes:
git pull
+If you *have* made changes and committed them to "master", do this:
+git fetch
+git rebase origin
+
*Before* you commit changes
===
--
1.5.5.1.68.gbdcd8
>From 8dc387dba567da10e2e90107fd044baaef580145 Mon Sep 17 00:00:00 2001
From: Jim Meyering <[EMAIL PROTECTED]>
Date: Thu, 24 Apr 2008 12:43:20 +0200
Subject: [PATCH] tests: cp/perm (usually not run) was failing on systems with
SELinux
* tests/cp/perm: Use stat to get the permission string, not ls.
This test was run only when RUN_VERY_EXPENSIVE_TESTS=yes was set
in the environment. It would fail on SELinux-enable systems
because ls-generated permission strings would not match, e.g.,
"test _-rw-r--r--+ = _-rw-r--r--" would fail.
Signed-off-by: Jim Meyering <[EMAIL PROTECTED]>
---
tests/cp/perm |9 +++--
1 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/tests/cp/perm b/tests/cp/perm
index b24221f..b482c17 100755
--- a/tests/cp/perm
+++ b/tests/cp/perm
@@ -41,9 +41,7 @@ for u in 31 37 2; do
for o_perm in r w x rw wx xr rwx; do
touch src || exit 1
chmod u=r,g=rx,o= src || exit 1
- set _ `ls -l src`
- shift
- expected_perms=$1
+ expected_perms=$(stat --format=%A src)
rm -f dest
test $existing_dest = yes && {
touch dest || exit 1
@@ -52,8 +50,7 @@ for u in 31 37 2; do
$cmd $force src dest || exit 1
test "$cmd" = mv && test -f src && exit 1
test "$cmd" = cp && { test -f src || exit 1; }
- set _ `ls -l dest`
- shift
+ actual_perms=$(stat --format=%A dest)
case "$cmd:$force:$existing_dest" in
cp:*:yes)
@@ -68,7 +65,7 @@ for u in 31 37 2; do
expected_perms=`echo $expected_perms|sed
's/..\(..\).$/--\1-/'`
;;
esac
- test _$1 = _$expected_perms || exit 1
+ test _$actual_perms = _$expected_perms || exit 1
# Perform only one iteration when there's no existing destination.
test $existing_dest = no && break 3
done
--
1.5.5.1.68.gbdcd8
>From 427f179542e9f7228f77affdd5a4c50a573b4894 Mon Sep 17 00:00:00 2001
From: Jim Meyering <[EMAIL PROTECTED]>
Date: Thu, 24 Apr 2008 12:50:24 +0200
Subject: [PATCH] tests: move another file (expensive) into test-lib.sh
* tests/expensive: Remove file. Move contents into ...
* tests/test-lib.sh (expensive_): ...here. New function.
* tests/du/fd-leak: Update caller to use the new function.
* tests/mv/leak-fd: Likewise.
* tests/rm/hash: Likewise.
* tests/tail-2/big-4gb: Likewise.
* tests/Makefile.am (EXTRA_DIST): Remove its name.
Signed-off-by: Jim Meyering <[EMAIL PROTECTED]>
---
tests/Makefile.am|1 -
tests/du/fd-leak |5 +++--
tests/expensive | 28
tests/mv/leak-f
Re: FYI, 11 mostly-test-related patches
Hi Jim,
Regarding rev. 94a2bd5b, `tests: improve perl-based tempdir handling',
I wonder if it might be worthwhile to add a PID check in the cleanup
handler for race avoidance.
I think File::Temp does this internally as well, but it looks like
chmod_tree will just warn about the failed chdir and procede to
recursively chmod whatever directory it was in at the time if $dir is
yanked out from under it.
I've included a patch that sets up the signal handler as a closure
around a copy of the PID at the time of installation and only cleans
up in the original process.
Thanks,
Bo
On Mon, Apr 28, 2008 at 9:48 AM, Jim Meyering <[EMAIL PROTECTED]> wrote:
> [I hesitated to post so many small test-only patches all together,
> but it's better than not posting at all. Besides, maybe posting
> at all will evoke some review feedback. So here goes... ]
>
> From 6bbbd65d895505c506105ea12e44922433e63574 Mon Sep 17 00:00:00 2001
> From: Jim Meyering <[EMAIL PROTECTED]>
> Date: Thu, 24 Apr 2008 12:31:13 +0200
> Subject: [PATCH] tests: do define built_programs
>
> * tests/check.mk (built_programs): Define here, where it's used.
> This definition was in now-removed tests/misc/Makefile.am.
>
> Signed-off-by: Jim Meyering <[EMAIL PROTECTED]>
> ---
> tests/check.mk |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/tests/check.mk b/tests/check.mk
> index cb4373a..978313b 100644
> --- a/tests/check.mk
> +++ b/tests/check.mk
> @@ -36,6 +36,9 @@ vc_exe_in_TESTS: Makefile
> check: vc_exe_in_TESTS
> .PHONY: vc_exe_in_TESTS
>
> +built_programs = \
> + (cd $(top_builddir)/src && MAKEFLAGS= $(MAKE) -s built_programs.list)
> +
> # Append this, because automake does the same.
> TESTS_ENVIRONMENT =\
>abs_top_builddir='$(abs_top_builddir)' \
> --
> 1.5.5.1.68.gbdcd8
>
>
> From cca6a8d7e525edb232160046d074fff16a76bbb5 Mon Sep 17 00:00:00 2001
> From: Jim Meyering <[EMAIL PROTECTED]>
> Date: Sun, 27 Apr 2008 21:31:08 +0200
> Subject: [PATCH] tweak HACKING advice
>
>
> Signed-off-by: Jim Meyering <[EMAIL PROTECTED]>
> ---
> HACKING |9 +++--
> 1 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/HACKING b/HACKING
> index d616443..b40ff00 100644
> --- a/HACKING
> +++ b/HACKING
> @@ -26,11 +26,16 @@ That downloads the entire repository, including revision
> control history
> dating back to 1991. The repository (the part you download, and which
> resides in coreutils/.git) currently weighs in at about 30MB. So you
> don't want to download it more often than necessary. Once downloaded,
> -you can get incremental updates by running this command from inside
> -your new coreutils/ directory:
> +you can get incremental updates by running one of these commands from
> +inside your new coreutils/ directory:
>
> +If you have made *no* changes:
> git pull
>
> +If you *have* made changes and committed them to "master", do this:
> +git fetch
> +git rebase origin
> +
>
> *Before* you commit changes
> ===
> --
> 1.5.5.1.68.gbdcd8
>
>
> From 8dc387dba567da10e2e90107fd044baaef580145 Mon Sep 17 00:00:00 2001
> From: Jim Meyering <[EMAIL PROTECTED]>
> Date: Thu, 24 Apr 2008 12:43:20 +0200
> Subject: [PATCH] tests: cp/perm (usually not run) was failing on systems
> with SELinux
>
> * tests/cp/perm: Use stat to get the permission string, not ls.
> This test was run only when RUN_VERY_EXPENSIVE_TESTS=yes was set
> in the environment. It would fail on SELinux-enable systems
> because ls-generated permission strings would not match, e.g.,
> "test _-rw-r--r--+ = _-rw-r--r--" would fail.
>
> Signed-off-by: Jim Meyering <[EMAIL PROTECTED]>
> ---
> tests/cp/perm |9 +++--
> 1 files changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/tests/cp/perm b/tests/cp/perm
> index b24221f..b482c17 100755
> --- a/tests/cp/perm
> +++ b/tests/cp/perm
> @@ -41,9 +41,7 @@ for u in 31 37 2; do
> for o_perm in r w x rw wx xr rwx; do
> touch src || exit 1
> chmod u=r,g=rx,o= src || exit 1
> - set _ `ls -l src`
> - shift
> - expected_perms=$1
> + expected_perms=$(stat --format=%A src)
> rm -f dest
> test $existing_dest = yes && {
> touch dest || exit 1
> @@ -52,8 +50,7 @@ for u in 31 37 2; do
> $cmd $force src dest || exit 1
> test "$cmd" = mv && test -f src && exit 1
> test "$cmd" = cp && { test -f src || exit 1; }
> - set _ `ls -l dest`
> - shift
> + actual_perms=$(stat --format=%A dest)
>
> case "$cmd:$force:$existing_dest" in
> cp:*:yes)
> @@ -68,7 +65,7 @@ for u in 31 37 2; do
> expected_perms=`echo $expected_perms|sed
> 's/..\(..\).$/--\1-/'`
> ;;
> esac
> - test _$1 = _$expected_perms ||
Re: FYI, 11 mostly-test-related patches
"Bo Borgerson" <[EMAIL PROTECTED]> wrote: > Regarding rev. 94a2bd5b, `tests: improve perl-based tempdir handling', > I wonder if it might be worthwhile to add a PID check in the cleanup > handler for race avoidance. > > I think File::Temp does this internally as well, but it looks like > chmod_tree will just warn about the failed chdir and procede to > recursively chmod whatever directory it was in at the time if $dir is > yanked out from under it. Yes. Good catch. It should obviously skip the find in that case. Want to write the patch? > I've included a patch that sets up the signal handler as a closure > around a copy of the PID at the time of installation and only cleans > up in the original process. Protecting against clean-up races in multi-threaded tests is definitely worthwhile. Thanks! Luckily for my ego, I think none of the tests in coreutils are susceptible. I prefer the closure, too. Applied. > From dd8e78633f60a4a266b870326ac87d9844dab02b Mon Sep 17 00:00:00 2001 > From: Bo Borgerson <[EMAIL PROTECTED]> > Date: Mon, 28 Apr 2008 10:30:22 -0400 > Subject: [PATCH] Only cleanup test dirs from the process that created them. > > * tests/CuTmpdir.pm (import): Use closure around current PID to avoid cleanup > races. ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils
Re: FYI, 11 mostly-test-related patches
Jim Meyering wrote:
> "Bo Borgerson" <[EMAIL PROTECTED]> wrote:
>> I think File::Temp does this internally as well, but it looks like
>> chmod_tree will just warn about the failed chdir and procede to
>> recursively chmod whatever directory it was in at the time if $dir is
>> yanked out from under it.
>
> Yes. Good catch. It should obviously skip the find in that case.
> Want to write the patch?
Sure, I think this should do it.
Thanks,
Bo
>From 769e662c3643c0b3dc21e96dec2e2f1cd481fb70 Mon Sep 17 00:00:00 2001
From: Bo Borgerson <[EMAIL PROTECTED]>
Date: Mon, 28 Apr 2008 13:11:26 -0400
Subject: [PATCH] tests: don't chmod after a failed chdir in cleanup
* tests/CuTmpdir.pm (chmod_tree): Don't chmod if chdir failed.
Signed-off-by: Bo Borgerson <[EMAIL PROTECTED]>
---
tests/CuTmpdir.pm | 15 ++-
1 files changed, 10 insertions(+), 5 deletions(-)
diff --git a/tests/CuTmpdir.pm b/tests/CuTmpdir.pm
index 84312a0..e21306a 100644
--- a/tests/CuTmpdir.pm
+++ b/tests/CuTmpdir.pm
@@ -45,11 +45,16 @@ sub chmod_1
sub chmod_tree
{
- chdir $dir
-or warn "$ME: failed to chdir to $dir: $!\n";
- # Perform the equivalent of find . -type d -print0|xargs -0 chmod -R 700.
- my $options = {untaint => 1, wanted => \&chmod_1};
- find ($options, '.');
+ if (chdir $dir)
+{
+ # Perform the equivalent of find . -type d -print0|xargs -0 chmod -R 700.
+ my $options = {untaint => 1, wanted => \&chmod_1};
+ find ($options, '.');
+}
+ else
+{
+ warn "$ME: failed to chdir to $dir: $!\n";
+}
}
sub import {
--
1.5.4.3
___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils
Re: FYI, 11 mostly-test-related patches
Bo Borgerson <[EMAIL PROTECTED]> wrote: > Subject: [PATCH] tests: don't chmod after a failed chdir in cleanup > * tests/CuTmpdir.pm (chmod_tree): Don't chmod if chdir failed. Thanks! Applied and pushed. ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils
Re: FYI, 11 mostly-test-related patches
Jim Meyering meyering.net> writes: > > [I hesitated to post so many small test-only patches all together, > but it's better than not posting at all. Besides, maybe posting > at all will evoke some review feedback. So here goes... ] I'm now seeing this when running ./bootstrap: ./bootstrap: m4/xsize.m4 overrides ._bootmp2/m4/xsize.m4 Undefined subroutine &Test::test_vector called at tests/mk-script line 44. ./bootstrap: aclocal --force -I m4 ... and am not yet sure if it is from one of these patches or from an external change of upgrading from perl 5.8.8 to 5.10.0. -- Eric Blake ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils
Re: FYI, 11 mostly-test-related patches
Eric Blake wrote:
> I'm now seeing this when running ./bootstrap:
>
> ./bootstrap: m4/xsize.m4 overrides ._bootmp2/m4/xsize.m4
> Undefined subroutine &Test::test_vector called at tests/mk-script line 44.
> ./bootstrap: aclocal --force -I m4 ...
>
> and am not yet sure if it is from one of these patches or from an external
> change of upgrading from perl 5.8.8 to 5.10.0.
>
Hmm... I'm not seeing that (still on 5.8.8 here).
But I _am_ seeing some warnings about trying to mess around in
`tests/wc', which doesn't seem to exist anymore.
./bootstrap: cp -f ._bootmp2/po/remove-potcdate.sin po/remove-potcdate.sin
./bootstrap: 537: cannot create tests/wc/Makefile.amt: Directory nonexistent
./bootstrap: 537: cannot create tests/wc/Makefile.amt: Directory nonexistent
./bootstrap: 537: cannot create tests/wc/Makefile.amt: Directory nonexistent
./bootstrap: 537: cannot create tests/wc/Makefile.amt: Directory nonexistent
chmod: cannot access `tests/wc/Makefile.amt': No such file or directory
mv: cannot stat `tests/wc/Makefile.amt': No such file or directory
./bootstrap: aclocal --force -I m4 ...
I've attached a patch that addresses this.
Thanks,
Bo
>From 07cf6a226f75aa9e91060d657fdfebff6e74a618 Mon Sep 17 00:00:00 2001
From: Bo Borgerson <[EMAIL PROTECTED]>
Date: Mon, 28 Apr 2008 14:58:51 -0400
Subject: [PATCH] Remove references to tests/wc from bootstrap
* bootstrap: Don't try to initialize anything in tests/wc.
Signed-off-by: Bo Borgerson <[EMAIL PROTECTED]>
---
bootstrap |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/bootstrap b/bootstrap
index 1274af2..c4b9b62 100755
--- a/bootstrap
+++ b/bootstrap
@@ -535,7 +535,7 @@ fi
mam_template=tests/Makefile.am.in
if test -f $mam_template; then
PERL=perl
- for tool in cut head join pr sort tac tail test tr uniq wc; do
+ for tool in cut head join pr sort tac tail test tr uniq; do
m=tests/$tool/Makefile.am
t=${m}t
rm -f $m $t
--
1.5.4.3
___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils
Re: FYI, 11 mostly-test-related patches
Bo Borgerson <[EMAIL PROTECTED]> wrote:
> Eric Blake wrote:
>> I'm now seeing this when running ./bootstrap:
>>
>> ./bootstrap: m4/xsize.m4 overrides ._bootmp2/m4/xsize.m4
>> Undefined subroutine &Test::test_vector called at tests/mk-script line 44.
>> ./bootstrap: aclocal --force -I m4 ...
>>
>> and am not yet sure if it is from one of these patches or from an external
>> change of upgrading from perl 5.8.8 to 5.10.0.
>>
>
> Hmm... I'm not seeing that (still on 5.8.8 here).
>
> But I _am_ seeing some warnings about trying to mess around in
> `tests/wc', which doesn't seem to exist anymore.
>
>
> ./bootstrap: cp -f ._bootmp2/po/remove-potcdate.sin po/remove-potcdate.sin
> ./bootstrap: 537: cannot create tests/wc/Makefile.amt: Directory nonexistent
> ./bootstrap: 537: cannot create tests/wc/Makefile.amt: Directory nonexistent
> ./bootstrap: 537: cannot create tests/wc/Makefile.amt: Directory nonexistent
> ./bootstrap: 537: cannot create tests/wc/Makefile.amt: Directory nonexistent
> chmod: cannot access `tests/wc/Makefile.amt': No such file or directory
> mv: cannot stat `tests/wc/Makefile.amt': No such file or directory
> ./bootstrap: aclocal --force -I m4 ...
>
> I've attached a patch that addresses this.
Thanks to both of you.
Bo, I've applied your patch.
FYI, here's one more, that will eliminate the risk of my making
that mistake again ;-)
>From 28f1b61e594eeb91658e8db392d8911a4485dc7a Mon Sep 17 00:00:00 2001
From: Jim Meyering <[EMAIL PROTECTED]>
Date: Mon, 28 Apr 2008 22:59:24 +0200
Subject: [PATCH] tests: don't hard-code coreutils list of tests/ SUBDIRS in
bootstrap
* bootstrap: Extract the list from tests/Makefile.am, and die
when it is empty -- it will be, eventually.
---
bootstrap |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/bootstrap b/bootstrap
index c4b9b62..9e35444 100755
--- a/bootstrap
+++ b/bootstrap
@@ -535,7 +535,9 @@ fi
mam_template=tests/Makefile.am.in
if test -f $mam_template; then
PERL=perl
- for tool in cut head join pr sort tac tail test tr uniq; do
+ tool_subdirs=`$PERL -ne '/^SUBDIRS = (.*)/ and print $1' tests/Makefile.am`
+ test "$tool_subdirs" = '' && { echo "$0: no more SUBDIRS: FIXME"1>&2; exit
1;}
+ for tool in $tool_subdirs; do
m=tests/$tool/Makefile.am
t=${m}t
rm -f $m $t
--
1.5.5.1.68.gbdcd8
___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils
Re: FYI, 11 mostly-test-related patches
Eric Blake <[EMAIL PROTECTED]> wrote: > Jim Meyering meyering.net> writes: >> [I hesitated to post so many small test-only patches all together, >> but it's better than not posting at all. Besides, maybe posting >> at all will evoke some review feedback. So here goes... ] > > I'm now seeing this when running ./bootstrap: > > ./bootstrap: m4/xsize.m4 overrides ._bootmp2/m4/xsize.m4 > Undefined subroutine &Test::test_vector called at tests/mk-script line 44. > ./bootstrap: aclocal --force -I m4 ... > > and am not yet sure if it is from one of these patches or from an external > change of upgrading from perl 5.8.8 to 5.10.0. Hi Eric, Thanks for reporting that. Would you please pull the latest to see if Bo's patch fixed it? ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils
Re: [Elbert Pol] Re: coreutils-6.11 released
Hi Jim, Curious if the log give you some usefull info? Jim Meyering wrote: Elbert Pol<[EMAIL PROTECTED]> wrote: ... Remember when you chose not to run "make -k check"? ;-) Well, now would be a good time to go back and do that, but with working tools in your PATH, of course. Hoi Jim, As you told before this fails :( I attach check.log Try again, but this time be sure to use -k, as suggested above. Then it won't stop at the first failure. ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils
Re: [Elbert Pol] Re: coreutils-6.11 released
Elbert Pol <[EMAIL PROTECTED]> wrote: > Jim Meyering wrote: >> Elbert Pol<[EMAIL PROTECTED]> wrote: >>> Hoi Jim, >>> >>> When i use the new rm.exe and use it with the ./configure i get all >>> kind of errors like : >>> >>> rm.exe: cannot remove `conftest': Is a directory >>> rm.exe: cannot remove `conftest.dir': Is a directory >>> rm.exe: FATAL: cannot enter directory `conftest': Bad file number >>> rm.exe: FATAL: cannot enter directory `conftest.dir': Bad file number >>> >>> I don't have it when i use the rm.exe from 5.94 >>> I attach the log file. >>> Mayby you know what wrong ? >> >> Not yet. >> >> Remember when you chose not to run "make -k check"? ;-) >> Well, now would be a good time to go back and do that, >> but with working tools in your PATH, of course. >> > Hoi again, > > Hope i attach now the good one :) Lots of unusual errors. No diff program? I think "diff" must be missing or non-functional. Otherwise, we'd have better output telling us about particular test failures. The number and distribution of failures make me pretty sure that the tests are failing due to a mismatch between the tests' expecting newlines in output and your executables generating CRs. For example, run these cd tests/cut && ../../src/cut.exe -d: -f1,3- 1.I | diff -u - 1.O [or if you lack "diff", do this: ] cd tests/cut && ../../src/cut.exe -d: -f1,3- 1.I | cmp - 1.O ../../src/cut.exe -d: -f1,3- 1.I | cat -A ../../src/cut.exe -d: -f1,3- 1.I | od -ac That last one gives me this: $ ../../src/cut -d: -f1,3- 1.I | od -ac 000 a : c nl a : c \n 004 Also, the test output suggests that there is a broken "rm" command in your path. ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils
ubuntu command non-recognition
hello i have just installed Ubuntu Edgy and i find that it is unable to install proprietary free software specifically made for Linux - Adobe Flash Player and RealPlayer - that is a bug in the Installer, and it means that i cannot watch my MySpace Videos, Yahoo and Google Videos and the BBC news, the latter being my daily habit (until ubuntu came along). to try to install myself myself then, i downloaded flashplayer 9 and put it in my home directory, and then in a terminal i used the following command to change directory: cd /home/graham/flashplayer/ where i had placed flashplayer 9; then i gave the command: sudo install /home/graham/flashplayer/flash-plugin-9.0.124.0-release.i386.rpm (1) this sudo install command does not work. i was directed to install help but i could not understand a word of what was written there:it did not show the command line to give to install a program (2) the install help does not help, and needs to be rewritten clearly, so that it will aid a user so i tried the following, which is a standard Linux command line, and worked for me on Mandriva: rpm -Uvh /home/graham/flashplayer/flash-plugin-9.0.124.0-release.i386.rpm (3) Ubuntu did not recognise the command "rpm" i am disappointed in the quality of this O/S which i had heard so much about. can you tell me what specific commands to use to install rpm files in Ubuntu, please? This lack of direction is spoiling my user experience, as i cannot install programs made for Linux. thank you. graham cee __ Sent from Yahoo! Mail. A Smarter Email http://uk.docs.yahoo.com/nowyoucan.html ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils
Re: ubuntu command non-recognition
Graham Cunnington wrote: > i have just installed Ubuntu Edgy and i find that You would have better luck asking your question in the Ubuntu User mailing list. This is the GNU Coreutils mailing list. The GNU Coreutils are the basic file, shell and text manipulation utilities of the GNU Operating System. You can learn more about GNU Coreutils here: http://www.gnu.org/software/coreutils/ The GNU Coreutils are part of the GNU Operating System. You can learn more about the GNU Project here: http://www.gnu.org/ But you are asking about installing Adobe and RealPlayer. > it is unable to install proprietary free software specifically made > for Linux - Adobe Flash Player and RealPlayer - The term "free software" has specific meaning for us. It doesn't mean free of charge. It means freedom as in freedom. One of the big problems with Adobe Flash Player and RealPlayer are that they are not free software. It makes working with it very difficult. You can learn more about why we distinguish free(dom) software from free(charge) software here: http://www.gnu.org/philosophy/free-sw.html > that is a bug in the Installer, The GNU Coreutils community, the people you have reached with your message to the bug-coreutils mailing list, are not associated with any of those programs Flash or RealPlayer. You have taken your motorcycle to a boat shop. > and it means that i cannot watch my MySpace Videos, Yahoo and Google > Videos and the BBC news, the latter being my daily habit (until > ubuntu came along). The Ubuntu Users mailing list would be the best place to get help. http://www.ubuntu.com/support/community/mailinglists > to try to install myself myself then, i downloaded flashplayer 9 and > put it in my home directory, and then in a terminal i used the > following command to change directory: cd /home/graham/flashplayer/ > where i had placed flashplayer 9; then i gave the command: > > sudo install /home/graham/flashplayer/flash-plugin-9.0.124.0-release.i386.rpm > > (1) this sudo install command does not work. That sudo install command is completely incorrect. It is wrong for two reasons. 1) The install command install individual files and not packages. But you are giving it a package. This is incorrect usage for the "install" program. It is the wrong program for the job. 2) Ubuntu does not use rpm packages. Ubuntu uses .deb packages. You shouldn't be trying to install a .rpm file on your Ubuntu system. > i was directed to install help but i could not understand a word of > what was written there:it did not show the command line to give to > install a program The install program is not the program you think it is. Don't use it for this purpose. (I could describe what it is for but it isn't important at this step because you shouldn't be using it.) > so i tried the following, which is a standard Linux command line, > and worked for me on Mandriva: > > rpm -Uvh /home/graham/flashplayer/flash-plugin-9.0.124.0-release.i386.rpm > > (3) Ubuntu did not recognise the command "rpm" Correct. Ubuntu does not use rpm. Ubuntu uses dpkg and other related programs which are completely different than Mandriva's rpm. > i am disappointed in the quality of this O/S which i had heard so much about. You have tried to install incompatible software on your machine. It didn't work. But because it was incompatible it could not possibly have worked. This should not disappoint you but should confirm that the universe works as expected. This is a good thing! :-) But as far as the closed source proprietary non-free software is concerned, such as Flash and RealPlayer, those will be more difficult to use because they are NON-FREE. People have made them work but the licenses on those programs make them problematic. They can't be pre-installed and cannot be ported to other systems. The blame for the problem rests solely with them. > can you tell me what specific commands to use to install rpm files > in Ubuntu, please? This lack of direction is spoiling my user > experience, as i cannot install programs made for Linux. Sorry but I cannot help you on this mailing list as that is off-topic for this community. But if you go to the Ubuntu pages I am sure that you will be able to get the appropriate help there. http://www.ubuntu.com/support/communitysupport Bob ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils
