search

AdaCore / gnatcoverage

GNATcoverage: Coverage Analysis Tool
39 stars 9 forks source link

Please update 19.1 branch tools/gnatcov/README.md info about binutils version #12

Closed okellogg closed 3 years ago

okellogg commented 3 years ago

Hi,

The 19.1 branch in tools/gnatcov/README.md section Build from sources point 2. says

Best is to use a Binutils release for this, at least 2.29 which is known to work.

binutils-2.29 $ ./configure --enable-targets=all \        --disable-werror --without-zlib --disable-gdb \        --disable-sim --disable-ld --disable-libquadmath \        --disable-readline binutils-2.29 $ make

However, the branch contains the commit 0bce932,

commit 0bce9324f56a29833627ac776b532cad7b2ece6e
Author: Pierre-Marie de Rodat <derodat@adacore.com>
Date:   Wed May 31 18:33:22 2017 +0200

    libopcodes_bind: update to build with latest binutils

    Change-Id: I3f83ce8ead8210fb2aa4faf08be450b7e9f24ff4
    TN: Q529-032

M       tools/gnatcov/libopcodes_bind/dis_wrapper.c

which means that the binutils version should be at least 2.31.

HTH, Oliver

okellogg commented 3 years ago

This turned out to be a misconfiguration on our part. The build works with binutils-2.29 as well. Sorry for the noise.

hainque commented 3 years ago

Hello Olivier

This turned out to be a misconfiguration on our part. The build works with binutils-2.29 as well. Sorry for the noise.

No pb, thanks for letting us know.

Cheers,

Olivier (as well)