From owner-freebsd-apache@FreeBSD.ORG Mon Sep 8 22:50:33 2014 Return-Path: Delivered-To: apache@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 51271461 for ; Mon, 8 Sep 2014 22:50:33 +0000 (UTC) Received: from mout.gmx.net (mout.gmx.net [212.227.17.22]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mout.gmx.net", Issuer "TeleSec ServerPass DE-1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id D004663409 for ; Mon, 8 Sep 2014 22:50:32 +0000 (UTC) Received: from [192.168.0.101] ([87.139.233.65]) by mail.gmx.com (mrgmx102) with ESMTPSA (Nemesis) id 0MYtId-1XvfCg2BUJ-00Vcoi; Tue, 09 Sep 2014 00:50:21 +0200 Message-ID: <540E32AD.3000101@gmx.de> Date: Tue, 09 Sep 2014 00:50:21 +0200 From: olli hauer User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:31.0) Gecko/20100101 Thunderbird/31.1.0 MIME-Version: 1.0 To: apache@freebsd.org Subject: Re: apache 2.2 ports References: <8DF8037F-F9EC-488D-86C4-0923789C174C@adamw.org> <8c7c200293f6d84e756d6eec4a1edcc2@mail.feld.me> <1078D71E80C9283D7F23513F@ogg.in.absolight.net> <540E1B41.8020804@gmx.de> In-Reply-To: <540E1B41.8020804@gmx.de> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Provags-ID: V03:K0:6RaohQb3XD6HwyxLuVJELaA+lGN8EYjxPD8AT243/cW2xU6LSq9 JYE16EbLaCOvYEJm5IDNFSn9g9YjoNPOJBC3KooOzXzrStm8ASEif/JymM1HcXTx6zhwGB5 OD8b9Qj8Tph7S2BM59mGPSWZszuOsAgnA6bobmyKc11xYJfn2nFxkeH1jid003iirObFS6D RNZXz/lbf3gJT7YR8JHdA== X-UI-Out-Filterresults: notjunk:1; Cc: Mathieu Arnold , Baptiste Daroussin , Adam Weinberger X-BeenThere: freebsd-apache@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Support of apache-related ports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Sep 2014 22:50:33 -0000 On 2014-09-08 23:10, olli hauer wrote: > On 2014-09-08 15:51, Mathieu Arnold wrote: >> >> >> +--On 8 septembre 2014 13:41:54 +0000 Mark Felder wrote: >> | September 6 2014 10:24 AM, "Adam Weinberger" wrote: >> |> apache team - >> |> >> |> So it seems that setting USE_APACHE=22 doesn’t actually depend on >> |> apache-2.2. This means that every USE_APACHE=22 port is broken, and as >> |> of right now won’t have any 10.1 packages. >> |> >> |> I’m happy to go through and add "DEFAULT_VERSIONS= APACHE=22” to >> |> this ports, but I wanted to check with you guys first. Is that the right >> |> way to fix it? Is there something else that can be done to make >> |> USE_APACHE=22 actually depend on apache-2.2? >> |> >> | >> | FYI I had to solve this problem today with the following two make.conf >> | options: >> | >> | DEFAULT_VERSIONS= apache=2.2 >> | >> | (note it's 2.2, not 22 -- bsd.default-versions.mk shows a period should >> | be used) >> | >> | APACHE_PORT= www/apache22 >> | >> | >> | Previously I only had the DEFAULT_VERSIONS entry and it worked fine. >> | Suddenly I needed to add APACHE_PORT as well. This is with an up to date >> | ports tree -- my packages build automatically every 8 hours. >> >> Yes, bapt sent a patch to fix that >> > > Sorry for breaking and delay! > > Seems I miss the mail with the notice from babt, anyway looking right now into his patch and found an issue with the patch on 8.x / 9.x but it is working on 10. > > Simple test on 8.4 or 9.3 with an apache22 only port > $ make -V APACHE_PORT > www/apache} => no value but a closing '}' > > It looks like the following line cannot be processed on 8.4 / 9.3 > _APACHE_WANTED_VERSION:= ${_APACHE_WANTED_VERSIONS:O:u:M${DEFAULT_APACHE_VERSION}} > > > Before this line the values are (verified with .warning line): > - _APACHE_WANTED_VERSIONS=22 > - DEFAULT_APACHE_VERSION=24 > > => M${DEFAULT_APACHE_VERSION} is not expanded and the construct returns '}' > > Now all other checks are failing and no valid port can be calculated > > > If I set this line to > _APACHE_WANTED_VERSION:= ${_APACHE_WANTED_VERSIONS:O:u:M24} > > I get the expected www/apache22 value. > > > Scratch my head and remember PR 181717, but thats not the case here. The patch from babt seems to work on 8.4 when the surrounding ${} is removed. - _APACHE_WANTED_VERSION:= ${_APACHE_WANTED_VERSIONS:O:u:M${DEFAULT_APACHE_VERSION}} + _APACHE_WANTED_VERSION:= ${_APACHE_WANTED_VERSIONS:O:u:MDEFAULT_APACHE_VERSION} Not sure why M${opt} is working with an .if statement but not here except on FreeBSD 10 Even with a custom make (source from 10 build on 8.4) the issue persist it must be something different. As first workaround I reverted bsd.apache.mk to r366839 until I have more information ...