Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 25 Feb 2013 11:35:08 +0100
From:      Christoph Mallon <christoph.mallon@gmx.de>
To:        Pawel Jakub Dawidek <pjd@FreeBSD.org>
Cc:        freebsd-arch@FreeBSD.org
Subject:   Re: Large Capsicum patch for review.
Message-ID:  <512B3E5C.2090506@gmx.de>
In-Reply-To: <20130224193058.GW1377@garage.freebsd.pl>
References:  <20130213025547.GA2025@garage.freebsd.pl> <20130213230221.GB1375@garage.freebsd.pl> <20130223221116.GR1377@garage.freebsd.pl> <5129ADC5.5040306@gmx.de> <20130224193058.GW1377@garage.freebsd.pl>

next in thread | previous in thread | raw e-mail | index | archive | help
On 24.02.2013 20:30, Pawel Jakub Dawidek wrote:
> On Sun, Feb 24, 2013 at 07:05:57AM +0100, Christoph Mallon wrote:
>> On 23.02.2013 23:11, Pawel Jakub Dawidek wrote:
>>> On Thu, Feb 14, 2013 at 12:02:22AM +0100, Pawel Jakub Dawidek wrote:
>>>> I'd like to commit this patch:
>>>>
>>>> 	http://people.freebsd.org/~pjd/patches/capkern.diff
>>>
>>> The patch was updated after the following changes and is available at
>>> the link above:
>>
>> I was not able to apply this patch cleanly and had to fudge with the diff:
>> - Two diff headers (contrib/openbsm/etc/audit_event and lib/libc/gen/Makefile.inc) have an extra space after --- and +++, which is recognized as part of the filename.
>>   Was this patch manually altered?
> 
> Nope, but I'm using some script to generate patch(1)-compatbile diff
> from a perforce diff.

Ugh, why is p4 still in use, if it is just a hassle and hides history?

>> - Two diffs (lib/libc/sys/cap_new.2 and sys/kern/uipc_mqueue.c) contain unexpanded $FreeBSD$ tags.
>> I also had to guess, that the patch is to be applied onto r247201.
>> I placed a cleaned up patch at http://tron.homeunix.org/zeug/FreeBSD/capsicum/0001-Capsicum-update.patch.
>>
>> This is a really big patch bomb changing lots of unrelated things.
>> Do you have smaller, more managable diffs for easier review?
> 
> I don't have smaller patch, unfortunately, but I don't think there are
> unrelated things in there. Can you point me at them?

The long list in your other email points out the unrealated stuff.

	Christoph



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?512B3E5C.2090506>