From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from lists.gentoo.org ([140.105.134.102] helo=robin.gentoo.org) by finch.gentoo.org with esmtp (Exim 4.60) (envelope-from ) id 1IquWK-0007gx-QN for garchives@archives.gentoo.org; Sat, 10 Nov 2007 17:53:25 +0000 Received: from robin.gentoo.org (localhost [127.0.0.1]) by robin.gentoo.org (8.14.2/8.14.0) with SMTP id lAAHqUoX006709; Sat, 10 Nov 2007 17:52:30 GMT Received: from mail.obsidian-studios.com (72-17-152-164.jax.fdn.com [72.17.152.164]) by robin.gentoo.org (8.14.2/8.14.0) with ESMTP id lAAHoT9j004232 for ; Sat, 10 Nov 2007 17:50:29 GMT Received: (qmail 17679 invoked from network); 10 Nov 2007 17:50:30 -0000 Received: from mail.obsidian-studios.com (HELO ?192.168.1.211?) (192.168.0.51) by mail.obsidian-studios.com with SMTP; 10 Nov 2007 17:50:30 -0000 Received: from [192.168.1.211] ([192.168.1.211] helo=[192.168.1.211]) by assp.obsidian-studios.com; 10 Nov 2007 12:50:30 -0500 Subject: Re: [gentoo-dev] to patch or sed was -> repoman and checking for correct quoting From: "William L. Thomson Jr." To: gentoo-dev@lists.gentoo.org In-Reply-To: <20071110173623.GA32402@shuttle> References: <47358D39.8050901@gentoo.org> <20071110173623.GA32402@shuttle> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-vI1EHqrRpNwsuzeDrGZD" Organization: Gentoo Date: Sat, 10 Nov 2007 12:50:29 -0500 Message-Id: <1194717029.13773.26.camel@wlt.obsidian-studios.com> Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@gentoo.org Reply-to: gentoo-dev@lists.gentoo.org Mime-Version: 1.0 X-Mailer: Evolution 2.12.1 X-Archives-Salt: 2a24f0aa-df81-4d97-bf8f-debc2965d82b X-Archives-Hash: fba8b4190e89d0d128048a5bccc5a0db --=-vI1EHqrRpNwsuzeDrGZD Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Sat, 2007-11-10 at 18:36 +0100, Bo =C3=98rsted Andresen wrote: > On Sat, Nov 10, 2007 at 11:51:37AM +0100, Krzysiek Pawlik wrote: > > It's purpose is to remove the ${D} from makefile, additionally ${D} is = in single > > quotes, so it will not be expanded - is it a bug in repoman check? >=20 > What ${D} ? I see none in that makefile. Which is why I think a patch wou= ld be=20 > better in this case. A patch would actually tell you when half of the > substitutions don't match anything... I think this is a point worth makin= g even > if it doesn't change your point. I have kinda gone back and forth between patches and sed. While I agree patches are better since they fail to apply at times. Sed stuff can be more portable amongst revisions, as I have found with maintaining assp. Where the patch grew to the point it had to be mirrored :(. Now I know some will scream, puke, throw up in their mouth, and so on. But seems like sed should have an OPTIONAL argument or etc to tell sed to either fail if it can't make the change anywhere. And/or keep/output a count of how many things were modified. Wrt to assp, how I am presently using it, is on a rev bump or etc. I will have to ebuild run it's sed. Then I will diff the installed version that was sed during merge with the original. Then review that diff/patch to make sure all was modified per the sed. If not address the sed. Once the resulting diff/patch is what it would be if I were using a patch. Then I commit the ebuild. Where I had to make a patch per assp version. The above technique allowed me to rip through testing 3 different versions with the same sed for each. Verses having to make 3 different patches for each. It seems much faster and way more ideal. There is still the possibility of sed failing. But since I rule that out before commit. Once committed chance of it failing is pretty nill. Not to mention no external files. Less to maintain. less for users to sync, download, etc. It's a really hard call, but for most single line modifications. These days I am really leaning toward sed vs patches. --=20 William L. Thomson Jr. Gentoo/Java --=-vI1EHqrRpNwsuzeDrGZD Content-Type: application/pgp-signature; name=signature.asc Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.7 (GNU/Linux) iD8DBQBHNe9fPrChP8zZLyYRAkVbAJ9XDjL4lHyJbXKCMSwU1PrCqSm9UACfanJy NzShPGwTdjZLU5LYDF1mHac= =MrkD -----END PGP SIGNATURE----- --=-vI1EHqrRpNwsuzeDrGZD-- -- gentoo-dev@gentoo.org mailing list