Bug 212289

Summary: devel/binutils: should provide a test/regression target
Product: Ports & Packages Reporter: Pedro F. Giffuni <pfg>
Component: Individual Port(s)Assignee: Baptiste Daroussin <bapt>
Status: New ---    
Severity: Affects Only Me CC: pfg
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (bapt)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Proof-of-concept patch none

Description Pedro F. Giffuni freebsd_committer 2016-08-31 16:50:06 UTC
Created attachment 174258 [details]
Proof-of-concept patch

Being such a critical third-party module we should have testing options for this port so we can catch regressions or evaluate better before enabling features.

I borrowed the test option from the GCC port. I am not sure this adapts perfectly to binutils as the tests don't finish but it's a start.
Comment 1 Baptiste Daroussin freebsd_committer 2017-10-02 11:34:13 UTC
The approach is wrong, all what is needed is a simple:
TEST_TARGET=check

Also note that to depend on dejagnu, TEST_DEPENDS=... should be enough.

Btw the testsuite works here to the exception of some weird -Wa,-madd-bnd-prefix which clang does not accept