From owner-svn-ports-head@freebsd.org Sat Jan 14 14:16:26 2017 Return-Path: Delivered-To: svn-ports-head@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 A0B3ACAE19C for ; Sat, 14 Jan 2017 14:16:26 +0000 (UTC) (envelope-from junchoon@dec.sakura.ne.jp) Received: from dec.sakura.ne.jp (dec.sakura.ne.jp [210.188.226.8]) (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 670C111D6; Sat, 14 Jan 2017 14:16:26 +0000 (UTC) (envelope-from junchoon@dec.sakura.ne.jp) Received: from fortune.joker.local (123-48-19-110.dz.commufa.jp [123.48.19.110]) (authenticated bits=0) by dec.sakura.ne.jp (8.15.2/8.15.2/[SAKURA-WEB]/20080708) with ESMTPA id v0EEGHmm052090; Sat, 14 Jan 2017 23:16:17 +0900 (JST) (envelope-from junchoon@dec.sakura.ne.jp) Date: Sat, 14 Jan 2017 23:16:17 +0900 From: Tomoaki AOKI To: svn-ports-head@freebsd.org Cc: "Carlos J. Puga Medina" Subject: Re: Fw: Re: svn commit: r431339 - head/www/chromium Message-Id: <20170114231617.7892e1a41bdecd1ac960c5c0@dec.sakura.ne.jp> In-Reply-To: <1484310802.1425.1.camel@FreeBSD.org> References: <20170113154515.028ab8592c271164e8e2083e@dec.sakura.ne.jp> <1484310802.1425.1.camel@FreeBSD.org> Organization: Junchoon corps X-Mailer: Sylpheed 3.5.1 (GTK+ 2.24.29; amd64-portbld-freebsd11.0) Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-2022-JP Content-Transfer-Encoding: 7bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.23 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: Sat, 14 Jan 2017 14:16:26 -0000 Thanks for looking in. Yours looks better than mine. In most cases, currently-committed one would be fine. But once someone sticking on revision range I noted (with some reason) complains (i.e., sent PR) , your new one should be better committed. Thanks again. On Fri, 13 Jan 2017 13:33:22 +0100 "Carlos J. Puga Medina" wrote: > On Fri, 2017-01-13 at 15:45 +0900, Tomoaki AOKI wrote: > > Hi! > > Resent as missingly sent res to improper ML (freebsd-ports). > > It should be unclear there. Sigh... > > > > > > Begin forwarded message: > > > > Date: Fri, 13 Jan 2017 15:29:06 +0900 > > From: Tomoaki AOKI > > To: freebsd-ports@freebsd.org > > Cc: cpm@FreeBSD.org > > Subject: Re: svn commit: r431339 - head/www/chromium > > > > > > Hi! > > > > Can we assume all -head users update src at least "per month" basis? > > If so, no problem. > > > > If not, as __FreeBSD_version 1200018 came on Dec 7 15:04:22 2016 UTC > > at > > r309676, users sticking on src before this should need > > EXTRA_PATCHES+= ${FILESDIR}/extra-patch-libc++-old. > > > > This case, version check portion for 11 and 12 should be as below. > > > > > > .if ${OSVERSION} < 1200017 > > .if ${OSVERSION} > 1100507 && ${OSVERSION} < 1200000 > > EXTRA_PATCHES+= ${FILESDIR}/extra-patch-libc++-new > > .else > > EXTRA_PATCHES+= ${FILESDIR}/extra-patch-libc++-old > > .endif > > .else > > EXTRA_PATCHES+= ${FILESDIR}/extra-patch-libc++-new > > .endif > > > > > > To simplify the OSVERSION checks: > > .if ${OSVERSION} < 1200017 || (${OSVERSION} >= 1100507 && ${OSVERSION} > < 1200000) > EXTRA_PATCHES+= ${FILESDIR}/extra-patch-libc++-new > .else > EXTRA_PATCHES+= ${FILESDIR}/extra-patch-libc++-old > .endif > > > This could be restored with yours after stable/12 is branched. > > > > Or making ports/UPDATING entry for -head users to update to latest > > base (at least after r309676) first before updating chromium could > > be enough. > > > > Regards. > > > > > > > > Author: cpm > > > Date: Fri Jan 13 03:18:05 2017 > > > New Revision: 431339 > > > URL: https://svnweb.freebsd.org/changeset/ports/431339 > > > > > > Log: > > > 〓 - Wrap the entire block with OPSYS check > > > 〓〓 > > > 〓 Reported by: marino > > > 〓 MFH: 2017Q1 (blanket) > > > > > > Modified: > > > 〓 head/www/chromium/Makefile > > > > > > Modified: head/www/chromium/Makefile > > > =================================================================== > > > =========== > > > --- head/www/chromium/Makefile Fri Jan 13 02:41:25 2017 > > 〓(r431338) > > > > > > +++ head/www/chromium/Makefile Fri Jan 13 03:18:05 > > 2017 (r431339) > > > > > > @@ -187,15 +187,17 @@ GN_ARGS+= > > is_clang=true > > > > > > 〓EXTRA_PATCHES+= ${FILESDIR}/extra-patch-clang > > > 〓.endif > > > 〓 > > > -.if ${OPSYS} == FreeBSD && ${OSVERSION} < 1100508 > > > +.if ${OPSYS} == FreeBSD > > > +.if ${OSVERSION} < 1100508 > > > 〓EXTRA_PATCHES+= ${FILESDIR}/extra-patch-libc++-old > > > 〓.else > > > 〓EXTRA_PATCHES+= ${FILESDIR}/extra-patch-libc++-new > > > 〓.endif > > > 〓 > > > -.if ${OPSYS} == FreeBSD && ${OSVERSION} < 1100000 > > > +.if ${OSVERSION} < 1100000 > > > 〓EXTRA_PATCHES+= ${FILESDIR}/extra-patch-10 > > > 〓.endif > > > +.endif > > > 〓 > > > 〓.if ! ${PORT_OPTIONS:MKERBEROS} > > > 〓GN_ARGS+= use_kerberos=true > > --〓 > > Tomoaki AOKI〓〓〓〓 > > > > > -- > Carlos Jacobo Puga Medina > PGP fingerprint = C60E 9497 5302 793B CC2D BB89 A1F3 5D66 E6D0 5453 -- 青木 知明 [Tomoaki AOKI]