Bug 250502
| Summary: | graphics/mesa-dri Patching breaks on FreeBSD 11 | ||
|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Stewart Morgan <stewart+FreeBSD-BugZilla> |
| Component: | Individual Port(s) | Assignee: | freebsd-x11 (Nobody) <x11> |
| Status: | Closed FIXED | ||
| Severity: | Affects Some People | CC: | manu, rkoberman, zeising |
| Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(x11) |
| Version: | Latest | ||
| Hardware: | Any | ||
| OS: | Any | ||
Hi! While you are probably right in that the file should be renamed, I can't reproduce any build failure, and there has been no reports of it from the cluster either. The patch will be applied for all builds, but it is not having any effect on FreeBSD >= 12, so I'll leave it as-is for now, and I'll add it to the list of things to look att for the next mesa update. Regards (In reply to Niclas Zeising from comment #1) Hi, Thanks for responding! I presume the cluster isn't building under 11.x regularly? The only place the build fails for me is within my FreeBSD 11.4 poudriere jail when that Makefile-conditional kicks in and make/patch rightly complains it can't find the nonexistant ${PATCHDIR}/extra-src_util_build__id.c file. For when you next look at this, I'd humbly suggest the solution is to either: a) rename the patch-file so it is only applied, via the Makefile's conditional, where it has an effect (FreeBSD < 12) — bonus points maybe to remove the then-redundant conditional within the patch-file too :) b) remove the Makefile conditional and rely solely on the patch-file's conditional Thanks, Stewart. 11.4 packages are regularly built (although I don't know the exact schedule). I had no problems building this in my 11.4 jail locally as well. (In reply to Stewart Morgan from comment #2) 11.4 runs for amd64 were last completed today. They may not have reached the mirrors yet. It's purely coincidental that both the quarterly and head completed on the same day. Quarterly runs typically take only a few hours, but head can take two or three days, depending on what ports need to be rebuilt. Fixed in r554140 |
Hello, As of revision 552109, this fails at the patching stage for FreeBSD 11 (well, anything less than 12) due to the addition of the the following: .if ${OSVERSION} < 1200000 EXTRA_PATCHES+= ${PATCHDIR}/extra-src_util_build__id.c .endif There is a ***patch***-src_util_build__id.c in the files directory, which also contains a "__FreeBSD__ < 12" test; was this file supposed to be renamed? Thanks, Stewart.