From owner-svn-ports-head@freebsd.org Fri Apr 13 09:55:19 2018 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 99C26FA0379; Fri, 13 Apr 2018 09:55:19 +0000 (UTC) (envelope-from idefix@fechner.net) Received: from anny.lostinspace.de (anny.lostinspace.de [IPv6:2001:608:a02::33]) (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 2AFAF6CBAA; Fri, 13 Apr 2018 09:55:19 +0000 (UTC) (envelope-from idefix@fechner.net) Received: from server.idefix.lan (unknown [IPv6:2a01:5c0:15:f731:21b:21ff:fe7b:3468]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) (Authenticated sender: idefix@fechner.net) by anny.lostinspace.de (Postfix) with ESMTPSA id 010BE114279B; Fri, 13 Apr 2018 11:55:17 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=fechner.net; s=default; t=1523613317; bh=7g5jSLdpARl5zTFqKb12Pt++rT3vBTAR91gJpS6EaxE=; h=Subject:To:Cc:References:From:Date:In-Reply-To; b=VFqE0GvHORdLAJ6OOsAavO0LXn/UdusdgW0kCOvqt3aApHOiK1d/w/xsd2d63u5R4 tjMenGJaVgcoi5zgto6ZfxfolVuwVLKNM9Ok/FeDXexWcCTlR6CFG8J8ruJM/t0Tj2 08XMY9FMJzOgj/f3baWFThrYW6KuEK9PXlKy3mxk= Received: from [192.168.0.151] (143-058-210-188.ip-addr.inexio.net [188.210.58.143]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by server.idefix.lan (Postfix) with ESMTPSA id C82F11514BC3; Fri, 13 Apr 2018 11:55:14 +0200 (CEST) Subject: Re: svn commit: r467193 - in head/www/gitlab: . files To: Sunpoet Po-Chuan Hsieh Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org References: <201804121833.w3CIXtgW077267@repo.freebsd.org> From: Matthias Fechner Openpgp: preference=signencrypt Message-ID: <431aaec9-51c2-c0c9-7a1f-2f29f79edb5d@fechner.net> Date: Fri, 13 Apr 2018 11:55:13 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.25 X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.25 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, 13 Apr 2018 09:55:19 -0000 Am 13.04.2018 um 10:41 schrieb Sunpoet Po-Chuan Hsieh: > > I think that=C2=A0default_value_for 3.0.4 update is a different case. > gitlab wats=C2=A0default_value_for ~> 3.0.0 and it fits the rule. > > I would like to keep less number of gems with multiple versions in the > ports tree. > It will sometimes make the dependency tree complex. > Therefore,=C2=A0I would not copy the old version unless it's necessary.= > > In this case, the only change between default_value_for=C2=A03.0.5 and > 3.1.0 is Rails 5.2 support [1]. > I don't think it breaks gitlab. > Can you please confirm if it really breaks gitlab? > If so, I'll revert it as soon as possible. I spent already days to find out why gitlab does not work or better that some features are not working. An it most of the cases it was the last feature I tested that failed. All these problems are related to the fact that the Gemfile that is delivered by gitlab was patched to use a newer version. And please believe me, that is not fun. This is the reason why I removed nearly all patches from the Gemfile. I include only patches that are absolutely necessary to get the gitlab port install-able with FreeBSD ports (e.g. sentry-raven, httparty, sanitize and some more). To make tests for all features in Gitlab you will require many hours and I'm not willing to do the work the gitlab team is doing already for us. They provide us with clear instructions what version are tested and should work. To be 100% save we should use what is in the Gemfile.lock, but this would really a major effort and I think to use what is in Gemfile defined is relatively safe. I remember only 2 cases where even the definition in the Gemfiles cause gitlab to break. One was the upgrade of default_value_for from 3.0.3 to 3.0.4. So please stop to modify the Gemfile that comes with gitlab. Please create a new port rubygem-default_value_for30 which is copied from version 3.0.5. If you like you can also fix the dep in the gitlab port to point to this rubygem-default_value_for30 or I do not have a problem to do it by myself= =2E If ports are not required by gitlab anymore, I will mark them as expired to clean up obsolete ports. Thanks a lot Sunpoet for you support i really appreciate. Gru=C3=9F Matthias --=20 "Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the universe trying to produce bigger and better idiots. So far, the universe is winning." -- Rich Cook