Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 18 Sep 2007 13:26:59 -0700
From:      "Bruce A. Mah" <bmah@freebsd.org>
To:        Scott Long <scottl@samsco.org>
Cc:        Kostik Belousov <kostikbel@gmail.com>, cvs-src@freebsd.org, Tim Kientzle <kientzle@freebsd.org>, src-committers@freebsd.org, cvs-all@freebsd.org
Subject:   Re: cvs commit: src/lib/libarchive archive_write_disk.c src/lib/libarchive/test test_write_disk.c
Message-ID:  <46F03493.7050808@freebsd.org>
In-Reply-To: <46F02B62.8080504@samsco.org>
References:  <200709180420.l8I4KMFG057365@repoman.freebsd.org> <20070918145953.GW79542@deviant.kiev.zoral.com.ua> <46F02B62.8080504@samsco.org>

next in thread | previous in thread | raw e-mail | index | archive | help
This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
--------------enig1E796523A547853458C33860
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

If memory serves me right, Scott Long wrote:
> Kostik Belousov wrote:
>> On Tue, Sep 18, 2007 at 04:20:21AM +0000, Tim Kientzle wrote:
>>> kientzle    2007-09-18 04:20:21 UTC
>>>
>>>   FreeBSD src repository
>>>
>>>   Modified files:
>>>     lib/libarchive       archive_write_disk.c=20
>>>     lib/libarchive/test  test_write_disk.c=20
>>>   Log:
>>>   Correct the return value from archive_write_data()
>>>   (when used to restore files to disk) to match:
>>>     * The documentation
>>>     * The return values of this function when used
>>>       to write files into an archive.
>>>  =20
>>>   Approved by: re (bmah)
>>>   Pointy hat: \me
>>>   MFC after: 5 days
>>>  =20
>>>   Revision  Changes    Path
>>>   1.15      +7 -4      src/lib/libarchive/archive_write_disk.c
>>>   1.4       +26 -1     src/lib/libarchive/test/test_write_disk.c

[snip]

>> Stop in /usr/ports/audio/sdl_mixer.
>> *** Error code 1
>>
>> It seems that tar is broken.
>=20
> I think this patch needs to be backed out until further review is=20
> obtained.  This isn't a good time to leave things in a broken state for=
=20
> a week while we try to figure out the "right" thing to do.  Just back i=
t
> out to a known good state, then discuss and argue at your leisure.

With my RE hat on, I've backed out this commit.

Bruce.



--------------enig1E796523A547853458C33860
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFG8DST2MoxcVugUsMRAioqAKDMB125+Lm0GatJNAfhLI1h1upG+wCfXiy8
uluRtjCsuaq1GqPo4HK9vtI=
=mnt0
-----END PGP SIGNATURE-----

--------------enig1E796523A547853458C33860--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?46F03493.7050808>