Hi,
as reported in https://bugzilla.redhat.com/show_bug.cgi?id=1280333 ,
Oracle ADVM/ACFS is not listed among the remote filesystems and in
stat.c . Can you please add S_MAGIC_ACFS (0x61636673) to stat.c as a
remote filesystem? Not sending as a patch, as this is quite simple
change.
Thanks in
Bob Proulx píše v Pá 28. 08. 2015 v 13:10 -0600:
> tag 21369 + notabug moreinfo
> thanks
> billy_k_woo...@homedepot.com wrote:
> > /sbin/runuser issue on RHEL 6.7 when executing `whoami` as tomcat.
...
> For another you are reporting a problem about "Coreutils RHEL 6.7
> runuser" but runuser is
additionally this hack looks silly to me)
Greetings,
Ondřej Vašík
signature.asc
Description: Toto je digitálně podepsaná část zprávy
,
Ondřej Vašík
From 67bc8118e98f99f59d1f601d06c121f3ed2c4f58 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= ova...@redhat.com
Date: Mon, 1 Feb 2010 16:29:27 +0100
Subject: [PATCH] tests: cp-a-selinux :skip the test if no loopdevices available
* tests/cp/cp
Hi,
Eric Blake wrote:
According to David Venus on 12/23/2009 3:35 PM:
2) Be able to say things like fist day of year (20090101), week
(20091220), month (20091201), etc.
3) Be able to say things like last day of year (20091231), week
(20091226), month (20091231), etc.
Improvements to
Hi,
Ondřej Vašík wrote:
Hi,
I'm trying to package coreutils-8.1 for Fedora Rawhide, but few tests
are failing
( http://koji.fedoraproject.org/koji/getfile?taskID=1819634name=build.log ).
I removed all Fedora-only patches to prevent patch-related issues. Koji
build system uses mock build
Hi Jim,
Jim Meyering wrote:
Ondřej Vašík wrote:
I'm trying to package coreutils-8.1 for Fedora Rawhide, but few tests
are failing
( http://koji.fedoraproject.org/koji/getfile?taskID=1819634name=build.log
).
I removed all Fedora-only patches to prevent patch-related issues. Koji
,
Ondřej Vašík
signature.asc
Description: Toto je digitálně podepsaná část zprávy
was changed to not hardcode runcon program name.
Greetings,
Ondřej Vašík
From 3dbd792b3e9379a4795d1becdc4252de236429bc Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= ova...@redhat.com
Date: Mon, 5 Oct 2009 09:20:48 +0200
Subject: [PATCH] chcon: no longer abort on SELinux
Paul Eggert wrote:
Ondřej Vašík ova...@redhat.com writes:
as reported in https://bugzilla.redhat.com/show_bug.cgi?id=525134 by
Daniel Qarras, ls -l shows iso long format for en_* locales.
I just now read that Bugzilla report, and the diagnosis and the
patch do not seem correct
Jim Meyering wrote:
Here's an incomplete patch.
It needs a test and a NEWS entry.
Ondřej, can you adjust your test to work (or skip)
if there is no en* locale?
Maybe something like that (attachment)?
It's checking for existence of locale binary and en_US locale and
performing the test only if
(imho better than long iso style).
Greetings,
Ondřej Vašík
From e82f581055af6eadcf3e99ff7aa3f5f3479c7c22 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= ova...@redhat.com
Date: Fri, 25 Sep 2009 15:20:47 +0200
Subject: [PATCH] ls: do not show long iso time format
Pádraig Brady wrote:
Yes I agree that the change is required.
I've tweaked it so that the geteuid() syscall is only called
if readonly files. Also I removed the error message on chmod failure
as the user will still get an error message _if_ the copy_xattr fails.
Also I ran it through indent
Jim Meyering wrote:
Ondřej Vašík wrote:
Pádraig Brady wrote:
Yes I agree that the change is required.
I've tweaked it so that the geteuid() syscall is only called
if readonly files. Also I removed the error message on chmod failure
as the user will still get an error message _if_
it is not a bug in libattr (just strange requirement by
kernel), I modified the comment about workaround - as the culprit is
probably in kernel.
Greetings,
Ondřej Vašík
From 500acf63e78cc6bdb4fdd8b84c1f5e2305b96fb1 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk
.
As it breaks translation strings, I guess it would be better to postpone
commit after 7.6 (if accepted).
Greetings,
Ondřej Vašík
From f5f6875b2d8828218105b3601c80ebfdc7eeefb4 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= ova...@redhat.com
Date: Wed, 9 Sep 2009 10
Eric Blake wrote:
According to Ondřej Vašík on 9/9/2009 2:19 AM:
Hello,
as requested via https://bugzilla.redhat.com/show_bug.cgi?id=511188 , it
would be nice to clarify default blocksize used by du in manpage/--help
output. Although I closed that bugzilla as notabug, further
called emit_blocksize_note with constant string (uppercased).
Additionally I factored out duplicities about SIZE's - to same function
in system.h
Greetings,
Ondřej Vašík
From abec47c101b23d138bc185e590e3be969fb666f2 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= ova
Hi,
Jim Meyering wrote:
Pádraig Brady wrote:
Ondřej Vašík wrote:
As reported in
http://lists.gnu.org/archive/html/bug-coreutils/2009-08/msg00342.html by
Ernest N. Mamikonyan, cp/mv fails to preserve extended attributes for
read-only source files.
Following patch fixes the issue
Pádraig Brady wrote:
Ondřej Vašík wrote:
Ah, I knew I forgot to do something :). Thanks for spotting this.
Restoring to dest_mode ~omitted_permissions done in attached patch,
dropped redirections from the test as well. Additionally - I modified
the copy.c patch a bit - failure of mode
Pádraig Brady wrote:
Ondřej Vašík wrote:
Pádraig Brady wrote:
To minimize side affects perhaps we should only do the chmod(600)
if (geteuid () != 0 !access (src_name, W_OK)) ?
Good idea, it would reduce possibility of security leak, playing with
access rights is always a bit
at the moment.
Test included...
Greetings,
Ondřej Vašík
From cae691907fe50e2ab05198a7c647fe4140e3669e Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= ova...@redhat.com
Date: Thu, 3 Sep 2009 16:10:21 +0200
Subject: [PATCH] cp,mv: do preserve extended attributes
Ernest N. Mamikonyan wrote:
[returning mailing list to CC, please keep it there]
On Tue, 01 Sep 2009 02:16:28 -0400, Ondřej Vašík ova...@redhat.com wrote:
Ernest N. Mamikonyan wrote:
Cp(1) doesn't correctly copy extended attributes for read-only files:
touch foo
setfattr -n user.key -v
0m4.815s
new binary:
real6m8.560s
user0m3.947s
sys 0m30.572s
But correctness should be preferred as this is not so common usecase...
Review appreciated.
Looks ok from my point of view.
Greetings,
Ondřej Vašík
signature.asc
Description: Toto je digitálně podepsaná část
Pádraig Brady wrote:
So there was a lot more disk access with the new binary.
but the NEWS suggests that should only be the case for
systems with dysfunctional readdir.
What was your system?
Quite ancient system... I checked this on my old Fedora Core 6...
Anyway could check with something
Jim Meyering wrote:
Kamil Dudka wrote:
On Monday 24 of August 2009 10:46:26 Jim Meyering wrote:
Actually, the comment I added above has a typo.
s/ENOTSUP/ENOSYS/.
The diagnostic reported corresponds to ENOSYS.
The adjusted patch is here:
Hello,
while making coreutils update for F-11, I spotted one duplicity in
copy.c caused by
http://git.savannah.gnu.org/cgit/coreutils.git/commit/?id=e0cf592f48b4e67dc31d0482ae4f4dbc820e883e
factorization commit. Attached patch removes this duplicity.
Greetings,
Ondřej Vašík
From
time it would be better to use RHEL product support or RedHat
bugzilla. Please try to use it instead of upstream mailing list, if
--session-command is not solving your issue.
Greetings,
Ondřej Vašík
signature.asc
Description: Toto je digitálně podepsaná část zprávy
mentioning in NEWS is not required in that case...
Greetings,
Ondřej Vašík
From 5c9811896936341f462f67fd192748cac2f49dbd Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= ova...@redhat.com
Date: Fri, 22 May 2009 09:47:34 +0200
Subject: [PATCH] dd: do not unnecesarilly
Pádraig Brady wrote:
So that would not run skip(STDOUT_FILENO,...) if count==0.
Would this break existing scripts that for example used
this command to position a non seekable device?
dd count=0 of=/dev/tape seek=1234
Thanks for objection, you are right, my patch seems to be not correct
Ondřej Vašík wrote:
Hello,
as requested via IRC, adding separate test cases for patches added
previously in that thread. It should check correct diagnostics for
SELinux context/xattrs on filesystem without xattr/SELinux
support(fat32).
Sorry for added dosfstools dependency, ext2 mounted
... but is this intentional?
Greetings,
Ondřej Vašík
signature.asc
Description: Toto je digitálně podepsaná část zprávy
___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils
Hello,
as requested via IRC, adding separate test cases for patches added
previously in that thread. It should check correct diagnostics for
SELinux context/xattrs on filesystem without xattr/SELinux
support(fat32).
Greetings,
Ondřej
From e6d3f340ad40635425a63a2edceb7c4a833e048e Mon Sep
Ondřej Vašík wrote:
Pádraig Brady wrote:
Roman Kirillov wrote:
[kiril...@chiark py-src]$ join s1.txt s2.txt
Segmentation fault
files: http://sigizmund.info/upload/s12.tgz (28 mb)
This issue seems related to the i18n patch applied by your distro
Just would like to add that it's
Hello,
Ondřej Vašík wrote:
Jim Meyering píše v Pá 17. 04. 2009 v 13:54 +0200:
Ondřej Vašík wrote:
as reported in rhbz #496142 by Eric Sandeen, mv produces a lot of
unwanted messages when moving files to filesystem without xattr support.
I guess mv selinux/xattr diagnostic error
Ondřej Vašík wrote:
Ondřej Vašík wrote:
It reminds me another issue - due to reduce_diagnostic, if you use cp -a
--preserve=xattr,context , failure diagnostic is reduced, but the
command itself fails. Additionally for cp and --require-preserve=context
and ENODATA/ENOTSUP errors
in Fedora rawhide
package, so I'll fix it in next Fedora coreutils update for F-9/F-10.
In F-8 not, it's already EOL.
Greetings,
Ondřej Vašík
signature.asc
Description: Toto je digitálně podepsaná část zprávy
___
Bug-coreutils mailing list
Hello,
as reported in rhbz #496142 by Eric Sandeen, mv produces a lot of
unwanted messages when moving files to filesystem without xattr support.
I guess mv selinux/xattr diagnostic error messages should be reduced as
is done for cp -a, attached patch should do that.
Greetings,
Ondřej
Jim Meyering wrote:
Pushed.
http://git.savannah.gnu.org/cgit/gnulib.git/commit/?id=1721cf06d9
I guess info doc about suffix regex should be updated too...
Greetings,
Ondrej
From 88ebd00c46668f2b677c8da9a7e40584d5b97d9b Mon Sep 17 00:00:00 2001
From:
Hello,
Jim Meyering wrote:
Ondřej Vašík ova...@redhat.com wrote:
Jim Meyering wrote:
Ondřej Vašík ova...@redhat.com wrote:
What about to split that into two patches? One for SELinux changes and
one for xattr changes (commited after 7.1) - to make it more safe - and
to give other
Hello,
as reported in rhbz #485712 , nl incorrectly states that long format for
-v is --first-page. Info documentation is correct and informs about
--starting-line-number as correct long option for -v. I guess best way
is to fix usage (and to not add --first-page option) as afaik nobody
complained
Jim Meyering wrote:
When will the libattr fix be in F10?
Upstream report:
http://oss.sgi.com/archives/xfs/2009-02/msg00331.html
Talked with Fedora attr maintainer and he will do an update for F-10 and
Rawhide in day or two - upon the upstream reaction. So it should be in
F-10 early next week.
Jim Meyering wrote:
Ondřej Vašík ova...@redhat.com wrote:
What about to split that into two patches? One for SELinux changes and
one for xattr changes (commited after 7.1) - to make it more safe - and
to give other distros more time to fix the leak...
That sounds even better. I see
Jim Meyering wrote:
Unfortunately, with this patch, make check fails the cp/link-heap test.
Investigating why, I discovered that it introduces a huge leak into
cp -a, at least on F10. 80MB worth in this case.
I'll let you investigate that one ;-)
Leak is in libattr - added by 2.4.43 (~8
NOTE: Added documentation of --preserve=context, description of current
behaviour to cp --preserve=all and cp -a documentation. Added tests to
ensure that cp -a actually works with SELinux (until now it was checking
just failures.
Greetings,
Ondřej Vašík
From
Eric Blake wrote:
Bauke Jan Douma bjdouma at xs4all.nl writes:
~ $ date --date 4:59:54 1 hour 53 min 46 sec ago
Wed Jan 28 06:52:08 CST 2009
I've verified that the parse is unchanged between 6.10 and now (7.0+).
It's got to be one of these gnulib patches:
git log 30ffdfc..81e61b
conditional).
Greetings,
Ondřej Vašík
From 18b8f5bf5c18115d0546e42951dc951da58a549c Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= [EMAIL PROTECTED]
Date: Thu, 27 Nov 2008 15:04:10 +0100
Subject: [PATCH] chmod, chown, chgrp: Reduce verbosity of silent mode
* chmod.c
Hi Jim,
I spotted typo in your last factorization patch.
Patch to fix that attached, it also removes runcon/chcon item from TODO.
Ondřej
From 26563a7e759b5f9c2515d06b1efe7d1ccc2ca0ca Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= [EMAIL PROTECTED]
Date: Wed, 26 Nov 2008
Pádraig Brady wrote:
Ondřej Vašík wrote:
Let's do some summary, feel free to add/comment items if you have
something not mentioned here:
What patched ls --user-format can and upstream find -printf not:
1) colored files by LS_COLORS or automatically if requested
2) could be used
Jim Meyering wrote:
Ondřej Vašík [EMAIL PROTECTED] wrote:
as reported in
https://bugs.launchpad.net/ubuntu/+source/coreutils/+bug/187315 by Aaron
Toponce , chmod could display confusing messages when used for
SGID/SUID/sticky bits without permissions to change them.
e.g. with non-root
Jim Meyering wrote:
Ondřej Vašík [EMAIL PROTECTED] wrote:
Throwing out --changes should be ok, if user wants to be informed, he
can parse verbose output easily and common case is to use it without
verbose mode at all. Anyway --verbose output is affected by this issue
as well, so I would
Andreas Schwab wrote:
Eric Blake [EMAIL PROTECTED] writes:
Would a double --verbose make sense?
--verbose=2
Rather than some level-numbers I do prefer --verbose and
--verbose=high ...
Ondřej
signature.asc
Description: Toto je digitálně podepsaná část zprávy
Jim Meyering wrote:
- is it worthwhile to add a --printf option to ls?
I don't like the --user-format name)
You know my opinion ... I guess it is worthwhile, as the consumer of
ls output is generally human and human wishes do differ. I'm ok with
--printf option, you are right - it
is simply not install word and
could not cause confusion).
Greetings,
Ondřej Vašík
signature.asc
Description: Toto je digitálně podepsaná část zprávy
___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo
. Therefore please consider that long
option for ls again.
Greetings,
Ondřej Vašík
signature.asc
Description: Toto je digitálně podepsaná část zprávy
___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug
Pádraig Brady wrote:
Thanks for doing that.
However I thought that item had been removed from the TODO:
http://lists.gnu.org/archive/html/bug-coreutils/2008-02/msg00115.html
This is a large patch to do essentially what `find -printf` already does.
Sorry, haven't seen that thread before.
I
Hello,
thanks for review and objections.
Paul Eggert wrote:
Ondřej Vašík [EMAIL PROTECTED] writes:
- bool changed = (chmod_succeeded
- mode_changed (file, old_mode, new_mode));
+ bool mode_change = mode_changed (file, old_mode, new_mode);
+ bool changed
,
Ondřej Vašík
From 4eaf35f250ab6eec036b7ab21a482a76289f8303 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= [EMAIL PROTECTED]
Date: Fri, 24 Oct 2008 17:24:09 +0200
Subject: [PATCH] chmod: inform in verbose if used mode for chmod was different than requested
* chmod
Hello,
as reported in rh bz#446294 , ls is not very clear with it's exit status
documentation. This patch should clarify it a bit.
Greetings,
Ondřej Vašík
From a3ef5a58f396b53287d7870d2cf35198789240cf Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= [EMAIL
Hi,
Jim Meyering wrote:
Ondřej Vašík [EMAIL PROTECTED] wrote:
as those commands were not documented in info documentation, I wrote
basic info documentation for runcon and chcon command (and related
SELinux context menu section). It is completely based on man
documentation, I would say
it should try) SELinux context. Following changes were done to
make documentation more precise and preserve=context and preserve=all
behaviour after failure SELinux context preserving is documented by that
patch as well.
Greetings,
Ondřej Vašík
From
Hello,
as reported in RedHat bugzilla #463883, chcon option -c is described in
help, but not implemented. After short talk with Jim via IRC decision
was to remove this option and related things from chcon source code
completetely.
It is done in attached patch.
Greetings,
Ondřej Vašík
is in attachement.
Greetings,
Ondřej Vašík
From 0b46a52b8f94fcbf145e0def84174cadd06bc4d2 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= [EMAIL PROTECTED]
Date: Mon, 6 Oct 2008 14:18:53 +0200
Subject: [PATCH] Coreutils.texi: Document runcon and chcon in SELinux context section
Hello,
as reported in RH bugzilla #253817
(https://bugzilla.redhat.com/show_bug.cgi?id=253817), there is an issue
with ls -v (and there will be same issue with sort -V soon). Problem is
with sorting files with extensions or dist-tags (like .tar.gz)
because .tar.gz is more than .1.tar.gz. Therefore
Bruno Haible wrote:
Ondřej Vašík wrote:
3) to use/create different function for handling version sort (like
rpmvercmp in rpm/rpmlib.h recommended in glibc strverscmp() bugzilla)
Does rpmvercmp have the above property?
Similar. It separates name, epoch, version, release and architecture
(-)
diff --git a/ChangeLog b/ChangeLog
index 6c48308..b19680f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2008-07-25 Ondřej Vašík [EMAIL PROTECTED]
+
+ getdate.y: Allow only timezone range specified by POSIX
+ * lib/getdate.y (time_zone_hhmm): Allow only TZ in the range
+ -24 to +24 hours
Hello,
Jim Meyering wrote:
I'm a little leery of this patch, because it makes interpretation
of UTC+dd and UTC-dd dependent on the value of dd. For dd = 14, dd
represents hours. Otherwise, it represents minutes. That would have
to be documented, but seems odd enough that my reflex is to
100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2008-07-04 Ondřej Vašík [EMAIL PROTECTED]
+
+ getdate.y: Allow only timezone range specified by POSIX
+ * lib/getdate.y (time_zone_hhmm): Allow only TZ in the range
+ -24 to +24 hours, consider first two digits between -14 and 14
+ as hours
10db2e5e05c67eea205b3ec76a2408f46356a7fd Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= [EMAIL PROTECTED]
Date: Wed, 2 Jul 2008 14:11:05 +0200
Subject: [PATCH] *doc/coreutils.texi: Mention some missing options for who command
Signed-off-by: Ondřej Vašík [EMAIL PROTECTED]
---
doc
Ondřej Vašík wrote:
Jim Meyering wrote:
Would you mind separating that into a few separate change sets?
...
I hope I will do the rest of patches ( 1 or 2 more
are still necessary to cover all the issues I found).
Here is the second set of patches.
First one (getdate_4) addresses issue
for new fixes of date format
Signed-off-by: Ondřej Vašík [EMAIL PROTECTED]
---
tests/misc/date |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/tests/misc/date b/tests/misc/date
index fb700b5..98651a3 100755
--- a/tests/misc/date
+++ b/tests/misc/date
@@ -118,6 +118,11
relative offset for days/months/years is specified.
Signed-off-by: Ondřej Vašík [EMAIL PROTECTED]
---
lib/getdate.y | 40
1 files changed, 20 insertions(+), 20 deletions(-)
diff --git a/lib/getdate.y b/lib/getdate.y
index 1deec51..7b55c31 100644
--- a/lib
specified user
Signed-off-by: Ondřej Vašík [EMAIL PROTECTED]
---
NEWS |4
src/id.c |7 +--
2 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/NEWS b/NEWS
index 04893c6..ef3feda 100644
--- a/NEWS
+++ b/NEWS
@@ -75,6 +75,10 @@ GNU coreutils NEWS
Jim Meyering wrote:
...
So you'll have to make the three .c files include the new .h,
and you'll have to adjust src/Makefile.am to link the new .c
file to each of those three programs.
...
Then, create the file patch like this:
git format-patch --stdout --signoff HEAD~1 patch
and
Hi Jim,
Jim Meyering wrote:
However, there's a simple work-around: simply to drop the +, e.g.,
$ date -d '20050101 1 day'
Sun Jan 2 00:00:00 CST 2005
Yes, I know about that work-around, you could use unsigned offset/
unsigned offset ago as work-around, but the behaviour still differs
Hello,
date command has some troubles with MMDD date format.
When you try 'date -d 20050101 +1 day' (or whatever
relative offset), you will get invalid date because
of conflicts in translation tables. Nearly same command
date -d 20050101 UTC +1 day is working correctly with
current version
76 matches
Mail list logo