From owner-freebsd-ports@FreeBSD.ORG Sat Mar 21 18:03:03 2015 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 943AA464; Sat, 21 Mar 2015 18:03:03 +0000 (UTC) Received: from mail.madpilot.net (grunt.madpilot.net [78.47.145.38]) (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 38F385EC; Sat, 21 Mar 2015 18:03:02 +0000 (UTC) Received: from mail (mail [192.168.254.3]) by mail.madpilot.net (Postfix) with ESMTP id 3l8VF62K1kzZs9; Sat, 21 Mar 2015 19:02:50 +0100 (CET) Received: from mail.madpilot.net ([192.168.254.3]) by mail (mail.madpilot.net [192.168.254.3]) (amavisd-new, port 10024) with ESMTP id nFa2PO4yxvPI; Sat, 21 Mar 2015 19:02:48 +0100 (CET) Received: from marvin.madpilot.net (micro.madpilot.net [88.149.173.206]) by mail.madpilot.net (Postfix) with ESMTPSA; Sat, 21 Mar 2015 19:02:48 +0100 (CET) Message-ID: <550DB247.2050800@FreeBSD.org> Date: Sat, 21 Mar 2015 19:02:47 +0100 From: Guido Falsi User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Geoffrey Mainland , Jeremie Le Hen Subject: Re: net/unison240 depends on lang/ocaml-nox11 References: <5507555C.7030508@FreeBSD.org> <5507F80F.70609@FreeBSD.org> <55080F30.9010104@FreeBSD.org> <550B65A1.9080402@apeiron.net> <550DAD30.4080905@FreeBSD.org> <550DB13D.7020005@apeiron.net> In-Reply-To: <550DB13D.7020005@apeiron.net> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: freebsd-ports@freebsd.org X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 21 Mar 2015 18:03:03 -0000 On 03/21/15 18:58, Geoffrey Mainland wrote: > On 3/21/15 10:41 AM, Guido Falsi wrote: >> On 03/20/15 01:11, Geoffrey Mainland wrote: >>> On 03/17/2015 07:25 AM, Guido Falsi wrote: >>>> On 03/17/15 11:44, Jeremie Le Hen wrote: >>>>> On Tue, Mar 17, 2015 at 10:46 AM, Guido Falsi wrote: >>>>>> On 03/17/15 09:31, Jeremie Le Hen wrote: >>>>>>> On Mon, Mar 16, 2015 at 11:12 PM, Guido Falsi wrote: >>>>>>>> On 03/16/15 22:37, Jeremie Le Hen wrote: >>>>>>>>> Actually, I've just realized that I fixed net/unison232 in my local tree :o). >>>>>>>>> >>>>>>>>> Would you mind submitting it and applying the same for unison240? >>>>>>>>> >>>>>>>> I never noticed this since it never happened to me and nobody else >>>>>>>> reported it. >>>>>>>> >>>>>>>> Anyway now that you draw my attention, yes, it needs fixing. >>>>>>>> >>>>>>>> Your patch looks correct, but please allow me a little time for some >>>>>>>> testing. >>>>>>> OK thanks! :) >>>>>> Just committed it. Thanks again for reporting the issue! >>>>> Thanks! >>>>> >>>>> I don't want to abuse your time, but "make checksum" is broken on >>>>> net/unison240 :). >>>>> >>>> Looks like distfiles were rerolled. 2.48 has the same problem. >>>> >>>> Will commit updated checksums shortly! >>>> >>>> Thanks again. >>> I still get the same error Jeremie was getting with poudriere. >>> >>> I think the BUILD_DEPENDS should not be set unconditionally. The below >>> patch fixes the problem for me. >>> >>> Does this look correct to you? >>> >>> Thanks, >>> Geoff >>> >>> diff --git a/net/unison/Makefile b/net/unison/Makefile >>> index 7404beb..461ebab 100644 >>> --- a/net/unison/Makefile >>> +++ b/net/unison/Makefile >>> @@ -15,7 +15,7 @@ COMMENT?= User-level file synchronization tool >>> >>> LICENSE= GPLv3 >>> >>> -BUILD_DEPENDS= ocamlc:${PORTSDIR}/lang/ocaml >>> + >>> >>> PLIST_SUB= PORTVERSION=${PORTVERSION} >>> USES= gmake >>> @@ -38,11 +38,13 @@ OPTIONS_DEFAULT?= DOCS X11 >>> .if ${PORT_OPTIONS:MX11} >>> MAKE_ARGS+= UISTYLE=gtk2 >>> PLIST_SUB+= TEXT="" >>> +BUILD_DEPENDS= ocamlc:${PORTSDIR}/lang/ocaml >>> BUILD_DEPENDS+= lablgtk2:${PORTSDIR}/x11-toolkits/ocaml-lablgtk2 \ >>> icotool:${PORTSDIR}/graphics/icoutils >>> RUN_DEPENDS+= lablgtk2:${PORTSDIR}/x11-toolkits/ocaml-lablgtk2 >>> SUB_FILES+= ${PORTNAME}.desktop >>> .else >>> +BUILD_DEPENDS= ocamlc:${PORTSDIR}/lang/ocaml-nox11 >>> MAKE_ARGS+= UISTYLE=text >>> PLIST_SUB+= TEXT="@comment " >>> PKGMESSAGE= ${PKGDIR}/pkg-message.nox11 >>> >> This is the same as the port was before the latest patch. >> >> (having BUILD_DEPENDS= followed by BUILD_DEPENDS+= is semantically the >> same as one unified BUILD_DEPENDS+=) >> >> Please send me your build log. I need to understand what is going on. >> Can you also tell me what other ports you are building in the same >> poudriere run together with unison? I'm interested in other ports which >> depend on ocaml. Maybe there is some other port in the tree with the >> same problem unison had, which are unmasked by my latest patch. >> >> BTW I'm going to investigate the option of modifying ocaml port >> behavior, I'm not sure it is really correct anymore. >> > > I can send you the build logs, but no, it's not the same as before the > patch. > > Note the dependency is changed to lang/ocaml-nox11 instead of > lang/ocaml. That is, we depend on lang/ocaml when the X11 option is > enabled, and lang/ocaml-nox11 when the X11 option is not enabled. It is not the same as teh port is now, but it is the same as the port was before r381478, you can check this diff: https://svnweb.freebsd.org/ports/head/net/unison240/Makefile?r1=381478&r2=381477 (other changes are there too, but irrelevant for the depends matter) > > If you still think there is no difference, I will send you full build > logs both with and without the patch. I'd need the logs anyway, since I don't know what the cause of the error you are seeing is. I also need to know if in the same poudriere run you were building any other port depending on ocaml. -- Guido Falsi