From owner-svn-ports-head@FreeBSD.ORG Fri Jan 4 01:37:42 2013 Return-Path: Delivered-To: svn-ports-head@FreeBSD.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 83A87905; Fri, 4 Jan 2013 01:37:42 +0000 (UTC) (envelope-from hrs@FreeBSD.org) Received: from mail.allbsd.org (gatekeeper.allbsd.org [IPv6:2001:2f0:104:e001::32]) by mx1.freebsd.org (Postfix) with ESMTP id 6F2D3239; Fri, 4 Jan 2013 01:37:41 +0000 (UTC) Received: from alph.allbsd.org (p1137-ipbf1505funabasi.chiba.ocn.ne.jp [118.7.212.137]) (authenticated bits=128) by mail.allbsd.org (8.14.5/8.14.5) with ESMTP id r041bOSS081240 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 4 Jan 2013 10:37:34 +0900 (JST) (envelope-from hrs@FreeBSD.org) Received: from localhost (localhost [127.0.0.1]) (authenticated bits=0) by alph.allbsd.org (8.14.5/8.14.5) with ESMTP id r041bLPY096244; Fri, 4 Jan 2013 10:37:24 +0900 (JST) (envelope-from hrs@FreeBSD.org) Date: Fri, 04 Jan 2013 10:30:13 +0900 (JST) Message-Id: <20130104.103013.1716297184205635398.hrs@allbsd.org> To: gjb@FreeBSD.org Subject: Re: svn commit: r309910 - head/textproc/docproj From: Hiroki Sato In-Reply-To: <20130104010502.GB21652@glenbarber.us> References: <201301040042.r040g3cj017454@svn.freebsd.org> <20130104010502.GB21652@glenbarber.us> X-PGPkey-fingerprint: BDB3 443F A5DD B3D0 A530 FFD7 4F2C D3D8 2793 CF2D X-Mailer: Mew version 6.5 on Emacs 23.4 / Mule 6.0 (HANACHIRUSATO) Mime-Version: 1.0 Content-Type: Multipart/Signed; protocol="application/pgp-signature"; micalg=pgp-sha1; boundary="--Security_Multipart(Fri_Jan__4_10_30_13_2013_533)--" Content-Transfer-Encoding: 7bit X-Virus-Scanned: clamav-milter 0.97.4 at gatekeeper.allbsd.org X-Virus-Status: Clean X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.7 (mail.allbsd.org [133.31.130.32]); Fri, 04 Jan 2013 10:37:34 +0900 (JST) X-Spam-Status: No, score=-98.1 required=13.0 tests=CONTENT_TYPE_PRESENT, ONLY1HOPDIRECT,SAMEHELOBY2HOP,USER_IN_WHITELIST autolearn=no version=3.3.2 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on gatekeeper.allbsd.org Cc: svn-ports-head@FreeBSD.org, svn-ports-all@FreeBSD.org, jgh@FreeBSD.org, ports-committers@FreeBSD.org X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 04 Jan 2013 01:37:42 -0000 ----Security_Multipart(Fri_Jan__4_10_30_13_2013_533)-- Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Glen Barber wrote in <20130104010502.GB21652@glenbarber.us>: gj> On Thu, Jan 03, 2013 at 05:02:01PM -0800, Jason Helfman wrote: gj> > > Log: gj> > > Revert previous commit. The more correct solution to this port gj> > > is to convert to the new options framework. gj> > > gj> > [...] gj> > gj> > Not sure why this was reverted. If this is built as a package on pointyhat, gj> > it would've remained WITH_JADETEX=yes per PACKAGE_BUILDING. That is, if I gj> > am understanding the discussion correctly. gj> gj> You are correct. However, the fact that files/JADETEX is not consistent gj> with the behavior if WITH_JADETEX is not set needs to be corrected, too. gj> So the better solution is to update this to the new options framework, gj> which is long overdue. FYI, setting YES by default for release package build, and null and display a warning in "make install" are the intentional behavior because people who blindly install textproc/docproj port tend to complain about "PDF version does not build" or "dependency is too large". This is why simply "make install" does not work. The reason why it is not "no by default" is that it causes inconsistency between pkg_add and make install (actually all of docproj* packages have jadetex dependency). One has to choose either, anyway. $JADETEX knob is a hack added back in the era when there was no OPTIONS, so converting it to OPTIONS would be more user-friendly. However, setting the default value is still a separate topic. "no" is harmful in terms of inconsistency with the pre-built package, and people will complain even if it is "yes" by default. For this reason, we have textproc/docproj-jadetex, textproc/docproj-nojadetex, and textproc/docproj with no default WITH_JADETEX. These are redundant and a bit messy due to historical reasons and they can be reconstructed into the two in a cleaner way, but if we want to change them, it should be done in whole. -- Hiroki ----Security_Multipart(Fri_Jan__4_10_30_13_2013_533)-- Content-Type: application/pgp-signature Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (FreeBSD) iEYEABECAAYFAlDmMKUACgkQTyzT2CeTzy3/qwCfcJ6P0zg6g/m1yqUHhavA96MU +nUAnRg0qfsfymWWBpzmMMjYnWVhywXy =jO2J -----END PGP SIGNATURE----- ----Security_Multipart(Fri_Jan__4_10_30_13_2013_533)----