Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 18 Sep 2007 15:41:52 -0600
From:      Scott Long <scottl@samsco.org>
To:        "Bruce A. Mah" <bmah@freebsd.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:  <46F04620.3090500@samsco.org>
In-Reply-To: <46F03493.7050808@freebsd.org>
References:  <200709180420.l8I4KMFG057365@repoman.freebsd.org> <20070918145953.GW79542@deviant.kiev.zoral.com.ua> <46F02B62.8080504@samsco.org> <46F03493.7050808@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Bruce A. Mah wrote:
> 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 
>>>>     lib/libarchive/test  test_write_disk.c 
>>>>   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.
>>>>   
>>>>   Approved by: re (bmah)
>>>>   Pointy hat: \me
>>>>   MFC after: 5 days
>>>>   
>>>>   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.
>> I think this patch needs to be backed out until further review is 
>> obtained.  This isn't a good time to leave things in a broken state for 
>> a week while we try to figure out the "right" thing to do.  Just back it
>> 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.
> 
> 

Thanks a lot Bruce!

Scott




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