Date: Sun, 12 Feb 2017 21:47:57 +0000 From: bugzilla-noreply@freebsd.org To: multimedia@FreeBSD.org Subject: maintainer-feedback requested: [Bug 217051] [PATCH] multimedia/vlc: Build in C++11 mode by default Message-ID: <bug-217051-12827-odOM6hHaeI@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-217051-12827@https.bugs.freebsd.org/bugzilla/> References: <bug-217051-12827@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
Raphael Kubo da Costa <rakuco@FreeBSD.org> has reassigned Bugzilla Automati= on <bugzilla@FreeBSD.org>'s request for maintainer-feedback to multimedia@FreeBSD.org: Bug 217051: [PATCH] multimedia/vlc: Build in C++11 mode by default https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D217051 --- Description --- Created attachment 179920 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D179920&action= =3Dedit Proposed patch This is in preparation for the Qt 5.6.2 -> 5.7.1 update (bug 216797): Qt 5.7 requires C++11 support. VLC 2.2.4 has issues building in C++11 mode, so incorporate a few upstream commits to make things work, and adjust the atomics-related patches landed = in ports r416225, as including <atomic> only with libc++ >=3D 3.8.0 leaves out FreeBSD 10.3. include/vlc_atomic.h is now closer to its git (post-2.2.4) version, in that we now have 3 possible conditions: 1. The header is included in C11 mode, in which case <stdatomic.h> is inclu= ded. 2. The header is included by C code but we're not building in C11 mode, in which case some shims are defined. 3. The header is included by C++ code, so we can assume C++11 and just incl= ude <atomic>. It will likely be possible to drop the changes once a new VLC version is released, as upstream has since started requiring C+11 and passes the appropriate flags to the compiler.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-217051-12827-odOM6hHaeI>