From owner-freebsd-arch@FreeBSD.ORG Sun Feb 24 19:29:59 2013 Return-Path: Delivered-To: freebsd-arch@FreeBSD.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id A42116A for ; Sun, 24 Feb 2013 19:29:59 +0000 (UTC) (envelope-from pawel@dawidek.net) Received: from mail.dawidek.net (garage.dawidek.net [91.121.88.72]) by mx1.freebsd.org (Postfix) with ESMTP id 6317FF82 for ; Sun, 24 Feb 2013 19:29:59 +0000 (UTC) Received: from localhost (89-73-195-149.dynamic.chello.pl [89.73.195.149]) by mail.dawidek.net (Postfix) with ESMTPSA id 1204EAF7; Sun, 24 Feb 2013 20:26:54 +0100 (CET) Date: Sun, 24 Feb 2013 20:30:58 +0100 From: Pawel Jakub Dawidek To: Christoph Mallon Subject: Re: Large Capsicum patch for review. Message-ID: <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> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="vRjLMflJ/YJvC+Q3" Content-Disposition: inline In-Reply-To: <5129ADC5.5040306@gmx.de> X-OS: FreeBSD 10.0-CURRENT amd64 User-Agent: Mutt/1.5.21 (2010-09-15) Cc: freebsd-arch@FreeBSD.org X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 24 Feb 2013 19:29:59 -0000 --vRjLMflJ/YJvC+Q3 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Feb 24, 2013 at 07:05:57AM +0100, Christoph Mallon wrote: > Hi. >=20 > 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 > >=20 > > The patch was updated after the following changes and is available at > > the link above: >=20 > 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/Make= file.inc) have an extra space after --- and +++, which is recognized as par= t of the filename. > Was this patch manually altered? Nope, but I'm using some script to generate patch(1)-compatbile diff =66rom a perforce diff. > - Two diffs (lib/libc/sys/cap_new.2 and sys/kern/uipc_mqueue.c) contain u= nexpanded $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/caps= icum/0001-Capsicum-update.patch. >=20 > 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? > I started reading the patch and found some minor glitches, e.g. mode in c= ap_sandboxed() should be u_int, not int. > I will report more later. I updated the patch. It is now against r247212. Thanks! --=20 Pawel Jakub Dawidek http://www.wheelsystems.com FreeBSD committer http://www.FreeBSD.org Am I Evil? Yes, I Am! http://tupytaj.pl --vRjLMflJ/YJvC+Q3 Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (FreeBSD) iEYEARECAAYFAlEqanIACgkQForvXbEpPzQCogCg+Rqb9z9u79/kKuYpRFhXYmpO bMMAoO5QsaHey4doF5qHEL9lHe2ax2Xx =gs/B -----END PGP SIGNATURE----- --vRjLMflJ/YJvC+Q3--