Thought about that, but was too lazy to check that libtorrent is only
used by rtorrent. It is.

ok

On Fri, Jul 31, 2015 at 03:31:41PM +0100, Stuart Henderson wrote:
> On 2015/07/31 15:53, Tobias Ulmer wrote:
> > Is David around? Any objections should be voiced in the next few hours,
> > before I commit this.
> 
> It's actually libtorrent not rtorrent that should be marked broken.
> 
> Would it be worth building on i386 with compiler flags to fix that,
> at the expense of not running on crappy old machines that you probably
> wouldn't want to run this on anyway?
> 
> Index: libtorrent/Makefile
> ===================================================================
> RCS file: /cvs/ports/net/libtorrent/Makefile,v
> retrieving revision 1.40
> diff -u -p -r1.40 Makefile
> --- libtorrent/Makefile       24 Mar 2015 13:35:09 -0000      1.40
> +++ libtorrent/Makefile       31 Jul 2015 14:31:14 -0000
> @@ -2,6 +2,10 @@
>  
>  COMMENT=             BitTorrent library written in C++
>  
> +BROKEN-hppa =                undefined references to __sync atomic ops
> +BROKEN-mips64 =              undefined references to __sync atomic ops
> +BROKEN-sh =          undefined references to __sync atomic ops
> +
>  # requires C++ tr1 headers
>  NOT_FOR_ARCHS=               ${GCC3_ARCHS}
>  
> @@ -28,5 +32,13 @@ CONFIGURE_ARGS=            ${CONFIGURE_SHARED} \
>                       --with-kqueue \
>                       --without-epoll \
>                       --disable-debug
> +
> +.include <bsd.port.arch.mk>
> +
> +# for 64-bit atomic ops
> +.if ${ARCH:Mi386}
> +CFLAGS+=     -march=i586 -fomit-frame-pointer
> +CXXFLAGS+=   -march=i586 -fomit-frame-pointer
> +.endif
>  
>  .include <bsd.port.mk>
> Index: rtorrent/Makefile
> ===================================================================
> RCS file: /cvs/ports/net/rtorrent/Makefile,v
> retrieving revision 1.48
> diff -u -p -r1.48 Makefile
> --- rtorrent/Makefile 28 May 2015 10:17:24 -0000      1.48
> +++ rtorrent/Makefile 31 Jul 2015 14:31:25 -0000
> @@ -1,10 +1,5 @@
>  # $OpenBSD: Makefile,v 1.48 2015/05/28 10:17:24 pascal Exp $
>  
> -BROKEN-hppa =                undefined references to __sync atomic ops
> -BROKEN-mips64 =              undefined references to __sync atomic ops
> -BROKEN-sh =          undefined references to __sync atomic ops
> -BROKEN-i386 =                64-bit atomics (fetch_and_and, add_and_fetch)
> -
>  COMMENT=             ncurses BitTorrent client based on libTorrent
>  
>  DISTNAME=            rtorrent-0.9.4
> 

Reply via email to