From owner-freebsd-current@freebsd.org Thu Oct 6 15:23:12 2016 Return-Path: Delivered-To: freebsd-current@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 7FA24BE9C6B for ; Thu, 6 Oct 2016 15:23:12 +0000 (UTC) (envelope-from sbruno@freebsd.org) Received: from mail.ignoranthack.me (ignoranthack.me [199.102.79.106]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 519C5AFA for ; Thu, 6 Oct 2016 15:23:12 +0000 (UTC) (envelope-from sbruno@freebsd.org) Received: from [192.168.0.6] (97-123-129-224.albq.qwest.net [97.123.129.224]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: sbruno@ignoranthack.me) by mail.ignoranthack.me (Postfix) with ESMTPSA id 5B4981928AD for ; Thu, 6 Oct 2016 15:23:11 +0000 (UTC) Subject: Re: Large patch/diff refuses to apply To: freebsd-current@freebsd.org References: <20161006142339.GF6177@zxy.spb.ru> From: Sean Bruno Message-ID: Date: Thu, 6 Oct 2016 09:23:10 -0600 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="XpslqhaiTrVv4iKSDRvJx9lsjNj7qMkX5" X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 06 Oct 2016 15:23:12 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --XpslqhaiTrVv4iKSDRvJx9lsjNj7qMkX5 Content-Type: multipart/mixed; boundary="7NAX0BuCvSPqDvwN6XbLCugbxVLeP0u76" From: Sean Bruno To: freebsd-current@freebsd.org Message-ID: Subject: Re: Large patch/diff refuses to apply References: <20161006142339.GF6177@zxy.spb.ru> In-Reply-To: --7NAX0BuCvSPqDvwN6XbLCugbxVLeP0u76 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 10/06/16 09:19, Sean Bruno wrote: >=20 >=20 > On 10/06/16 09:00, Alan Somers wrote: >> On Thu, Oct 6, 2016 at 8:41 AM, Sean Bruno wrote:= >>> >>> >>> On 10/06/16 08:23, Slawa Olhovchenkov wrote: >>>> On Thu, Oct 06, 2016 at 07:57:57AM -0600, Sean Bruno wrote: >>>> >>>>> I'm doing a large amount of change to ixgbe(4) in support of IFLIB >>>>> implementations and running into failures when trying to apply larg= e >>>>> diffs. This is causing phabricator reviews to be unuseable as well= =2E >>>>> >>>>> I've setup two trees to test this. The first tree is used to gener= ate >>>>> the diff and the second (vanilla) is used to apply the diff. The e= ntire >>>>> patch fails to apply, so I'm assuming that the size of the diff is >>>>> failing because of a sanity check or something. >>>> >>>> No. This is expanded/collapsed keywords issuse: >>>> >>>> =3D=3D=3D >>>> -*******************************************************************= ***********/ >>>> -/*$FreeBSD$*/ >>>> =3D=3D=3D >>>> >>>> svn diff over repo generate patch w/ collapsed keywords. >>>> At working copy all keywords expanded. >>>> >>> >>> Ah, I see. Thank you. >>> >>> I am regenerating the failed files now. That seems to work (if I lea= ve >>> the keywords alone). >>> >>> sean >>> >> >> Also, I think the "svn patch" command (as opposed to plain "patch") >> can deal with the RCS keywords. >> >=20 > The use of "svn cp" also seems to generate a lot of failures as well. >=20 > I do want to maintain history between certain files when I go to commit= =2E > e.g. svn cp fileA new_fileA, but this seems to confuse the crap out of= > diff/patch no matter what I do. This leads to suffering with > phabricator as well. >=20 > For now, I'm not going to do the "svn cp" part of my commit, but I will= > try and do this when I am ready to shove my work into current. >=20 > sean >=20 A good example of what I was trying to do: svn mkdir sys/modules/ix/ix_iflib svn cp sys/modules/ix/Makefile sys/modules/ix/ix/ix_iflib/Makefile That seems to really confuse things. sean --7NAX0BuCvSPqDvwN6XbLCugbxVLeP0u76-- --XpslqhaiTrVv4iKSDRvJx9lsjNj7qMkX5 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQF8BAEBCgBmBQJX9mxeXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRCQUFENDYzMkU3MTIxREU4RDIwOTk3REQx MjAxRUZDQTFFNzI3RTY0AAoJEBIB78oecn5kMGkH/31eD8SE/b6oAv2wtJMTtzHp 2Ey8lcrRZyg7P21jw10yfdcVFwURzQqTuyMFJU8VAtb/1BvlIRVi0wG18jqNTbo4 XJhXUe6SbL5Pq+SgIPR/bDGtqXB6eDeYJTsaiLRdVErC5C1QKeVuyi48P6+GNfII TMLBNsZT/MUeBykXt0/AtIQ3gaE1X6fe794bVlkeWqReqgDTcqembAoq1ByhRybo 6DFKOoCqdR1qz+Q+o8hDWD3TOrJmoKJDIx+RGM6X0IzF0bwJaiswCtF8LQzyG3Js Ht9hir173cINmKFcvrqML9MxIV/0SpLKbLff8b/r3zUNLG5FQl/gbuXvk6EeADI= =iLQy -----END PGP SIGNATURE----- --XpslqhaiTrVv4iKSDRvJx9lsjNj7qMkX5--