Hi,

Thanks again for taking care of it!

> On Tue, May 14 2019, Jeremie Courreges-Anglas <j...@wxcvbn.org> wrote:
> > On Mon, May 13 2019, Jeremie Courreges-Anglas <j...@wxcvbn.org>
> > wrote:
> >> Now that 10.2 has been committed, here's an update to the latest.
> >> Upstream removed the example config file so I decided to create one
> >> based on
> >>
> >>   https://mariadb.com/kb/en/library/configuring-mariadb-with-option-files/
> >>
> >> No symbol change in the SHARED_LIBS.  The diff also includes a fix
> >> for the test target (mysql-test tests don't finish here).
> >>
> >> Looking for real use reports and oks.
> >
> > no_server doesn't build any more because my_global.h file is
> > missing, so I'm holding off for now.  Test reports still welcome.
> 
> bah, I'm not sure why breaking the no_server PSEUDO_FLAVOR would
> matter. The problem is known upstream and I worked around it in the
> diff below.
> 
>   https://jira.mariadb.org/browse/MDEV-19152
> 
> Meanwhile, cwen@ reported that mariadb-10.2.23 doesn't build on
> powerpc because it's lacking __sync_* atomics.  10.3.x switched to
> __atomic_* builtins which are better supported by gcc-8.3.0, this
> fixes mariadb on powerpc according to cwen's tests (thanks!).

I've retested your diff and met no issues at all. 

Please note that it also needs the infamous atomics block - they
provide proper atomics detection... but no -latomic flag test.

I'm sending a diff of the Makefile, but you know very well what i'm
talking about ;)

> So here's a refreshed diff.  The test target is still kinda broken, so
> real use reports would be nice.
> 
> Brad, still ok?

Charlène.


Index: Makefile
===================================================================
RCS file: /cvs/ports/databases/mariadb/Makefile,v
retrieving revision 1.77
diff -u -p -r1.77 Makefile
--- Makefile    20 May 2019 10:31:23 -0000      1.77
+++ Makefile    25 May 2019 20:29:02 -0000
@@ -7,7 +7,7 @@ COMMENT-main=   multithreaded SQL database
 COMMENT-server=        multithreaded SQL database (server)
 COMMENT-tests= multithreaded SQL database (regression test suite/benchmark)
 
-VERSION=       10.2.23
+VERSION=       10.3.14
 DISTNAME=      mariadb-${VERSION}
 PKGNAME-main=  mariadb-client-${VERSION}
 PKGNAME-server=        mariadb-server-${VERSION}
@@ -46,7 +46,7 @@ BUILD_DEPENDS=        devel/bison
 
 LIB_DEPENDS-main=      ${LIB_DEPENDS} \
                        converters/libiconv
-RUN_DEPENDS-server=    ${BASE_PKGPATH},-main>=10.2v1,<10.3v1 \
+RUN_DEPENDS-server=    ${BASE_PKGPATH},-main>=10.3v1,<10.4v1 \
                        databases/p5-DBD-mysql
 LIB_DEPENDS-server=    ${LIB_DEPENDS} \
                        archivers/bzip2 \
@@ -56,7 +56,7 @@ LIB_DEPENDS-server=   ${LIB_DEPENDS} \
                        archivers/xz \
                        converters/libiconv \
                        devel/pcre
-RUN_DEPENDS-tests=     ${BASE_PKGPATH},-main>=10.2v1,<10.3v1
+RUN_DEPENDS-tests=     ${BASE_PKGPATH},-main>=10.3v1,<10.4v1
 LIB_DEPENDS-tests=     ${LIB_DEPENDS} \
                        converters/libiconv \
                        devel/pcre
@@ -115,6 +115,11 @@ CFLAGS+=   -I${LOCALBASE}/include
 CXXFLAGS+=     -I${LOCALBASE}/include
 LDFLAGS+=      -L${LOCALBASE}/lib
 
+.if ${MACHINE_ARCH} == "powerpc"
+LDFLAGS +=     -latomic
+WANTLIB +=     atomic
+.endif
+
 CONFIGURE_ENV= LDFLAGS="${LDFLAGS}"
 
 MULTI_PACKAGES=        -main -server -tests
@@ -143,10 +148,12 @@ post-install:
                cd mysql && \
                ln -sf libmariadb.so.${LIBmariadb_VERSION} 
libmysqlclient.so.${LIBmysqlclient_VERSION} && \
                ln -sf libmariadb.so.${LIBmariadb_VERSION} 
libmysqlclient_r.so.${LIBmysqlclient_r_VERSION}
+       ${INSTALL_DATA} ${FILESDIR}/my.cnf ${PREFIX}/share/examples/mysql/
        rm -Rf ${PREFIX}/share/mysql-test/plugin/rocksdb/
 
 .if ${BUILD_PACKAGES:M-tests}
 do-test:
+       (${MODCMAKE_TEST_TARGET}) || true # Keep on going even if regress tests 
fail
        @cd ${WRKBUILD}/mysql-test && ./mysql-test-run.pl --force --verbose 
--max-test-fail=0
 .else
 NO_TEST=       Yes

Reply via email to