Upstream have switched to meson so do that now before configure is
removed and try to catch bugs sooner.
This should be a non functional change.
Signed-off-by: Emmanuel Vadot <manu@FreeBSD.org>
Details
Details
Diff Detail
Diff Detail
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 31420 Build 29039: arc lint + arc unit
Event Timeline
| graphics/mesa-dri/Makefile | ||
|---|---|---|
| 85 | Why not use :ts variable modifier e.g., ${VULKAN_DRIVERS:ts,}? != pessimizes make describe (used by make index). | |
| 122 | Why not use PLIST_SUB += ARCH=${ARCH:S/amd/x86_/}? Meson simply converts amd64 to x86_64, it's not specific to Vulkan. | |
| 134 | PREFIX is the place where current port should install while LOCALBASE is where dependencies are. LOCALBASE may not be writeable e.g., when INSTALL_AS_USER is defined. Debug PREFIX != LOCALBASE with poudriere testport -P. | |
| graphics/mesa-dri/Makefile.common | ||
| 53 | Please, sort USES alphabetically. | |
| 94 | Why not use BINARY_ALIAS += llvm-config=llvm-config${LLVM_DEFAULT} like graphics/mesa-devel does? | |
| 105 | If you're only going to patch out -DGLX_USE_TLS later why not make it conditional or convert to global-dynamic. https://gitlab.freedesktop.org/myfreeweb/mesa/-/commit/e8213333ca638378eab8c7a7b959e2c6b172b6e6 | |
| graphics/mesa-dri/files/patch-meson.build | ||
| 1 | Format timestamps in patches via make makepatch. | |
| 8 | Please, bring back the rationale (as a patch comment) why this is necessary. | |
| graphics/mesa-dri/files/patch-src_util_futex.h | ||
| 1 | Why discard the original rationale? | |
| 1 | Why regress make makepatch timestamps to let portlint -C complain about? | |
Comment Actions
Remove the hacky ln and properly version the lib in the meson build system.
This was lost in the autotool->meson conversion.
Upstream is a bit different on this target but I'll submit a patch there.
Comment Actions
It looks very good. I've been quite nit-picky about the patch meta data changes.
| graphics/libosmesa/Makefile | ||
|---|---|---|
| 38–41 | Flip this conditional to only add osmesa=classic on sparc64. It's the only arch not covered by the stuff on L38, and makes it clearer that it's a sparc64 thing. | |
| graphics/libxatracker/Makefile | ||
| 40 | Any reason this is not added to the args above? | |
| graphics/mesa-dri/Makefile | ||
| 22 | Whitespace, line up -Dgallium-vdpau=true with the rest of the stuff. | |
| 32 | Do we need to keep RADEON, isn't it renamed R100? | |
| graphics/mesa-dri/Makefile.common | ||
| 89 | This should probably be flipped to special case sparc64, similar to libosmesa. | |
| graphics/mesa-dri/files/patch-src_amd_vulkan_radv__device.c | ||
| 1–19 ↗ | (On Diff #72538) | This change only changes the meta data of the patch, it's not needed. |
| graphics/mesa-dri/files/patch-src_amd_vulkan_winsys_amdgpu_radv__amdgpu__cs.c | ||
| 1–14 ↗ | (On Diff #72538) | This change only changes the meta data of the patch, it's not needed. |
| graphics/mesa-dri/files/patch-src_intel_compiler_brw__fs__bank__conflicts.cpp | ||
| 1–18 ↗ | (On Diff #72538) | This change only changes the meta data of the patch, it's not needed. |
| graphics/mesa-dri/files/patch-src_intel_vulkan_anv__allocator.c | ||
| 1–36 ↗ | (On Diff #72538) | This change only changes the meta data of the patch, it's not needed. |
| graphics/mesa-dri/files/patch-src_mapi_glapi_gen_gl__gentable.py | ||
| 1–11 ↗ | (On Diff #72538) | This change only changes the meta data of the patch, it's not needed. |
| graphics/mesa-dri/files/patch-src_mesa_main_macros.h | ||
| 2–14 | as a side note, I wonder why this wasn't needed before. | |
| graphics/mesa-dri/files/patch-src_util_u__thread.h | ||
| 1–42 ↗ | (On Diff #72538) | This change only changes the meta data of the patch, it's not needed. |
| graphics/mesa-dri/Makefile.common | ||
|---|---|---|
| 51–53 | Are gettext-tools and libtool still required? | |
| 52 | Maybe check if localbase can be dropped. Meson tracks dependencies a bit more strictly, so pkgconfig is often enough. | |
| 87 | Maybe switch to +=, so adding more to BINARY_ALIAS is not error-prone. .common in filename expects such a scenario. | |
| graphics/libxatracker/Makefile | ||
|---|---|---|
| 40 | Ok, thanks! | |