From owner-freebsd-net@FreeBSD.ORG Thu May 28 17:12:22 2015 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 88A39C4A; Thu, 28 May 2015 17:12:22 +0000 (UTC) (envelope-from delphij@delphij.net) Received: from anubis.delphij.net (anubis.delphij.net [64.62.153.212]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "anubis.delphij.net", Issuer "StartCom Class 1 Primary Intermediate Server CA" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 68A3D27A; Thu, 28 May 2015 17:12:22 +0000 (UTC) (envelope-from delphij@delphij.net) Received: from zeta.ixsystems.com (c-71-202-112-39.hsd1.ca.comcast.net [71.202.112.39]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by anubis.delphij.net (Postfix) with ESMTPSA id D85E3141D9; Thu, 28 May 2015 10:12:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=delphij.net; s=anubis; t=1432833135; x=1432847535; bh=IKot36W9NfLqJhSb+V7kl8g9aNZvdLLazxqdZ9radGw=; h=Date:From:Reply-To:To:CC:Subject:References:In-Reply-To; b=kJWxS1m6thfRl9+Ka7cmvyTZRCqJNxzp2MALBBBhLHrJ31rc2/Jyl3Cpgyx1Ah662 cjcq3XEE6xRzffkJ/n8JUlEzW5PCYjic4xNjFnZNN13hTUsT+gToeuBBEsja0SJp+5 XNeXDDopiGMTJDbfznHp5SsCQhqEiFTocrFEEtI4= Message-ID: <55674C6B.9050700@delphij.net> Date: Thu, 28 May 2015 10:12:11 -0700 From: Xin Li Reply-To: d@delphij.net Organization: The FreeBSD Project MIME-Version: 1.0 To: hiren panchasara , Patrick Kelsey CC: "freebsd-net@freebsd.org" , delphij@FreeBSD.org Subject: Re: Looking for input on "locally patch tcpdump or merge in latest release from upstream?" References: <20150528045551.GS95600@strugglingcoder.info> In-Reply-To: <20150528045551.GS95600@strugglingcoder.info> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 May 2015 17:12:22 -0000 -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 Hi, Hiren, On 05/27/15 21:55, hiren panchasara wrote: > On 05/28/15 at 12:40P, Patrick Kelsey wrote: >> Hi, >> >> I've had a patch for a capsicum-related issue in tcpdump sitting >> around since last September ( >> https://lists.freebsd.org/pipermail/freebsd-current/2014-September/05 2049.html) >> >> that is still needed and that I want finally address in the tree (the pa tch >> was reviewed by rwatson@ and pjd@ back then). >> >> This issue was patched separately in the upstream tcpdump sources >> in February ( >> https://github.com/the-tcpdump-group/tcpdump/commit/887bf88fd058f8c0e f9a5af1a95b43753e3ad2eb), >> >> along with a refactor of the associated capsicum code, and that work has >> been present in tcpdump releases since 4.7.3 ( >> http://www.tcpdump.org/tcpdump-changes.txt). >> >> The last tcpdump release imported into the FreeBSD tree was 4.6.2 >> ( http://svnweb.freebsd.org/base/vendor/tcpdump/). >> >> tcpdump release import/merges have recently resulted in some >> confusion/lost local patches due to the extent of the diffs >> (e.g., the thread at >> https://lists.freebsd.org/pipermail/svn-src-head/2015-February/067853 .html). >> >> >> I see three possible ways to proceed: >> >> 1. Apply the minimal-local-diff patch from last September to our >> local tcpdump sources. This seems like it might contribute to a >> future difficult/lossy tcpdump vendor import/merge. >> >> 2. Import tcpdump 4.7.3 or later to address this issue. Are >> there any reasons why this might not be desired? I don't have a >> feel for when/why past tcpdump vendor imports have been performed >> or avoided. >> >> 3. Cherry-pick the upstream patch and apply it to our local >> sources, directly addressing only this issue and avoiding future >> tcpdump vendor import/merge problems related to this issue. >> >> I'm looking for input on the above. If left to my own devices, >> I'd go with (3). > > Latest upstream release is 4.7.4 and the one before that was 4.6.2 > which we already have in the tree. I think we should get latest > instead of picking bits and pieces when possible. > > CCing Xin for his input as he has been doing last few imports. Yes, I think we should do the new import. (Are you willing to do that? I'm doing it only because nobody else were doing it...) Cheers, - -- Xin LI https://www.delphij.net/ FreeBSD - The Power to Serve! Live free or die -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.1.4 (FreeBSD) iQIcBAEBCgAGBQJVZ0xrAAoJEJW2GBstM+nstbUP/jER9jTYevNftIMcTrbDhtKn iV2QjquvGHQdf7bkKQujLuaMoZG9GKqzyg0XT+d9yL7vk+kBvAqPRuQ6hJgaf4GH VgzL1nN7g+U5PCA+98nszF81/NiFHkz1Ag5ayLl/X2TxQaTgMijT4hulkMEN8idh 6c17RC7YCWkkQAerrB4N0aj08MiXZJ3L2/SRsJZwHCfpzXZNSubA/mjxwD4QMtR2 kAFkLc0FfAVIWN8rxYEmmnj+YlYQRrMKhs0/ZfyrkPDHQDJ5MJZWHcrkTAdaJ8I+ Fp2sTezfCrkRrNyVOu2XQWRkV6IfQxXNsFmvzn3kbAzSO68SiKvZ6fG6uFaa2A21 anjoqBshTaxEXMq+v0BI2oCZDbJF9CKpjMUmvevDRofPWsr6ROb1ElUsXgLdvPoe QcvYC7S/Qf2DnRz3lbvSh4YeRowj9ridSQm3g3fdRQURCEWnsWK6NahclBqJqWy1 A15ZKPMLj/Su9po3dtl4gr0AvS2g7TPa9lcGMSjebQ+hiMYxE0Bc0RsGAvytZ+AE uUdbT+0u6IGn3QldkYJfwZFwuVYLJQFug8L3QOScGxZEBrxZcZdzAMVQffypPgPt bsiAk974pwqHHmf8C1mdmYf5H65wpnOTNg5o/uQ19KiMsjxpRPspWTy7BRDZlkUO kWm4dBZUgYxYIqZcFoT7 =0LqE -----END PGP SIGNATURE-----