From owner-freebsd-ports@freebsd.org Tue Apr 3 12:27:31 2018 Return-Path: Delivered-To: freebsd-ports@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 B15AEF7CAAD for ; Tue, 3 Apr 2018 12:27:31 +0000 (UTC) (envelope-from w.schwarzenfeld@utanet.at) Received: from taro.utanet.at (taro.utanet.at [IPv6:2001:938:1337:25::25:45]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 20D657A72F for ; Tue, 3 Apr 2018 12:27:31 +0000 (UTC) (envelope-from w.schwarzenfeld@utanet.at) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=utanet.at; s=rev1; h=Content-Type:In-Reply-To:MIME-Version:Date:Message-ID:From:Subject:References:To; bh=YwSMxUqyxTG0JHVwdW1hXk+e058Z7cHuNbnqR1vtaz0=; b=rVc4M/L5R2omYDjSJOMLwiX8b/MkvEZIDfy869VwqJNUIeOjPor9DQETxNf3hQJTNBqS3u+mu6n1xHZ1nHUE+86ZP2+rVnTY49FdcXz8drAMHakBRfQ03yynnAtXJb1KIUqFXiaNTAuy94cpcP5QnN0E7ytyIBnlUYam4xgPs84=; Received: from pam.xoc.tele2net.at ([213.90.36.6]) by taro.utanet.at with esmtp (Exim 4.80) (envelope-from ) id 1f3L29-0001UJ-C9 for freebsd-ports@freebsd.org; Tue, 03 Apr 2018 14:27:29 +0200 Received: from 193-80-170-124.adsl.highway.telekom.at ([193.80.170.124] helo=[10.0.0.93]) by pam.xoc.tele2net.at with esmtpsa (TLSv1:DHE-RSA-AES128-SHA:128) (Exim 4.80) (envelope-from ) id 1f3L29-000708-5i for freebsd-ports@freebsd.org; Tue, 03 Apr 2018 14:27:29 +0200 To: freebsd-ports@freebsd.org References: <86vad8iuq5.fsf@newsrv.interne.associated-bears.org> Subject: Re: Logitechmediaserver fails to build From: Walter Schwarzenfeld Message-ID: Date: Tue, 3 Apr 2018 14:27:25 +0200 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <86vad8iuq5.fsf@newsrv.interne.associated-bears.org> Content-Language: en-US X-TELE2-Authenticated-As: cf62768e218b4d81d95cc5390f1bd9dfc8b05683 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-Content-Filtered-By: Mailman/MimeDel 2.1.25 X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 03 Apr 2018 12:27:31 -0000 It is a little more complicated as I thought in the first moment: You also need add to your patch: @@ -520,7 +537,7 @@ function build {              ;;          Class::XSAccessor) -            if [ "$PERL_516" -o "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" ]; then +            if [ "$PERL_516" -o "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" -o "$PERL_526" ]; then                  build_module Class-XSAccessor-1.18                  cp -pR $PERL_BASE/lib/perl5/$ARCH/Class $PERL_ARCH/              else @@ -536,7 +553,7 @@ function build {              ;;          DBI) -            if [ "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" ]; then +            if [ "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" -o "$PERL_526" ]; then                  build_module DBI-1.628                  cp -p $PERL_BASE/lib/perl5/$ARCH/DBI.pm $PERL_ARCH/                  cp -pR $PERL_BASE/lib/perl5/$ARCH/DBI $PERL_ARCH/ @@ -546,7 +563,7 @@ function build {              ;;          DBD::SQLite) -            if [ "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" ]; then +            if [ "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" -o "$PERL_526" ]; then                  build_module DBI-1.628 "" 0              else                  build_module DBI-1.616 "" 0 @@ -632,7 +649,7 @@ function build {                  rm -rf DBD-SQLite-1.34_01              else                  cd .. -                if [ "$PERL_516" -o "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" ]; then +                if [ "$PERL_516" -o "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" -o "$PERL_526" ]; then                     build_module DBD-SQLite-1.34_01 "" 0                  fi                  build_module DBD-SQLite-1.34_01 @@ -714,7 +731,7 @@ function build {          JSON::XS)              build_module common-sense-2.0 -            if [ "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" ]; then +            if [ "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" -o "$PERL_526" ]; then                  build_module JSON-XS-2.34                  cp -pR $PERL_BASE/lib/perl5/$ARCH/JSON $PERL_ARCH/              else @@ -744,7 +761,7 @@ function build {              ;;          YAML::LibYAML) -            if [ "$PERL_516" -o "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" ]; then +            if [ "$PERL_516" -o "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" -o "$PERL_526" ]; then                  build_module YAML-LibYAML-0.35 "" 0              else                  build_module YAML-LibYAML-0.35 @@ -1403,7 +1420,7 @@ find $BUILD -name '*.packlist' -exec rm  # create our directory structure  # rsync is used to avoid copying non-binary modules or other extra stuff -if [ "$PERL_512" -o "$PERL_514" -o "$PERL_516" -o "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" ]; then +if [ "$PERL_512" -o "$PERL_514" -o "$PERL_516" -o "$PERL_518" -o "$PERL_520" -o "$PERL_522" -o "$PERL_524" -o "$PERL_526" ]; then      # Check for Perl using use64bitint and add -64int      ARCH=`$PERL_BIN -MConfig -le 'print $Config{archname}' | sed 's/gnu-//' | sed 's/^i[3456]86-/i386-/' | sed 's/armv.*?-/arm-/' `  fi but this conflicts with the extrapatch. (vendorpatch-CPAN_buildme.sh) It build if I remove the first part (./slimserver-vendor-1463b00/CPAN/buildme.sh) of this patch. But in this patch is another conditional patch. And if I remove the first part of it, there is no condition anymore: # because it takes so damn long on slow platforms              if [ ! -f build/lib/libicudata_s.a ]; then                  tar_wrapper zxvf icu4c-4_6-src.tgz =>+               patch << END  So the second part will applied if needed or not. I got it to build. For installation we need a new pkg-plist. I could install it with a new pkg-plist, but it has the variable %%AMD64_ONLY%% (must added per hand). And I am not really clear in which line it is needed. So I think it is better, you make a PR.