Page MenuHomeFreeBSD

devel/valgrind-devel: update to Paul Floyd's FreeBSD valgrind branch
AcceptedPublic

Authored by kevans on Thu, Jun 25, 4:08 PM.

Details

Summary
devel/valgrind-devel: update to Paul Floyd's FreeBSD valgrind branch

This branch rebases us forward to Valgrind 3.17.0, incredibly current as
of today. It also fixes a vast array of known issues, making valgrind-devel
actually functional on 13.0-CURRENT and fixing most of the regression tests
on clang/amd64.

All patches previously in the port have been incorporated.

There are still some open issues remaining:
- https://github.com/paulfloyd/freebsd_valgrind/issues

This port is believed to be largely functional enough to get wider testing. Many,
many thanks to the excellent work of Paul Floyd to modernize our valgrind port.
Test Plan

Q/A:

  • portlint (some pre-existing issues to address separately, some addressed here)
  • testport (11.3/amd64, 12.1/i386, -CURRENT/amd64)

This is largely untested (at runtime) on 11.x, it would be good to get some testing there before commit as well. I've briefly tested it on -CURRENT/amd64 and Paul develops on 12.1/amd64.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

kevans created this revision.Thu, Jun 25, 4:08 PM
kevans requested review of this revision.Thu, Jun 25, 4:08 PM
lwhsu added a subscriber: lwhsu.Thu, Jun 25, 6:12 PM

I had a go at building, replacing valgrind-dev Makefile and distinfo. I got

fetch: https://codeload.github.com/paulfloyd/freebsd_valgrind/tar.gz/0511105?dummy=/paulfloyd-freebsd_valgrind-freebsd-3.17.0.g20200625-0511105_GH0.tar.gz: size of remote file is not known
paulfloyd-freebsd_valgrind-freebsd-3.17.0.g202          14 MB   18 kBps 13m18s
=> Fetched file size mismatch (expected 15485736, actual 15408039)

The pkg-plist diffs look OK from a Valgrind pov.

Whoops- I'll update this to the latest commit and upload a new diff soonish.

kevans updated this revision to Diff 73853.Mon, Jun 29, 2:38 AM
kevans retitled this revision from [WIP] devel/valgrind-devel: update to Paul Floyd's FreeBSD valgrind branch to devel/valgrind-devel: update to Paul Floyd's FreeBSD valgrind branch.
kevans edited the summary of this revision. (Show Details)

Looks OK now. make && make install worked, running the regression suite on the installed Valgrind gave me expected redults.

This revision is now accepted and ready to land.Tue, Jun 30, 11:38 AM
markj added a comment.Tue, Jun 30, 2:56 PM

JFYI, I submitted a status update referencing the port update: https://github.com/freebsd/freebsd-quarterly/pull/212

zeising accepted this revision.Wed, Jul 1, 1:05 PM