svn commit: r485187 - in head/cad/openvsp: . files

Fernando Apesteguía fernape at freebsd.org
Sat Nov 17 18:35:26 UTC 2018


On Sat, Nov 17, 2018 at 7:08 PM Fernando Apesteguía <fernape at freebsd.org> wrote:
>
> Author: fernape
> Date: Sat Nov 17 18:08:10 2018
> New Revision: 485187
> URL: https://svnweb.freebsd.org/changeset/ports/485187
>
> Log:
>   cad/openvsp: Unbreak port
>
>   Port was broken after last compilers updates in 11 and 12.
>   Clang is now able to compile the port but we need to patch as_callfunc_x86.cpp
>   unconditionally, otherwise clang complains (or dies, depending on the version)
>   with:
>
>   this directive must appear between .cfi_startproc and .cfi_endproc directives
>
>   Also use localbase to simplify flags handling.
>
>   Reported by:  pkg-fallout
Sorry, I should've added Approved by: mentors (implicit)

>
> Added:
>   head/cad/openvsp/files/patch-src_external_angelscript_sdk_angelscript_source_as__callfunc__x86.cpp
>      - copied unchanged from r485186, head/cad/openvsp/files/extra-src_external_angelscript_sdk_angelscript_source_as__callfunc__x86.cpp
> Deleted:
>   head/cad/openvsp/files/extra-src_external_angelscript_sdk_angelscript_source_as__callfunc__x86.cpp
> Modified:
>   head/cad/openvsp/Makefile
>
> Modified: head/cad/openvsp/Makefile
> ==============================================================================
> --- head/cad/openvsp/Makefile   Sat Nov 17 18:04:08 2018        (r485186)
> +++ head/cad/openvsp/Makefile   Sat Nov 17 18:08:10 2018        (r485187)
> @@ -30,22 +30,15 @@ BUILD_DEPENDS=      ${LOCALBASE}/lib/libopenNURBS.a:graphic
>                 ${LOCALBASE}/include/glm/glm.hpp:math/glm \
>                 ${LOCALBASE}/libdata/pkgconfig/pthread-stubs.pc:devel/libpthread-stubs
>
> -CMAKE_SOURCE_PATH=     ${WRKSRC}/SuperProject
> -CMAKE_INSTALL_PREFIX=  ${STAGEDIR}${PREFIX}
> +USES=          compiler:c++11-lang cmake:outsource,noninja localbase\
> +               gl gnome jpeg
>
> -USES=          cmake:outsource,noninja jpeg
> -
>  USE_GITHUB=    yes
>  GH_PROJECT=    OpenVSP
>  GH_ACCOUNT=    ${GH_PROJECT}
>
> -# On FreeBSD < 11 libc++ lacks support for sized delete operators.
> -# GCC dropped the builtin version of these operators in 6+ by
> -# defaulting to gnu++14. Force c++11 instead.
> -# For a more detailed discussion see:
> -# https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=219484
> -USE_CXXSTD=    gnu++11
> -
> +CMAKE_SOURCE_PATH=     ${WRKSRC}/SuperProject
> +CMAKE_INSTALL_PREFIX=  ${STAGEDIR}${PREFIX}
>  CMAKE_ARGS=    -DVSP_USE_SYSTEM_CMINPACK:BOOLEAN=yes \
>                 -DVSP_USE_SYSTEM_CPPTEST:BOOLEAN=yes \
>                 -DVSP_USE_SYSTEM_CODEELI:BOOLEAN=no \
> @@ -60,23 +53,8 @@ USE_XORG=    ice sm x11 xau xcb xdamage xdmcp xext xfixes
>  USE_GL=                gl glu
>
>  USE_GNOME=     libxml2
> -CFLAGS+=       -I${LOCALBASE}/include/openNURBS
>
>  .include <bsd.port.options.mk>
> -
> -# In 12 gcc can not compile the port so switch to clang 5.0.0 in base.
> -# But wait! It crashes while compiling so apply a patch to avoid weird asm code
> -# that makes clang die
> -# For details see:
> -# https://lists.freebsd.org/pipermail/freebsd-hackers/2017-August/051352.html
> -
> -.if ${OPSYS} == FreeBSD && ${OSVERSION} >= 1200000
> -USES+= compiler:c++11-lib
> -EXTRA_PATCHES= ${FILESDIR}/extra-src_external_angelscript_sdk_angelscript_source_as__callfunc__x86.cpp
> -CFLAGS+=       -I${LOCALBASE}/include
> -.else
> -USES+= compiler:gcc-c++11-lib
> -.endif
>
>  BUILD_PREFIX=  OpenVSP-prefix/src/OpenVSP-build/_CPack_Packages/FreeBSD/ZIP/OpenVSP-${DISTVERSION}-FreeBSD
>
>
> Copied: head/cad/openvsp/files/patch-src_external_angelscript_sdk_angelscript_source_as__callfunc__x86.cpp (from r485186, head/cad/openvsp/files/extra-src_external_angelscript_sdk_angelscript_source_as__callfunc__x86.cpp)
> ==============================================================================
> --- /dev/null   00:00:00 1970   (empty, because file is newly added)
> +++ head/cad/openvsp/files/patch-src_external_angelscript_sdk_angelscript_source_as__callfunc__x86.cpp  Sat Nov 17 18:08:10 2018        (r485187, copy of r485186, head/cad/openvsp/files/extra-src_external_angelscript_sdk_angelscript_source_as__callfunc__x86.cpp)
> @@ -0,0 +1,164 @@
> +--- src/external/angelscript/sdk/angelscript/source/as_callfunc_x86.cpp.orig   2017-08-02 20:09:54 UTC
> ++++ src/external/angelscript/sdk/angelscript/source/as_callfunc_x86.cpp
> +@@ -340,7 +340,7 @@ endcopy:
> +       volatile asPWORD a[] = {asPWORD(args), asPWORD(paramSize), asPWORD(func)};
> +
> +       asm __volatile__(
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // When compiled with optimizations the stack unwind doesn't work properly,
> +               // causing exceptions to crash the application. By adding this prologue
> +               // and the epilogue below, the stack unwind works as it should.
> +@@ -385,7 +385,7 @@ endcopy:
> +               // Pop the alignment bytes
> +               "popl  %%esp            \n"
> +               "popl  %%ebx            \n"
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // Epilogue
> +               "movl %%ebp, %%esp         \n"
> +               ".cfi_def_cfa_register esp \n"
> +@@ -462,7 +462,7 @@ endcopy:
> +       volatile asPWORD a[] = {asPWORD(obj), asPWORD(args), asPWORD(paramSize), asPWORD(func)};
> +
> +       asm __volatile__ (
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // When compiled with optimizations the stack unwind doesn't work properly,
> +               // causing exceptions to crash the application. By adding this prologue
> +               // and the epilogue below, the stack unwind works as it should.
> +@@ -508,7 +508,7 @@ endcopy:
> +               // Pop the alignment bytes
> +               "popl  %%esp            \n"
> +               "popl  %%ebx            \n"
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // Epilogue
> +               "movl %%ebp, %%esp         \n"
> +               ".cfi_def_cfa_register esp \n"
> +@@ -585,7 +585,7 @@ endcopy:
> +       volatile asPWORD a[] = {asPWORD(obj), asPWORD(args), asPWORD(paramSize), asPWORD(func)};
> +
> +       asm __volatile__ (
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // When compiled with optimizations the stack unwind doesn't work properly,
> +               // causing exceptions to crash the application. By adding this prologue
> +               // and the epilogue below, the stack unwind works as it should.
> +@@ -631,7 +631,7 @@ endcopy:
> +               // Pop the alignment bytes
> +               "popl  %%esp            \n"
> +               "popl  %%ebx            \n"
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // Epilogue
> +               "movl %%ebp, %%esp         \n"
> +               ".cfi_def_cfa_register esp \n"
> +@@ -717,7 +717,7 @@ endcopy:
> +       volatile asPWORD a[] = {asPWORD(obj), asPWORD(args), asPWORD(paramSize), asPWORD(func), asPWORD(retPtr)};
> +
> +       asm __volatile__ (
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // When compiled with optimizations the stack unwind doesn't work properly,
> +               // causing exceptions to crash the application. By adding this prologue
> +               // and the epilogue below, the stack unwind works as it should.
> +@@ -767,7 +767,7 @@ endcopy:
> +               // Pop the alignment bytes
> +               "popl  %%esp            \n"
> +               "popl  %%ebx            \n"
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // Epilogue
> +               "movl %%ebp, %%esp         \n"
> +               ".cfi_def_cfa_register esp \n"
> +@@ -849,7 +849,7 @@ endcopy:
> +       volatile asPWORD a[] = {asPWORD(args), asPWORD(paramSize), asPWORD(func), asPWORD(retPtr)};
> +
> +       asm __volatile__ (
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // When compiled with optimizations the stack unwind doesn't work properly,
> +               // causing exceptions to crash the application. By adding this prologue
> +               // and the epilogue below, the stack unwind works as it should.
> +@@ -896,7 +896,7 @@ endcopy:
> +               // Pop the alignment bytes
> +               "popl  %%esp            \n"
> +               "popl  %%ebx            \n"
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // Epilogue
> +               "movl %%ebp, %%esp         \n"
> +               ".cfi_def_cfa_register esp \n"
> +@@ -980,7 +980,7 @@ endcopy:
> +       volatile asPWORD a[] = {asPWORD(obj), asPWORD(args), asPWORD(paramSize), asPWORD(func), asPWORD(retPtr)};
> +
> +       asm __volatile__ (
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // When compiled with optimizations the stack unwind doesn't work properly,
> +               // causing exceptions to crash the application. By adding this prologue
> +               // and the epilogue below, the stack unwind works as it should.
> +@@ -1030,7 +1030,7 @@ endcopy:
> +               // Pop the alignment bytes
> +               "popl  %%esp            \n"
> +               "popl  %%ebx            \n"
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // Epilogue
> +               "movl %%ebp, %%esp         \n"
> +               ".cfi_def_cfa_register esp \n"
> +@@ -1102,7 +1102,7 @@ endcopy:
> +       volatile asPWORD a[] = {asPWORD(args), asPWORD(paramSize), asPWORD(func)};
> +
> +       asm __volatile__ (
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // When compiled with optimizations the stack unwind doesn't work properly,
> +               // causing exceptions to crash the application. By adding this prologue
> +               // and the epilogue below, the stack unwind works as it should.
> +@@ -1145,7 +1145,7 @@ endcopy:
> +               // Pop the alignment bytes
> +               "popl  %%esp            \n"
> +               "popl  %%ebx            \n"
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // Epilogue
> +               "movl %%ebp, %%esp         \n"
> +               ".cfi_def_cfa_register esp \n"
> +@@ -1233,7 +1233,7 @@ endcopy:
> +       volatile asPWORD a[] = {asPWORD(obj), asPWORD(args), asPWORD(paramSize), asPWORD(func)};
> +
> +       asm __volatile__ (
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // When compiled with optimizations the stack unwind doesn't work properly,
> +               // causing exceptions to crash the application. By adding this prologue
> +               // and the epilogue below, the stack unwind works as it should.
> +@@ -1285,7 +1285,7 @@ endcopy:
> +               // Pop the alignment bytes
> +               "popl  %%esp            \n"
> +               "popl  %%ebx            \n"
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // Epilogue
> +               "movl %%ebp, %%esp         \n"
> +               ".cfi_def_cfa_register esp \n"
> +@@ -1380,7 +1380,7 @@ endcopy:
> +       volatile asPWORD a[] = {asPWORD(obj), asPWORD(args), asPWORD(paramSize), asPWORD(func), asPWORD(retPtr)};
> +
> +       asm __volatile__ (
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // When compiled with optimizations the stack unwind doesn't work properly,
> +               // causing exceptions to crash the application. By adding this prologue
> +               // and the epilogue below, the stack unwind works as it should.
> +@@ -1445,7 +1445,7 @@ endcopy:
> +               // Pop the alignment bytes
> +               "popl  %%esp           \n"
> +               "popl  %%ebx           \n"
> +-#ifdef __OPTIMIZE__
> ++#if defined(__OPTIMIZE__) && !defined(__clang__)
> +               // Epilogue
> +               "movl %%ebp, %%esp         \n"
> +               ".cfi_def_cfa_register esp \n"
>


More information about the svn-ports-all mailing list