Skip to content

Commit

Permalink
doc: update topotests doc for address sanitizer
Browse files Browse the repository at this point in the history
Use the `--enable-address-sanitizer` option instead of the manual
version using environment flags.

This also avoids the problem of having to remember to skip clippy with
the custom flags:

```
make -C lib CFLAGS="-g -O2" LDFLAGS="-g" clippy
```

The snippet above is not needed with `--enable-address-sanitizer`!

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
  • Loading branch information
rzalamena committed Oct 9, 2019
1 parent 65209e4 commit 4260f16
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions doc/developer/topotests.rst
Original file line number Diff line number Diff line change
Expand Up @@ -189,11 +189,8 @@ for ``master`` branch:
git clone https://github.com/FRRouting/frr.git
cd frr
./bootstrap.sh
export CC=gcc
export CFLAGS="-O1 -g -fsanitize=address -fno-omit-frame-pointer"
export LD=gcc
export LDFLAGS="-g -fsanitize=address -ldl"
./configure --enable-shared=no \
./configure \
--enable-address-sanitizer \
--prefix=/usr/lib/frr --sysconfdir=/etc/frr \
--localstatedir=/var/run/frr \
--sbindir=/usr/lib/frr --bindir=/usr/lib/frr \
Expand Down

0 comments on commit 4260f16

Please sign in to comment.