From owner-svn-ports-all@freebsd.org Thu Aug 29 14:04:45 2019 Return-Path: Delivered-To: svn-ports-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id A3A48D6D68; Thu, 29 Aug 2019 14:04:45 +0000 (UTC) (envelope-from cy.schubert@cschubert.com) Received: from smtp-out-so.shaw.ca (smtp-out-so.shaw.ca [64.59.136.139]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "Client", Issuer "CA" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 46K49n0y6Qz3DJM; Thu, 29 Aug 2019 14:04:44 +0000 (UTC) (envelope-from cy.schubert@cschubert.com) Received: from spqr.komquats.com ([70.67.125.17]) by shaw.ca with ESMTPA id 3L2XiJNFXSrVc3L2Yix9JD; Thu, 29 Aug 2019 08:04:43 -0600 X-Authority-Analysis: v=2.3 cv=L5ZjvNb8 c=1 sm=1 tr=0 a=VFtTW3WuZNDh6VkGe7fA3g==:117 a=VFtTW3WuZNDh6VkGe7fA3g==:17 a=kj9zAlcOel0A:10 a=FmdZ9Uzk2mMA:10 a=mi56gJdQAAAA:8 a=6I5d2MoRAAAA:8 a=mDV3o1hIAAAA:8 a=YxBL1-UpAAAA:8 a=grIyg7WZeuLLqAYMdNYA:9 a=CjuIK1q_8ugA:10 a=m6W23KLcDyq3lIHOBnQi:22 a=IjZwj45LgO3ly-622nXo:22 a=_FVE-zBwftR9WsbkzFJk:22 a=Ia-lj3WSrqcvXOmTRaiG:22 Received: from slippy.cwsent.com (slippy8 [10.2.2.6]) by spqr.komquats.com (Postfix) with ESMTPS id 05ABE5F1; Thu, 29 Aug 2019 07:04:36 -0700 (PDT) Received: from slippy.cwsent.com (localhost [127.0.0.1]) by slippy.cwsent.com (8.15.2/8.15.2) with ESMTP id x7TE4af1006595; Thu, 29 Aug 2019 07:04:36 -0700 (PDT) (envelope-from Cy.Schubert@cschubert.com) Received: from slippy (cy@localhost) by slippy.cwsent.com (8.15.2/8.15.2/Submit) with ESMTP id x7TE4aqc006592; Thu, 29 Aug 2019 07:04:36 -0700 (PDT) (envelope-from Cy.Schubert@cschubert.com) Message-Id: <201908291404.x7TE4aqc006592@slippy.cwsent.com> X-Authentication-Warning: slippy.cwsent.com: cy owned process doing -bs X-Mailer: exmh version 2.9.0 11/07/2018 with nmh-1.7.1 Reply-to: Cy Schubert From: Cy Schubert X-os: FreeBSD X-Sender: cy@cwsent.com X-URL: http://www.cschubert.com/ To: Baptiste Daroussin cc: Cy Schubert , Kirill Ponomarev , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r510145 - head/sysutils/screen In-reply-to: <20190829133713.5l3jk67el47gdmfb@ivaldir.net> References: <201908290638.x7T6cVFc033808@repo.freebsd.org> <201908291230.x7TCUgKI027625@slippy.cwsent.com> <20190829124225.r7gpumnvtbeguuoz@ivaldir.net> <201908291254.x7TCsFCI051889@slippy.cwsent.com> <20190829130900.7qqm5rv3z3zgnmdw@ivaldir.net> <6624DB64-13E5-494D-AC1D-16D002AD5D2E@cschubert.com> <20190829133713.5l3jk67el47gdmfb@ivaldir.net> Comments: In-reply-to Baptiste Daroussin message dated "Thu, 29 Aug 2019 15:37:13 +0200." Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Thu, 29 Aug 2019 07:04:36 -0700 X-CMAE-Envelope: MS4wfO2pO6Xi1AMx3kZapcqxROT5tCk+zwCNjVLELi0LHrQCb0MaRM8b4sgYVOsp7GYeyF/yJrjmmA1XL6iCZ8wyn50GalulSlr2MwubM0+0O0XFgC4bnL4S ueEMA0MSIOBUHCl9+tDiavjrfzmanooW60IFLL7VWCPap8pCELxeXA0u98osdLZ4wtdXPP8r3HpO6BEC+sc+8SrdtNQF3AgK5yWniVsgiNuv3qiBWXeIjUZE 8DieDYvmwFlX7jbf+Dwat67OTnXOENGjneOkM1J+52XogXHE9RY0lJbU9l/F1FA3C7+xaXPDuanQeajfVD46KyUdnTMvBKqTNz6jsnI/DTk= X-Rspamd-Queue-Id: 46K49n0y6Qz3DJM X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-6.94 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-0.999,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; REPLY(-4.00)[]; NEURAL_HAM_SHORT(-0.94)[-0.942,0] X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.29 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, 29 Aug 2019 14:04:45 -0000 In message <20190829133713.5l3jk67el47gdmfb@ivaldir.net>, Baptiste Daroussin wr ites: > > > --kv3xxe43po3ksi5m > Content-Type: text/plain; charset=us-ascii > Content-Disposition: inline > Content-Transfer-Encoding: quoted-printable > > On Thu, Aug 29, 2019 at 06:21:48AM -0700, Cy Schubert wrote: > > On August 29, 2019 6:09:00 AM PDT, Baptiste Daroussin = > wrote: > > >On Thu, Aug 29, 2019 at 05:54:15AM -0700, Cy Schubert wrote: > > >> In message <20190829124225.r7gpumnvtbeguuoz@ivaldir.net>, Baptiste=20 > > >> Daroussin wr > > >> ites: > > >> >=20 > > >> > > > >> > --vmfceac25l47d6el > > >> > Content-Type: text/plain; charset=3Dus-ascii > > >> > Content-Disposition: inline > > >> > Content-Transfer-Encoding: quoted-printable > > >> > > > >> > On Thu, Aug 29, 2019 at 05:30:42AM -0700, Cy Schubert wrote: > > >> > > In message <201908290638.x7T6cVFc033808@repo.freebsd.org>, Kirill > > >Ponomar=3D > > >> > ev=3D20 > > >> > > wri > > >> > > tes: > > >> > > > Author: krion > > >> > > > Date: Thu Aug 29 06:38:31 2019 > > >> > > > New Revision: 510145 > > >> > > > URL: https://svnweb.freebsd.org/changeset/ports/510145 > > >> > > > > > >> > > > Log: > > >> > > > Remove building man page by default. It's currently broken on > > >HEAD > > >> > > > and requires to build textproc/groff which requires > > >devel/llvm80 > > >> > > > which is an overkill for using screen. > > >> > > > > > >> > > > Modified: > > >> > > > head/sysutils/screen/Makefile > > >> > > > > > >> > > > Modified: head/sysutils/screen/Makefile > > >> > > > > > >=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D= > =3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D > > >> > > > >=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D= > =3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D > > >> > > > >=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D= > =3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D > > >> > =3D3D=3D3D=3D3D=3D3D > > >> > > > =3D3D > > >> > > > --- head/sysutils/screen/Makefile Thu Aug 29 06:37:17 > > >2019 (r51014 > > >> > > > 4) > > >> > > > +++ head/sysutils/screen/Makefile Thu Aug 29 06:38:31 > > >2019 (r51014 > > >> > > > 5) > > >> > > > @@ -3,7 +3,7 @@ > > >> > > > =3D20 > > >> > > > PORTNAME=3D3D screen > > >> > > > PORTVERSION=3D3D 4.6.2 > > >> > > > -PORTREVISION=3D3D 1 > > >> > > > +PORTREVISION=3D3D 2 > > >> > > > CATEGORIES=3D3D sysutils > > >> > > > MASTER_SITES=3D3D GNU \ > > >> > > > ftp://ftp.gnu.org/gnu/screen/ \ > > >> > > > @@ -20,7 +20,7 @@ COMMENT=3D3D Multi-screen window manager > > >> > > > LICENSE=3D3D GPLv3 > > >> > > > =3D20 > > >> > > > OPTIONS_DEFINE=3D3D INFO MAN NETHACK XTERM_256 SHOW > ENC SYST > > >> > EM_SCREE > > >> > > > NRC > > >> > > > -OPTIONS_DEFAULT=3D3D INFO MAN NETHACK XTERM_256 SOCKETS > > >SYSTEM_SCREE > > >> > NRC \ > > >> > > > +OPTIONS_DEFAULT=3D3D INFO NETHACK XTERM_256 SOCKETS > > >SYSTEM_SCREENRC=20 > > >> > \ > > >> > > > NCURSES_DEFAULT > > >> > > > OPTIONS_SINGLE=3D3D IPC NCURSES > > >> > > > OPTIONS_SINGLE_IPC=3D3D SOCKETS NAMED_PIPES > > >> > > > > > >> > >=3D20 > > >> > > Should you not have obtained maintainer approval for this first? > > >> > >=3D20 > > >> > > And now the package will not include man pages. Can I assign > > >those PRs to=3D > > >> > =3D20 > > >> > > you? > > >> > >=3D20 > > >> > > > >> > I do think the best approach to this is to drop the MAN page > > >option, it mak=3D > > >> > es > > >> > little sense not to install it. and not add groff as a runtime > > >dependency > > >> > anyway. > > >>=20 > > >> I've had this same issue with krb5. Pkg end users expect man pages. > > >>=20 > > >> > > > >> > The reason why man(1) is telling the users to install groff(1) is > > >exactly f=3D > > >> > or > > >> > that, the users do decide if he wants to read that manpage on his > > >system an=3D > > >> > d do > > >> > install groff, or he reads it online and do not install it at all. > > >the other > > >> > point is in the future screen(1) might become mandoc compatible. > > >and you wi=3D > > >> > ll > > >> > still grab in the groff(1) dependency. > > >>=20 > > >> I'm willing to live with this but it is still disrespectful not to > > >involve=20 > > >> the maintainer. > > >>=20 > > >> Can I point to this email thread as a precedent if or when I commit > > >to a=20 > > >> port maintained by someone else? > > > > > >For sure the maintainer should have been involved! and also my proposal > > >if > > >different from what has been done here. ;) > > > > > >Best regards, > > >Bapt > >=20 > > If and when I have the inclination to look at this, I will. In the mean t= > ime patches are welcome. > >=20 > https://reviews.freebsd.org/D21463 > > This is what I do propose Go for it. Thanks. -- Cheers, Cy Schubert FreeBSD UNIX: Web: http://www.FreeBSD.org The need of the many outweighs the greed of the few.