From owner-freebsd-ports@freebsd.org Fri May 5 10:46:19 2017 Return-Path: Delivered-To: freebsd-ports@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 321B4D5F423 for ; Fri, 5 May 2017 10:46:19 +0000 (UTC) (envelope-from mat@FreeBSD.org) Received: from prod2.absolight.net (prod2.absolight.net [79.143.243.136]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "plouf.absolight.net", Issuer "CAcert Class 3 Root" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id A3B44D44; Fri, 5 May 2017 10:46:18 +0000 (UTC) (envelope-from mat@FreeBSD.org) Received: from prod2.absolight.net (localhost [127.0.0.1]) by prod2.absolight.net (Postfix) with ESMTP id EF609BDC9F; Fri, 5 May 2017 12:46:16 +0200 (CEST) Received: from ogg.in.absolight.net (ogg.in.absolight.net [79.143.241.239]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client did not present a certificate) by prod2.absolight.net (Postfix) with ESMTPSA id 758A5BDC91; Fri, 5 May 2017 12:46:16 +0200 (CEST) Subject: Re: diff and submissions To: Baptiste Daroussin Cc: Franco Fichtner , FreeBSD Mailing List , Aristedes Maniatis References: <750748ab-0e0e-cbb2-0b3b-d236829ee958@ish.com.au> <063dd680-2600-b440-b06b-04323eed6e58@FreeBSD.org> <22B42BD4-B348-49F6-8CF6-82B7B00E6B84@lastsummer.de> <05fd8f53-30b8-88fb-4d70-68637b2261b1@FreeBSD.org> <20170505102006.5lzf6kt7rotzvzeg@ivaldir.net> <20170505103745.sa4oeq573szssqn4@ivaldir.net> From: Mathieu Arnold Organization: Absolight / The FreeBSD Foundation Message-ID: Date: Fri, 5 May 2017 12:46:14 +0200 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 MIME-Version: 1.0 In-Reply-To: <20170505103745.sa4oeq573szssqn4@ivaldir.net> Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="8kr5Pq1gWhURR6BNb9HN1cpU7MoLFrDrQ" X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 05 May 2017 10:46:19 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --8kr5Pq1gWhURR6BNb9HN1cpU7MoLFrDrQ Content-Type: multipart/mixed; boundary="AnwlDNLH8Vu38eMdXQar6VuDTxcg95kUo"; protected-headers="v1" From: Mathieu Arnold To: Baptiste Daroussin Cc: Franco Fichtner , FreeBSD Mailing List , Aristedes Maniatis Message-ID: Subject: Re: diff and submissions References: <750748ab-0e0e-cbb2-0b3b-d236829ee958@ish.com.au> <063dd680-2600-b440-b06b-04323eed6e58@FreeBSD.org> <22B42BD4-B348-49F6-8CF6-82B7B00E6B84@lastsummer.de> <05fd8f53-30b8-88fb-4d70-68637b2261b1@FreeBSD.org> <20170505102006.5lzf6kt7rotzvzeg@ivaldir.net> <20170505103745.sa4oeq573szssqn4@ivaldir.net> In-Reply-To: <20170505103745.sa4oeq573szssqn4@ivaldir.net> --AnwlDNLH8Vu38eMdXQar6VuDTxcg95kUo Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Content-Language: fr Le 05/05/2017 =C3=A0 12:37, Baptiste Daroussin a =C3=A9crit : > On Fri, May 05, 2017 at 12:27:13PM +0200, Mathieu Arnold wrote: >> Le 05/05/2017 =C3=A0 12:20, Baptiste Daroussin a =C3=A9crit : >>> I made a script which is a few lines of shell that converts pull >>> requests to >>> phabricator reviews automatically. >>> >>> I have handed it to some of the phabricator admins, I hope it will be= progress >>> soon. >> Please, do not do that. >> >> Phabricator is for code review, not bug reports. It all ends up in ou= r >> repository, but Phabricator does not have any maintainer notification >> like Bugzilla has. >> robak@ has a script that does pull-request -> bugzilla, and it has bee= n >> waiting for months for someone with access to do something about it. >> > Pull request are for code submissions and phabricator is the equivalent= for that It is great to review code, yes, but it will not notify maintainers that they have a patch for one of their ports, it adds more work for the ports committers that will have to go and review the code, and maybe notice it came from github, and then maybe notice that someone need to notify the maintainer of the port in question, so that they can accept, or not, the patch. It is mostly going to end up having code review that are not handled by anyone because they are not code review, they came from github, and the submitter on github is not aware that there has been some review and they need to update their code, or that they need to submit it to bugzilla afterwards so that it gets reviewed by the maintainer. --=20 Mathieu Arnold --AnwlDNLH8Vu38eMdXQar6VuDTxcg95kUo-- --8kr5Pq1gWhURR6BNb9HN1cpU7MoLFrDrQ 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 iQJ8BAEBCgBmBQJZDFf2XxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQzQUI2OTc4OUQyRUQxMjEwNjQ0MEJBNUIz QTQ1MTZGMzUxODNDRTQ4AAoJEDpFFvNRg85IK+UP/RFig/1enFtpjUDboeQwUC+K ugzEirksKlOt/5bSBqs4uInxfW40lb+bVvB/a9KD1LCzHNqZQhhs+4Rdig/zM2Dx Cy/4Ha+RaWVhnoQ2K5fRUbYOpj0JlYHfu9x4stG/VbYtdDQRafUVcqUXQLviSyti al09JTN3tH7jILBpU1aYpJiJ+k5JwsHZQQGcOOw6ytCNZzDQsaoNsydXC4FtEUqt ABCSf4f1owUoI879CSh83CAP/OxEKVeBYjzoYe6dhvihjEa7HbcV9dstdfwycKMW xMJZwr3XUWqUfjZvnmDNw0Pj+veaQD7yMRawxv9sqqUCE0WZYSHKEZwUb4B8U2cy ON5Mjc/bC+CTeI87r1r0/Uh1yuRwg8aOTYDk/4CyNSfUr+CZOTgHCoK92bSQ0ny3 lTarQ8VedO+O9QCrdC9REFnuTs51FG9vu6gVNZB8+renD9doju8R4rh5TW0DXZnI rQVIFsoM/BVhjKQtINd9GkQfSHfw6v7U+0lQxFGxRG4SXOuA4XQ0EwoAGDlc3tvv ni+FdaAzPr76m8woVDUB7YgPg1x/WdVn9HXuCaYReMpi1QWImmF2LuDC/KWQX5Or fjuzwxKFXor20id8PedLsrq5x7UmIcPlj4BrKVZhihmpE3rFHB5NLGX1d0dyqf/r p/4xCfUdR9v1Gc5q0XoX =c935 -----END PGP SIGNATURE----- --8kr5Pq1gWhURR6BNb9HN1cpU7MoLFrDrQ--