On Tue, Jun 26, 2018 at 11:10:29AM +0200, Sebastien Marie wrote:
> Hi,
> 
> Here the update of lang/rust for 1.27.0
> 
> Release notes: 
> https://github.com/rust-lang/rust/blob/stable/RELEASES.md#version-1270-2018-06-21
> 
> Port wise:
> - gdb tests now works. so I remove the patch that disable them, and
>   added devel/gdb dependency (on amd64 and i386)

in case somebody else than landry@ want to test...
an additional diff to apply over the previous: devel/gdb is a
BUILD_DEPENDS and not a TEST_DEPENDS, as the build script of rust will
check it presence if `gdb' is filled in config.toml file.

thanks to landry@ to spotted it.
-- 
Sebastien Marie

commit 570b706966bcdb513d1d8946b7a340b1d4e6b745
Author: Sébastien Marie <[email protected]>
Date:   Tue Jun 26 18:58:28 2018 +0200

    lang/rust: build check config.toml, so gdb should be BDEP

diff --git a/lang/rust/Makefile b/lang/rust/Makefile
index 3d90632..7b1e317 100644
--- a/lang/rust/Makefile
+++ b/lang/rust/Makefile
@@ -83,6 +83,10 @@ BUILD_DEPENDS +=     shells/bash
 BUILD_DEPENDS +=       devel/llvm>=6.0,<6.1
 BUILD_DEPENDS +=       devel/ninja
 
+.if ${MACHINE_ARCH} != aarch64
+BUILD_DEPENDS +=       devel/gdb
+.endif
+
 LIB_DEPENDS +=         devel/libgit2/libgit2 \
                        net/curl \
                        security/libssh2
@@ -106,10 +110,6 @@ USE_LIBTOOL =              gnu
 TEST_DEPENDS +=                devel/git \
                        sysutils/ggrep
 
-.if ${MACHINE_ARCH} != aarch64
-TEST_DEPENDS +=                devel/gdb
-.endif
-
 # - disable vendor checksum checks
 # - SUBSTR cargo testsuite
 SUBST_VARS +=  WRKBUILD

Reply via email to