From owner-svn-ports-all@freebsd.org Thu Aug 29 13:30:49 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 4F951D60B0; Thu, 29 Aug 2019 13:30:49 +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 46K3Qc5DLwz4fp4; Thu, 29 Aug 2019 13:30:48 +0000 (UTC) (envelope-from cy.schubert@cschubert.com) Received: from spqr.komquats.com ([70.67.125.17]) by shaw.ca with ESMTPA id 3KVhiJ81kSrVc3KViix3Dp; Thu, 29 Aug 2019 07:30:46 -0600 X-Authority-Analysis: v=2.3 cv=L5ZjvNb8 c=1 sm=1 tr=0 a=VFtTW3WuZNDh6VkGe7fA3g==:117 a=VFtTW3WuZNDh6VkGe7fA3g==:17 a=jpOVt7BSZ2e4Z31A5e1TngXxSK0=:19 a=IkcTkHD0fZMA:10 a=xqWC_Br6kY4A:10 a=FmdZ9Uzk2mMA:10 a=6I5d2MoRAAAA:8 a=mi56gJdQAAAA:8 a=mDV3o1hIAAAA:8 a=YxBL1-UpAAAA:8 a=bAeICEHnqln1DCMdZusA:9 a=QEXdDO2ut3YA:10 a=IjZwj45LgO3ly-622nXo:22 a=m6W23KLcDyq3lIHOBnQi:22 a=_FVE-zBwftR9WsbkzFJk:22 a=Ia-lj3WSrqcvXOmTRaiG:22 a=pHzHmUro8NiASowvMSCR:22 a=Ew2E2A-JSTLzCXPT_086:22 Received: from [IPv6:2605:8d80:403:7c6b:45f7:b036:707d:4d1f] (unknown [72.143.237.81]) by spqr.komquats.com (Postfix) with ESMTPSA id 2F35457A; Thu, 29 Aug 2019 06:30:44 -0700 (PDT) Date: Thu, 29 Aug 2019 06:21:48 -0700 In-Reply-To: <20190829130900.7qqm5rv3z3zgnmdw@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> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: svn commit: r510145 - head/sysutils/screen To: Baptiste Daroussin CC: Kirill Ponomarev , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org From: Cy Schubert Message-ID: <6624DB64-13E5-494D-AC1D-16D002AD5D2E@cschubert.com> X-CMAE-Envelope: MS4wfJGoftCbrRmOh4lMoJpS/PawwCm3NQv37KdNk4huKBx0hDwOo3vQZtqprgJhkqCoxIZ/rht6S5k5kuzZ9w3Rrs2gl8EZwZhs/mZjhm1YEvE45MFSRexp 9qHNS4KO6WHit1BK6NtPsd6wrrLqhD2MNbt9HH6bJhlcOJ6f8eDzcH0PEFH24RaxKurL3E9LWlGpXCtYX3LBbjQeXXq/M5P/xfQMtWFVo5vouZqItciSVG92 LVH1MqZFHJ7gWdMw3QZXpqPwLgkTXV1i+CXaf5BSRVW3koLKF+BzTAV1eTXzZLleWSXj6W9Xsb+uvvLEpLhbru6eki8zKhlpTvzrN33qGhc= X-Rspamd-Queue-Id: 46K3Qc5DLwz4fp4 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-6.92 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-0.999,0]; NEURAL_HAM_SHORT(-0.92)[-0.924,0]; REPLY(-4.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,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 13:30:49 -0000 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=2Er7gpumnvtbeguuoz@ivaldir=2Enet>, 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=2Ex7T6cVFc033808@repo=2Efreebsd=2Eorg>, Ki= rill >Ponomar=3D >> > ev=3D20 >> > > wri >> > > tes: >> > > > Author: krion >> > > > Date: Thu Aug 29 06:38:31 2019 >> > > > New Revision: 510145 >> > > > URL: https://svnweb=2Efreebsd=2Eorg/changeset/ports/510145 >> > > > >> > > > Log: >> > > > Remove building man page by default=2E It's currently broken on >HEAD >> > > > and requires to build textproc/groff which requires >devel/llvm80 >> > > > which is an overkill for using screen=2E >> > > > >> > > > Modified: >> > > > head/sysutils/screen/Makefile >> > > > >> > > > Modified: head/sysutils/screen/Makefile >> > > > >=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D= 3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D >> > >=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D3D=3D= 3D=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=3D= 3D=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=2E6=2E2 >> > > > -PORTREVISION=3D3D 1 >> > > > +PORTREVISION=3D3D 2 >> > > > CATEGORIES=3D3D sysutils >> > > > MASTER_SITES=3D3D GNU \ >> > > > ftp://ftp=2Egnu=2Eorg/gnu/screen/ \ >> > > > @@ -20,7 +20,7 @@ COMMENT=3D3D Multi-screen window manager >> > > > LICENSE=3D3D GPLv3 >> > > > =3D20 >> > > > OPTIONS_DEFINE=3D3D INFO MAN NETHACK XTERM_256 SHOWENC 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=2E 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=2E and not add groff as a runtime >dependency >> > anyway=2E >>=20 >> I've had this same issue with krb5=2E Pkg end users expect man pages=2E >>=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=2E >the other >> > point is in the future screen(1) might become mandoc compatible=2E >and you wi=3D >> > ll >> > still grab in the groff(1) dependency=2E >>=20 >> I'm willing to live with this but it is still disrespectful not to >involve=20 >> the maintainer=2E >>=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=2E ;) > >Best regards, >Bapt If and when I have the inclination to look at this, I will=2E In the mean = time patches are welcome=2E --=20 Pardon the typos and autocorrect, small keyboard in use=2E Cheers, Cy Schubert FreeBSD UNIX: Web: http://www=2EFreeBSD=2Eorg The need of the many outweighs the greed of the few=2E