--- /dev/null
+Return-Path: <mpn@google.com>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+ by olra.theworths.org (Postfix) with ESMTP id 4A5B7431FB6\r
+ for <notmuch@notmuchmail.org>; Mon, 3 Sep 2012 08:36:23 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.7\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
+ tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_LOW=-0.7]\r
+ autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+ by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+ with ESMTP id 4XG4eWRjZ6cb for <notmuch@notmuchmail.org>;\r
+ Mon, 3 Sep 2012 08:36:22 -0700 (PDT)\r
+Received: from mail-ee0-f53.google.com (mail-ee0-f53.google.com\r
+ [74.125.83.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client\r
+ certificate requested) by olra.theworths.org (Postfix) with ESMTPS id\r
+ 02D7E431FAF for <notmuch@notmuchmail.org>; Mon, 3 Sep 2012 08:36:21 -0700\r
+ (PDT)\r
+Received: by eekb47 with SMTP id b47so2346482eek.26\r
+ for <notmuch@notmuchmail.org>; Mon, 03 Sep 2012 08:36:19 -0700 (PDT)\r
+DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;\r
+ s=20120113; h=sender:from:to:subject:in-reply-to:organization:references\r
+ :user-agent:x-face:face:x-pgp:x-pgp-fp:date:message-id:mime-version\r
+ :content-type; bh=u/k3i18RAV1KI9kS84iqGTaXO9KIxfONtaTiwj+h7to=;\r
+ b=GPreNjW6g5qgjL3/k8Z7lCBSWNP/ky/rxHZdzFRFFz0MJcVlz22hbhgsoI6F1JXfC2\r
+ oa5kIr9yOOqfq55ACAdJjxdF6gbr+bwUPpqnJfMYskRD46N2y5/C4dqv0YhW1Qxph2Xv\r
+ bFOj509AF2A3Tj6aLpyinQEiVHVHEzMofObBuv/URDTKhsJPh9i/7inEqbtvL/7swrSr\r
+ MRg9b9NncSlpXOINh3zvFqjWZag/sQL5ZMzxiDZMHQGgTpEFFV+NHlSHenyVY9SGXgCj\r
+ +ulnBl+tzgm8DuED9624y0NqzpWL+I+xtgFEvUIE4AQxBEzcSpAbdoEdYHY1dISC65DX ZPFQ==\r
+X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
+ d=google.com; s=20120113;\r
+ h=sender:from:to:subject:in-reply-to:organization:references\r
+ :user-agent:x-face:face:x-pgp:x-pgp-fp:date:message-id:mime-version\r
+ :content-type:x-gm-message-state;\r
+ bh=u/k3i18RAV1KI9kS84iqGTaXO9KIxfONtaTiwj+h7to=;\r
+ b=SW1fJqPH3IRCmei69F76hczfgoL8PLeDeKiaOACY/2wkeBFCM0u9Z4cED+hYTqARw6\r
+ /xzKn32tZA+oUjXs+sEl9ZMQVgLviLz2M0Jm+XwPv/DgumxmPTGfT+bookjOrdnUC89Z\r
+ kK5juRen1E+F2QXV4lAeOFJnq4MBu2JwkSy7hIq1aA37FForEnaNDxAE2Mue2HN6Ppqs\r
+ 0N/59buOzHauRG1sb4zER/hkFG1PGEn5cU0lIYdHxXjYQW3ZE6uRUD6nuTennzh7oLzs\r
+ oXnks92wPoxHeR6HN95RzLquaKZxF9N5gHy8izaD1E+QxlgtIr1e+5d5przlpu7mPMzQ\r
+ cyPg==\r
+Received: by 10.14.4.201 with SMTP id 49mr22331554eej.0.1346686579375;\r
+ Mon, 03 Sep 2012 08:36:19 -0700 (PDT)\r
+Received: by 10.14.4.201 with SMTP id 49mr22331541eej.0.1346686579203;\r
+ Mon, 03 Sep 2012 08:36:19 -0700 (PDT)\r
+Received: from mpn-glaptop ([2620:0:105f:5:f2de:f1ff:fe35:1a72])\r
+ by mx.google.com with ESMTPS id e7sm37527584eep.2.2012.09.03.08.36.17\r
+ (version=TLSv1/SSLv3 cipher=OTHER);\r
+ Mon, 03 Sep 2012 08:36:18 -0700 (PDT)\r
+Sender: Michal Nazarewicz <mpn@google.com>\r
+From: Michal Nazarewicz <mina86@mina86.com>\r
+To: Tomi Ollila <tomi.ollila@iki.fi>, notmuch@notmuchmail.org\r
+Subject: Re: [PATCH V2 1/2] devel: add release-checks.sh\r
+In-Reply-To: <m2sjazdug1.fsf@guru.guru-group.fi>\r
+Organization: http://mina86.com/\r
+References: <1346491928-2356-1-git-send-email-tomi.ollila@iki.fi>\r
+ <xa1t7gsbwbxe.fsf@mina86.com> <m2sjazdug1.fsf@guru.guru-group.fi>\r
+User-Agent: Notmuch/0.14+2~g416b120 (http://notmuchmail.org) Emacs/24.2.50.1\r
+ (x86_64-unknown-linux-gnu)\r
+X-Face: PbkBB1w#)bOqd`iCe"Ds{e+!C7`pkC9a|f)Qo^BMQvy\q5x3?vDQJeN(DS?|-^$uMti[3D*#^_Ts"pU$jBQLq~Ud6iNwAw_r_o_4]|JO?]}P_}Nc&"p#D(ZgUb4uCNPe7~a[DbPG0T~!&c.y$Ur,=N4RT>]dNpd; KFrfMCylc}gc??'U2j,!8%xdD\r
+Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAAJFBMVEWbfGlUPDDHgE57V0jUupKjgIObY0PLrom9mH4dFRK4gmjPs41MxjOgAAACQElEQVQ4jW3TMWvbQBQHcBk1xE6WyALX1069oZBMlq+ouUwpEQQ6uRjttkWP4CmBgGM0BQLBdPFZYPsyFUo6uEtKDQ7oy/U96XR2Ux8ehH/89Z6enqxBcS7Lg81jmSuujrfCZcLI/TYYvbGj+jbgFpHJ/bqQAUISj8iLyu4LuFHJTosxsucO4jSDNE0Hq3hwK/ceQ5sx97b8LcUDsILfk+ovHkOIsMbBfg43VuQ5Ln9YAGCkUdKJoXR9EclFBhixy3EGVz1K6eEkhxCAkeMMnqoAhAKwhoUJkDrCqvbecaYINlFKSRS1i12VKH1XpUd4qxL876EkMcDvHj3s5RBajHHMlA5iK32e0C7VgG0RlzFPvoYHZLRmAC0BmNcBruhkE0KsMsbEc62ZwUJDxWUdMsMhVqovoT96i/DnX/ASvz/6hbCabELLk/6FF/8PNpPCGqcZTGFcBhhAaZZDbQPaAB3+KrWWy2XgbYDNIinkdWAFcCpraDE/knwe5DBqGmgzESl1p2E4MWAz0VUPgYYzmfWb9yS4vCvgsxJriNTHoIBz5YteBvg+VGISQWUqhMiByPIPpygeDBE6elD973xWwKkEiHZAHKjhuPsFnBuArrzxtakRcISv+XMIPl4aGBUJm8Emk7qBYU8IlgNEIpiJhk/No24jHwkKTFHDWfPniR4iw5vJaw2nzSjfq2zffcE/GDjRC2dn0J0XwPAbDL84TvaFCJEU4Oml9pRyEUhR3Cl2t01AoEjRbs0sYugp14/4X5n4pU4EHHnMAAAAAElFTkSuQmCC\r
+X-PGP: 50751FF4\r
+X-PGP-FP: AC1F 5F5C D418 88F8 CC84 5858 2060 4012 5075 1FF4\r
+Date: Mon, 03 Sep 2012 17:36:11 +0200\r
+Message-ID: <xa1tsjazunis.fsf@mina86.com>\r
+MIME-Version: 1.0\r
+Content-Type: multipart/mixed; boundary="=-=-="\r
+X-Gm-Message-State: ALoCoQn4zRJWSWCPibn/7y8u9mjZHKmBtE7FiZi99rDmU9HHFU12qNF5YBA8fGf8WSsv7q5VTtkwRti4mZIFG29VyjkiYZFgYT5GYRObUttOSgKa5Ch4I9YkRhBjvpZlVJveD1GDyRJlLMi9pw26zYChzlPmsxDJco/P638KHsiARjqKJ9DU3dK6VuRj/Jjv4wDoiDWWPozyL08vwgquAvUpVs1GUPQ2lg==\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+ <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Mon, 03 Sep 2012 15:36:23 -0000\r
+\r
+--=-=-=\r
+Content-Type: text/plain; charset=utf-8\r
+Content-Transfer-Encoding: quoted-printable\r
+\r
+>> Tomi Ollila <tomi.ollila@iki.fi> writes:\r
+>>> +for f in ./version debian/changelog NEWS "$PV_FILE"\r
+>>> +do\r
+>>> + test -f $f || { append_emsg "File '$f' is missing"; continue; }\r
+>>> + test -r $f || { append_emsg "File '$f' is unreadable"; continue; }\r
+>>> + test -s $f || append_emsg "File '$f' is empty"\r
+\r
+> On Mon, Sep 03 2012, Michal Nazarewicz <mina86@mina86.com> wrote:\r
+>> if ! [ -f "$f" ]; then\r
+>> append_emsg "File '$f' is missing"\r
+>> elif ! [ -r "$f" ]; then\r
+>> append_emsg "File '$f' is unreadable"\r
+>> elif ! [ -s "$f" ]; then\r
+>> append_emsg "File '$f' is empty"\r
+>> fi\r
+\r
+Tomi Ollila <tomi.ollila@iki.fi> writes:\r
+> IMHO this short-circuited or (||) version works well due to this\r
+> negation handling. The $f's should have been quoted ("$f"), though\r
+> (in case some of the items contain whitespace the construct=20\r
+> would't fail). I am open to other opinions, though.\r
+\r
+I personally don't like =E2=80=9C||=E2=80=9D or =E2=80=9C&&=E2=80=9D as a r=\r
+eplacement for if as I find\r
+it less readable. Especially here as you use braces and =E2=80=9Ccontinue=\r
+=E2=80=9D.\r
+\r
+>> if [ x"$deb_notmuch" =3D xnotmuch ]\r
+>>\r
+>> And so in the rest of the conditions below.\r
+>\r
+> builtin bash '[' seems to be robust in these cases(*) ... but I'm now bre=\r
+aking\r
+> my own rule about not using most portable expressions; maybe it is just\r
+> ugliness of the format in so many places. I'm not changing unless there is\r
+> more desire for it :)\r
+\r
+It's not even bash. Newest POSIX requires that behaviour, but I always\r
+use x anyway myself, hence the comment.\r
+\r
+>>> + set x $*\r
+\r
+>> Uh? Did you mean =E2=80=9Cset -- $*=E2=80=9D?\r
+\r
+> Nope, set in some shells don't know '--' (now I'm following the most\r
+> portable code snippet principle) so with 'x' and just referencing\r
+> positional parameters with number one larger than with '--' does the\r
+> trick.\r
+\r
+Interesting. I must start doing that. :)\r
+\r
+>>> + manthdata `sed -n '/^[.]TH NOTMUCH/ { y/"/ /; p; }' "$mp"`\r
+\r
+>> Alternatively =E2=80=9C\.=E2=80=9D instead of =E2=80=9C[.]=E2=80=9D.\r
+\r
+> I prefer [.] as a more robust alternative -- backslashes may get=20\r
+> "expanded away" in some cases; using [.] just frees me from checking\r
+> whether that may happen.\r
+\r
+I guess it's a valid point.\r
+\r
+> Thanks for the review, I'll probably send new version tomorrow...\r
+\r
+Sure thing.\r
+\r
+--=20\r
+Best regards, _ _\r
+.o. | Liege of Serenely Enlightened Majesty of o' \,=3D./ `o\r
+..o | Computer Science, Micha=C5=82 =E2=80=9Cmina86=E2=80=9D Nazarewicz =\r
+ (o o)\r
+ooo +----<email/xmpp: mpn@google.com>--------------ooO--(_)--Ooo--\r
+--=-=-=\r
+Content-Type: multipart/signed; boundary="==-=-=";\r
+ micalg=pgp-sha1; protocol="application/pgp-signature"\r
+\r
+--==-=-=\r
+Content-Type: text/plain\r
+\r
+\r
+--==-=-=\r
+Content-Type: application/pgp-signature\r
+\r
+-----BEGIN PGP SIGNATURE-----\r
+Version: GnuPG v1.4.10 (GNU/Linux)\r
+\r
+iQIcBAEBAgAGBQJQRM5rAAoJECBgQBJQdR/0G5sP/33VxTcHYq9fevuWAuVy7CKk\r
+RYaBegyKYo76qtdC8UV9RRl2vRAUn5NVeyvrD68O1y/UX2vfQ7DT7Oh1FkkOccrt\r
+s9ZUBWpGmuIGlJeO9uIs33ZjQGFLrk9v1c1lTC6wMHY5gkyVa/WRyDXT2jTKwfwH\r
+BMGHw0yafqUlFYVsI+y0rl9bP4EN9s37mtMzmqval4JGE6KSBeJMERG83Zq3MmxA\r
+YiB/kTViBVbf/j+t1asV+8Y82spE4KH+ErXuguyzKCEX+72hZxBaqhzgxfU2+MOe\r
+XZIGa30uFAtL9i1yy6QwWqSu4ajsxouHRAEuEmGPAAlncj7mdtNpwLjjTP9GmgwX\r
+4vSnicgRFUEscNENvhihuoJf/NE6nmtZ0D2l0+8FTCxVkVF+nN4cDqEHrwvxITpG\r
+vYGp1Kt6ga4wdzAezJqhph7+tzKX143RPK2NaB20184ZMjo7xd6sGfT7OH7Nprl+\r
+zFaJfr0ku9HD42Jvl6sPLCET2MWZYelQJjw+UDysu2pS9fImq2Z0VmJ/g9jV8WM6\r
+oqXKZ5cBXu00qcISGBbQl77vjEbQwGaB6LlwQOfEn+xGvE9w8UH1KU0dAtDA79Hn\r
+AM0eL28BqC5HrokWpkUUogOGSDEQGblzZjMPEratuTPKKi9ZZ0RqWO6htWRnJLxd\r
+a85JSyFasyvIn1jHMzfb\r
+=bon6\r
+-----END PGP SIGNATURE-----\r
+--==-=-=--\r
+\r
+--=-=-=--\r
projects / notmuch-archives.git / commitdiff