From owner-svn-ports-all@FreeBSD.ORG Thu Apr 9 16:51:10 2015 Return-Path: Delivered-To: svn-ports-all@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 3F7B94F3 for ; Thu, 9 Apr 2015 16:51:10 +0000 (UTC) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) (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 1A49461F for ; Thu, 9 Apr 2015 16:51:10 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.9/8.14.9) with ESMTP id t39Gp9Fu022329 for ; Thu, 9 Apr 2015 16:51:09 GMT (envelope-from bdrewery@freefall.freebsd.org) Received: (from bdrewery@localhost) by freefall.freebsd.org (8.14.9/8.14.9/Submit) id t39Gp9Ft022328 for svn-ports-all@freebsd.org; Thu, 9 Apr 2015 16:51:09 GMT (envelope-from bdrewery) Received: (qmail 50915 invoked from network); 9 Apr 2015 11:51:03 -0500 Received: from unknown (HELO ?10.10.1.139?) (freebsd@shatow.net@10.10.1.139) by sweb.xzibition.com with ESMTPA; 9 Apr 2015 11:51:03 -0500 Message-ID: <5526ADFD.2020500@FreeBSD.org> Date: Thu, 09 Apr 2015 11:51:09 -0500 From: Bryan Drewery Organization: FreeBSD User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Dmitry Marakasov Subject: Re: svn commit: r383191 - head/Mk References: <201504040535.t345ZJ9M028396@svn.freebsd.org> <20150408194202.GA45809@hades.panopticon> <5525A9C1.5010003@FreeBSD.org> <20150408232144.GX21982@hades.panopticon> <5525D48E.5090305@FreeBSD.org> <20150409110221.GB45809@hades.panopticon> <55269785.7030307@FreeBSD.org> <20150409153856.GA31951@FreeBSD.org> <55269FE4.2030208@FreeBSD.org> <20150409164926.GC45809@hades.panopticon> In-Reply-To: <20150409164926.GC45809@hades.panopticon> OpenPGP: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="TCR9j4xv8xS7rru1Su2fng3Pv2eclace9" Cc: svn-ports-head@freebsd.org, Alexey Dokuchaev , svn-ports-all@freebsd.org, ports-committers@freebsd.org X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 09 Apr 2015 16:51:10 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --TCR9j4xv8xS7rru1Su2fng3Pv2eclace9 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 4/9/2015 11:49 AM, Dmitry Marakasov wrote: > * Bryan Drewery (bdrewery@FreeBSD.org) wrote: >=20 >>>> *That* made sense. Frankly I think all dev warnings should be disabl= ed >>>> in that case though. Setting up ports like this may work but it is n= ot a >>>> good way to test a port before committing or submitting it. Many por= ts >>>> and their framework files reference "CURDIR/../..". In r327743 I >>>> modified www/linux-seamonkey to use PORTSDIR here and I believe I br= oke >>>> the workflow for the maintainers testing. The ports tree really need= s to >>>> be tested as a whole. This is why portshaker exists, so you can brin= g in >>>> your partial tree into a full tree. Unionfs in theory is for this to= o. >>> >>> Bryan, I'm on Dmitry's side here. Building some "foo" port from outs= ide >>> /usr/ports (or a properly checked out tree) is very handy feature, an= d it >>> had worked for many years; and while perhaps never officially adverti= sed, >>> silently breaking it is a huge POLA violation for power users. I wou= ld >>> really appreciate if we can have this behavior back. >> >> I am in favor of fixing it. I just think all the tests should be >> disabled. It won't make sense to have some tests enabled and some not.= >> It's no good if the port passes in your out-of-tree build for 1/2 the >> tests and then you just straight commit and find that it fails all of >> the other tests. >=20 > I don't see the point. Why disable all checks while they can detect > problems early (while the port is still out-of-tree), instead of > making developer experience inconsistent behavior without explanation? > In either case, we need something like >=20 > WARNING+=3D"the port doesn't reside in the ports tree directory structu= re, not all sanity checks could be performed" >=20 > but I'd really prefer only tests which are ctually affected by this to = be > disabled. >=20 This is what my proposed patch does! --=20 Regards, Bryan Drewery --TCR9j4xv8xS7rru1Su2fng3Pv2eclace9 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 iQEcBAEBAgAGBQJVJq39AAoJEDXXcbtuRpfP3f0H/RDNh0t6m/wJlNEBljk264ll URrLuIknNmpxN6TrAV85YCPgvND33yco2ccBnvKDfLQbkFX4zjfvusIrPW6vf41K Sa16h+Z23Vd7N5B2rEiwFg2RWNDv9s9CNZNZofNrrt/Xh9b6jMNv0ipPOY6tCyjZ XW5B/9488y6Y9tfUNIi4MuDemPgSKBArNN1OBkW1VUQn7MNNaB2RNE+v/aVKfWo5 TavnrAfIQy3Un3h/tsh2DppZQq0egtYs4Duap3peoyiTA7sn3m87in965pJcTlUh +xJkYuYLdlzxaomO2EkCjzmALZ+CVGjMe4SjgkDylU7e0vbOPMVpWuYwbMxLT/A= =UFIv -----END PGP SIGNATURE----- --TCR9j4xv8xS7rru1Su2fng3Pv2eclace9--