Hello community, here is the log from the commit of package coreutils for openSUSE:11.4 checked in at Tue Jun 7 14:06:06 CEST 2011.
-------- --- old-versions/11.4/all/coreutils/coreutils.changes 2011-01-14 14:27:11.000000000 +0100 +++ /mounts/work_src_done/11.4/coreutils/coreutils.changes 2011-06-07 12:14:47.000000000 +0200 @@ -1,0 +2,6 @@ +Sat Apr 2 17:48:37 UTC 2011 - sweet_...@gmx.de + +- fix "cp -al does not preserve timestamps of symlinks" + (bnc#696389, coreutils 8.0 - 8.10, openSUSE 11.3 and 11.4) + +------------------------------------------------------------------- Package does not exist at destination yet. Using Fallback old-versions/11.4/all/coreutils Destination is old-versions/11.4/UPDATES/all/coreutils calling whatdependson for 11.4-i586 New: ---- coreutils-cp-al-symlink-timestamp.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ coreutils.spec ++++++ --- /var/tmp/diff_new_pack.VekO3j/_old 2011-06-07 14:04:30.000000000 +0200 +++ /var/tmp/diff_new_pack.VekO3j/_new 2011-06-07 14:04:30.000000000 +0200 @@ -24,7 +24,7 @@ License: GFDLv1.2 ; GPLv2+ ; GPLv3+ Group: System/Base Version: 8.9 -Release: 2 +Release: 5.<RELEASE6> Provides: fileutils = %{version}, sh-utils = %{version}, stat = %version}, textutils = %{version}, mktemp = %{version} Obsoletes: fileutils < %{version}, sh-utils < %{version}, stat < %version}, textutils < %{version}, mktemp < %{version} Obsoletes: libselinux <= 1.23.11-3 libselinux-32bit = 9 libselinux-64bit = 9 libselinux-x86 = 9 @@ -56,6 +56,8 @@ Patch31: coreutils-getaddrinfo.patch Patch32: coreutils-ptr_int_casts.patch Patch33: coreutils-8.9-singlethreaded-sort.patch +# this patch is needed for coreutils 8.0 - 8.10 (openSUSE 11.3 and 11.4) +Patch34: coreutils-cp-al-symlink-timestamp.patch BuildRoot: %{_tmppath}/%{name}-%{version}-build # this will create a cycle, broken up randomly - coreutils is just too core to have other # prerequires @@ -96,6 +98,7 @@ %patch31 %patch32 %patch33 +%patch34 -p1 cp %{S:4} po/de.po %build ++++++ _service:set_version:coreutils.spec ++++++ --- /var/tmp/diff_new_pack.VekO3j/_old 2011-06-07 14:04:31.000000000 +0200 +++ /var/tmp/diff_new_pack.VekO3j/_new 2011-06-07 14:04:31.000000000 +0200 @@ -24,7 +24,7 @@ License: GFDLv1.2 ; GPLv2+ ; GPLv3+ Group: System/Base Version: 8.9 -Release: 2 +Release: 0 Provides: fileutils = %{version}, sh-utils = %{version}, stat = %version}, textutils = %{version}, mktemp = %{version} Obsoletes: fileutils < %{version}, sh-utils < %{version}, stat < %version}, textutils < %{version}, mktemp < %{version} Obsoletes: libselinux <= 1.23.11-3 libselinux-32bit = 9 libselinux-64bit = 9 libselinux-x86 = 9 @@ -56,6 +56,8 @@ Patch31: coreutils-getaddrinfo.patch Patch32: coreutils-ptr_int_casts.patch Patch33: coreutils-8.9-singlethreaded-sort.patch +# this patch is needed for coreutils 8.0 - 8.10 (openSUSE 11.3 and 11.4) +Patch34: coreutils-cp-al-symlink-timestamp.patch BuildRoot: %{_tmppath}/%{name}-%{version}-build # this will create a cycle, broken up randomly - coreutils is just too core to have other # prerequires @@ -96,6 +98,7 @@ %patch31 %patch32 %patch33 +%patch34 -p1 cp %{S:4} po/de.po %build ++++++ coreutils-cp-al-symlink-timestamp.patch ++++++ >From 594292a1d89332dd006cd7fe3116cf46ebe9acf6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?P=C3=A1draig=20Brady?= <p...@draigbrady.com> Date: Thu, 31 Mar 2011 11:28:58 +0100 Subject: copy: link rather than copy symlinks, when --link used This bug was introduced in commit ca9e212c, 2009-09-24, "cp, mv: use linkat to guarantee semantics", which inadvertently disabled the creation of hardlinks to symlinks. However rather than implementing the intention of that commit and relying on gnulib linkat emulation, we'll revert to the previous emulation as that maintains ownership and timestamps. * src/copy.c (copy_internal): Use our existing hardlink to symlink emulation when link() might dereference the symlink. Also ensure that we copy the timestamps of the original symlink when we use the emulation. * tests/cp/link-symlink: Add a test to ensure timestamps copied. * tests/Makefile.am: Reference the new test. * NEWS: Mention the fix. Reported by Ruediger Meier --- src/copy.c | 27 ++++++++++++++++++++------- tests/Makefile.am | 1 + tests/cp/link-symlink | 41 +++++++++++++++++++++++++++++++++++++++++ 4 files changed, 66 insertions(+), 7 deletions(-) create mode 100755 tests/cp/link-symlink diff --git a/src/copy.c b/src/copy.c index 104652d..e839f97 100644 --- a/src/copy.c +++ b/src/copy.c @@ -2169,13 +2169,24 @@ copy_internal (char const *src_name, char const *dst_name, } } - /* cp, invoked with `--link --no-dereference', should not follow the - link; we guarantee this with gnulib's linkat module (on systems - where link(2) follows the link, gnulib creates a symlink with - identical contents, which is good enough for our purposes). */ + /* POSIX 2008 states that it is implementation-defined whether + link() on a symlink creates a hard-link to the symlink, or only + to the referent (effectively dereferencing the symlink) (POSIX + 2001 required the latter behavior, although many systems provided + the former). Yet cp, invoked with `--link --no-dereference', + should not follow the link. We can approximate the desired + behavior by skipping this hard-link creating block and instead + copying the symlink, via the `S_ISLNK'- copying code below. + LINK_FOLLOWS_SYMLINKS is tri-state; if it is -1, we don't know + how link() behaves, so we use the fallback case for safety. + + Note gnulib's linkat module, guarantees that the symlink is not + dereferenced. However its emulation currently doesn't maintain + timestamps or ownership so we only call it when we know the + emulation will not be needed. */ else if (x->hard_link - && (!S_ISLNK (src_mode) - || x->dereference != DEREF_NEVER)) + && !(LINK_FOLLOWS_SYMLINKS && S_ISLNK (src_mode) + && x->dereference == DEREF_NEVER)) { if (linkat (AT_FDCWD, src_name, AT_FDCWD, dst_name, 0)) { @@ -2298,7 +2309,9 @@ copy_internal (char const *src_name, char const *dst_name, /* If we've just created a hard-link due to cp's --link option, we're done. */ - if (x->hard_link && ! S_ISDIR (src_mode)) + if (x->hard_link && ! S_ISDIR (src_mode) + && !(LINK_FOLLOWS_SYMLINKS && S_ISLNK (src_mode) + && x->dereference == DEREF_NEVER)) return delayed_ok; if (copied_as_regular) diff --git a/tests/Makefile.am b/tests/Makefile.am index 99b46e8..e7f3fff 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -328,6 +328,7 @@ TESTS = \ cp/link \ cp/link-no-deref \ cp/link-preserve \ + cp/link-symlink \ cp/no-deref-link1 \ cp/no-deref-link2 \ cp/no-deref-link3 \ diff --git a/tests/cp/link-symlink b/tests/cp/link-symlink new file mode 100755 index 0000000..443b337 --- /dev/null +++ b/tests/cp/link-symlink @@ -0,0 +1,41 @@ +#!/bin/sh +# Ensure that cp -a --link maintains timestamps if possible + +# Copyright (C) 2011 Free Software Foundation, Inc. + +# This program is free software: you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation, either version 3 of the License, or +# (at your option) any later version. + +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. + +# You should have received a copy of the GNU General Public License +# along with this program. If not, see <http://www.gnu.org/licenses/>. + +. "${srcdir=.}/init.sh"; path_prepend_ ../src +print_ver_ cp + +# Check that the timestamps of the symlink are copied +# if we're using hardlink to symlink emulation. +touch file +ln -s file link || framework_failure +touch -m -h -d 2011-01-01 link || + skip_test_ "Your system doesn't support updating symlink timestamps" +case `stat --format=%y link` in + 2011-01-01*) ;; + *) skip_test_ "Your system doesn't support updating symlink timestamps" ;; +esac + +# link.cp is probably a hardlink, but may also be a symlink +# In either case the timestamp should match the original. +cp -al link link.cp +case `stat --format=%y link.cp` in + 2011-01-01*) ;; + *) fail=1 ;; +esac + +Exit $fail -- 1.7.3.4 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Remember to have fun... -- To unsubscribe, e-mail: opensuse-commit+unsubscr...@opensuse.org For additional commands, e-mail: opensuse-commit+h...@opensuse.org