Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 12 Jan 2013 21:40:06 +0100
From:      =?ISO-8859-1?Q?Ren=E9_Ladan?= <rene@freebsd.org>
To:        Cy Schubert <Cy.Schubert@komquats.com>
Cc:        svn-ports-head@FreeBSD.org, svn-ports-all@FreeBSD.org, ports-committers@FreeBSD.org
Subject:   Re: svn commit: r310215 - in head/www/chromium: . files
Message-ID:  <50F1CA26.4050604@freebsd.org>
In-Reply-To: <201301121859.r0CIxUld069366@slippy.cwsent.com>
References:  <201301121859.r0CIxUld069366@slippy.cwsent.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 12-01-2013 19:59, Cy Schubert wrote:
> In message <201301110114.r0B1Eq5P003555@svn.freebsd.org>, Rene Ladan writes:
>> Author: rene
>> Date: Fri Jan 11 01:14:52 2013
>> New Revision: 310215
>> URL: http://svnweb.freebsd.org/changeset/ports/310215
>>
>> Log:
>>   Update to 24.0.1312.52, which add support for MathML.
>>   
>>   FreeBSD-specific improvements:
>>   - working WiFi geolocation [1] (can be turned off in preferences)
>>   - fix build with clang (use external libvpx)
>>   - use SysV SHM model and where possible OS_BSD, which saves some patches
>>   - no need to use ZygoteProcess switch
>>   
>>   HTML5 audio is currently broken.
>>   
>>   PR:		ports/174402 [1]
>>   Submitted by:	J.R. Oldroyd <fbsd@opal.com> [1]
>>   Security:	http://www.vuxml.org/freebsd/46bd747b-5b84-11e2-b06d-00262d5ed8
>> ee.html
> [...]
>
> Hi Rene,
>
> It doesn't build. I don't have the actual error before me but the the 
> following patch resolves the issue.
>
> ===== cut here
> --- media/filters/decrypting_audio_decoder.cc.orig	2013-01-07 
> 18:40:10.000000000 -0800
> +++ media/filters/decrypting_audio_decoder.cc	2013-01-12 09:05:12.737640582 
> -0800
> @@ -30,7 +30,7 @@
>    // Out of sync of 100ms would be pretty noticeable and we should keep any
>    // drift below that.
>    const int64 kOutOfSyncThresholdInMicroseconds = 100000;
> -  return std::abs(timestamp_1.InMicroseconds() - timestamp_2.InMicroseconds
> ()) >
> +  return std::labs(timestamp_1.InMicroseconds() - 
> timestamp_2.InMicroseconds()) >
>           kOutOfSyncThresholdInMicroseconds;
>  }
>  
> ===== cut here
>
This looks like the error I got when building chromium on 8 or 9 with
clang (from [1]):

In file included from media/filters/decrypting_audio_decoder.cc:1:
media/filters/decrypting_audio_decoder.cc:33:10: error: call to 'abs' is ambiguous
  return std::abs(timestamp_1.InMicroseconds() - timestamp_2.InMicroseconds()) >
         ^~~~~~~~
/usr/include/stdlib.h:83:5: note: candidate function
int abs(int) __attribute__((__const__));
    ^
/usr/include/c++/4.2/cstdlib:142:3: note: candidate function
  abs(long __i) { return labs(__i); }
  ^
1 error generated.
gmake: *** [out/Release/obj.target/media/media/filters/decrypting_audio_decoder.o] Error 1


I'll rebuild the port in my tinderbox with your patch.

[1] http://rene-ladan.nl/tb/errors/9-FreeBSD/chromium-24.0.1312.45.log (8.0 MB)




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?50F1CA26.4050604>