git: eb6c9c8c4aa5 - main - sysutils/cpu-x@ncurses: unbreak the port's packaging under Poudriere
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 08 Apr 2023 13:06:24 UTC
The branch main has been updated by danfe:
URL: https://cgit.FreeBSD.org/ports/commit/?id=eb6c9c8c4aa5981078c6b9b69527ecdaee5f22c9
commit eb6c9c8c4aa5981078c6b9b69527ecdaee5f22c9
Author: Alexey Dokuchaev <danfe@FreeBSD.org>
AuthorDate: 2023-04-08 13:05:44 +0000
Commit: Alexey Dokuchaev <danfe@FreeBSD.org>
CommitDate: 2023-04-08 13:05:44 +0000
sysutils/cpu-x@ncurses: unbreak the port's packaging under Poudriere
Back in 2020 diizzy@ had reported that daemon was installed in the wrong
path when the port was built under Poudriere without X11 (GTK+) support.
This seemed totally odd (and not reproducible in the Tinderbox) so I've
scratched my head and put it off for better times.
When the port was converted to flavors, the problem had resurfaced: now
the ncurses flavor was failing on the cluster!
Looking at the CMake code, it calls `include(GNUInstallDirs)' which
correctly sets CMAKE_INSTALL_FULL_LIBEXECDIR to `/usr/local/libexec'.
The code then checks for its existence, the failure logs indicate that
it does not exist, which should not be possible because it is part of
the `Templates/BSD.local.dist', so it should be mtree'd inside the build
jail. Tinderbox does exactly this, but not Poudriere, so unless this
directory is created (populated) by the dependencies or e.g. ccache(1),
one cannot assume that it really exists in pristine environment.
Regardless of whether mtree'ing jail with `BSD.local.dist' is the right
thing or not, checking for some common path existence and setting it to
something else if it's not found is definitely wrong.
This was a joint investigation by fluffy@ and yours truly.
PR: 251785
---
sysutils/cpu-x/files/patch-CMakeLists.txt | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/sysutils/cpu-x/files/patch-CMakeLists.txt b/sysutils/cpu-x/files/patch-CMakeLists.txt
new file mode 100644
index 000000000000..61c772ba735f
--- /dev/null
+++ b/sysutils/cpu-x/files/patch-CMakeLists.txt
@@ -0,0 +1,13 @@
+--- CMakeLists.txt.orig 2023-04-02 10:14:25 UTC
++++ CMakeLists.txt
+@@ -14,10 +14,6 @@ project(cpu-x
+ # Global variables
+ set(CMAKE_MODULE_PATH "${CMAKE_SOURCE_DIR}/cmake")
+ include(GNUInstallDirs)
+-if(NOT EXISTS ${CMAKE_INSTALL_FULL_LIBEXECDIR})
+- set(CMAKE_INSTALL_LIBEXECDIR lib/${CMAKE_PROJECT_NAME})
+- set(CMAKE_INSTALL_FULL_LIBEXECDIR ${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBEXECDIR})
+-endif(NOT EXISTS ${CMAKE_INSTALL_FULL_LIBEXECDIR})
+ set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/output/bin/)
+ set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/output/lib/)
+ set(CMAKE_DATA_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/output/data/)