Page MenuHomeFreeBSD

devel/valgrind: temporary build fix for FreeBSD 15.
ClosedPublic

Authored by des on Aug 27 2023, 6:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 1:33 AM
Unknown Object (File)
Tue, Apr 23, 2:32 PM
Unknown Object (File)
Mon, Apr 8, 4:20 PM
Unknown Object (File)
Sat, Apr 6, 7:34 PM
Unknown Object (File)
Mar 22 2024, 7:40 PM
Unknown Object (File)
Mar 22 2024, 7:40 PM
Unknown Object (File)
Mar 9 2024, 4:14 PM
Unknown Object (File)
Mar 9 2024, 3:57 PM

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des requested review of this revision.Aug 27 2023, 6:45 PM

Whilst that should work my plans were to do the following

Leave devel/valgrind alone until Valgrind 3.22 is released in a couple of months
When I have VMs to test on FreeBSD 15 make the fixes upstream and bump the version of devel/valgrind-devel.

How many failures are there with "gmake regtest" ?

Leave devel/valgrind alone until Valgrind 3.22 is released in a couple of months

Why would you intentionally leave the port broken for months when a simple fix exists?

FreeBSD 15.0 isn't even available for download yet.

I don't consider a quick hack to the configure script to be a fix.

FreeBSD 15.0 isn't even available for download yet.

I don't consider a quick hack to the configure script to be a fix.

It's not a hack. Right now FreeBSD 15 is essentially identical to FreeBSD 14 except for the version number.

FreeBSD 15.0 isn't even available for download yet.

Snapshots will start being available soon, and a decent fraction of the FreeBSD developer community is already running 15 or will be shortly.

In D41610#948312, @des wrote:

It's not a hack. Right now FreeBSD 15 is essentially identical to FreeBSD 14 except for the version number.

Really? Have you run the regression suite as I asked? There's a lot of platform specific code in Valgrind. It should work OK but I've seen cases that were wrong in the past and needed updating after a version upgrade.

Valgrind doesn't support FreeBSD 14 well just yet. I've made some improvements since the alpha cycle started.

I don't subscibe to the view "if it builds, ship it".

FreeBSD 15.0 isn't even available for download yet.

Snapshots will start being available soon, and a decent fraction of the FreeBSD developer community is already running 15 or will be shortly.

I'm planning on setting up FreeBSD 15 VMs for amd64 and i386 as soon as downloads are available.

Note, this would also apply to devel/valgrind-devel.

In D41610#948312, @des wrote:

Right now FreeBSD 15 is essentially identical to FreeBSD 14 except for the version number.

Really?

Yes, really.

In D41610#948355, @dim wrote:

Note, this would also apply to devel/valgrind-devel.

As I said

When I have VMs to test on FreeBSD 15 make the fixes upstream and bump the version of devel/valgrind-devel.

In D41610#948361, @des wrote:
In D41610#948312, @des wrote:

Right now FreeBSD 15 is essentially identical to FreeBSD 14 except for the version number.

Really?

Yes, really.

The rhetorical question was referring to your assertation that just fixing the build is not a hack.

When I have VMs to test on FreeBSD 15

You can easily upgrade a FreeBSD 14 VM to FreeBSD 15. Or you can take my word, as a FreeBSD developer, that this patch is fine.

Right now if you really need to use Valgrind on FreeBSD 15 the best thing to do would be

which will be essentially the same as what I'm planning for devel/valgrind-devel with the exception that it will be untested

Right now if you really need to use Valgrind on FreeBSD 15 [...]

My goal is not to use Valgrind, my goal is to fix a broken port. At the moment, this patch is all that is needed to make devel/valgrind build and run no better or worse on FreeBSD 15 than on FreeBSD 14.

In D41610#948369, @des wrote:

When I have VMs to test on FreeBSD 15

You can easily upgrade a FreeBSD 14 VM to FreeBSD 15.

I'm not in the habit of doing that and I expect that it would be rather long on my old workstation.

Or you can take my word, as a FreeBSD developer, that this patch is fine.

I wonder how that makes you feel qualified to judge a patch for Valgrind?

In D41610#948371, @des wrote:

Right now if you really need to use Valgrind on FreeBSD 15 [...]

My goal is not to use Valgrind, my goal is to fix a broken port. At the moment, this patch is all that is needed to make devel/valgrind build and run no better or worse on FreeBSD 15 than on FreeBSD 14.

Now this is getting annoying.

You presume that it will be no worse WHEN NOBODY HAS EVEN RUN THE VALGRIND TEST SUITE.

You presume that it will be no worse WHEN NOBODY HAS EVEN RUN THE VALGRIND TEST SUITE.

FreeBSD 14 and 15 are essentially identical. We just bumped the version number last week. The test suite produces the exact same result on both:

== 813 tests, 477 stderr failures, 83 stdout failures, 11 stderrB failures, 17 stdoutB failures, 37 post failures ==

I suggest you reevaluate your attitude.

In D41610#948383, @des wrote:

You presume that it will be no worse WHEN NOBODY HAS EVEN RUN THE VALGRIND TEST SUITE.

FreeBSD 14 and 15 are essentially identical. We just bumped the version number last week. The test suite produces the exact same result on both:

== 813 tests, 477 stderr failures, 83 stdout failures, 11 stderrB failures, 17 stdoutB failures, 37 post failures ==

I suggest you reevaluate your attitude.

You're doing something badly wrong if you're getting 477 stderr failures.

Did you read README.freebsd and install gsed and gdb as it says in the "Running regression tests" section?

On 13.2 I usually get zero regtest failures, on 14.0 I would expect somewhere between 5 and 10 failures.

tcberner added a reviewer: portmgr.
tcberner added a subscriber: tcberner.

Moin moin

portmgr values the idea of upstream CI. But given the recent branching of 14 from CURRENT, this looks like a safe enough change, to go forward under the "just fix it " blanket.

mfg Tobias/portmgr

This revision is now accepted and ready to land.Aug 28 2023, 2:54 PM

Moin moin

portmgr values the idea of upstream CI. But given the recent branching of 14 from CURRENT, this looks like a safe enough change, to go forward under the "just fix it " blanket.

mfg Tobias/portmgr

Moin moin

portmgr values the idea of upstream CI. But given the recent branching of 14 from CURRENT, this looks like a safe enough change, to go forward under the "just fix it " blanket.

mfg Tobias/portmgr

So basically being port maintainer means nothing here?

So basically being port maintainer means nothing here?

Being the port maintainer means the same everywhere, let me quote the Porter's Handbook for you:

A maintainer volunteers to keep a port in good working order. Maintainers have the primary responsibility for their ports, but not exclusive ownership. Ports exist for the benefit of the community and, in reality, belong to the community. What this means is that people other than the maintainer can make changes to a port. Large changes to the Ports Collection might require changes to many ports. The FreeBSD Ports Management Team or members of other teams might modify ports to fix dependency issues or other problems, like a version bump for a shared library update.

With a bit of luck it'll work reasonably well.