Bug 241241
| Summary: | [new port] science/openhantek6022 - OpenHantek6022 v2.15 | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Thomas Spielauer <tsfbsdport> | ||||||
| Component: | Individual Port(s) | Assignee: | Dmitry Marakasov <amdmi3> | ||||||
| Status: | Closed Feedback Timeout | ||||||||
| Severity: | Affects Only Me | ||||||||
| Priority: | --- | ||||||||
| Version: | Latest | ||||||||
| Hardware: | Any | ||||||||
| OS: | Any | ||||||||
| Attachments: |
|
||||||||
|
Description
Thomas Spielauer
2019-10-14 13:07:21 UTC
- Please don't use += unless necessary (not necessary in this port at all)
- USES=cmake adds dependency on cmake and configure/build/install targets for you, so you don't need to redefined these. Install directories belong to mtree so are also created automatically.
- Please don't display any messages from targets - package users won't see them. Use pkg-message
- This submission needs GIDs entry
- What's the purpose of USES=xorg?
- This doesn't build
===> Performing out-of-source build
/bin/mkdir -p /wrkdirs/usr/ports/science/openhantek6022/work/.build
-- The C compiler identification is Clang 6.0.1
-- The CXX compiler identification is Clang 6.0.1
-- Check for working C compiler: /usr/local/libexec/ccache/cc
-- Check for working C compiler: /usr/local/libexec/ccache/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/local/libexec/ccache/c++
-- Check for working CXX compiler: /usr/local/libexec/ccache/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at /usr/local/lib/cmake/Qt5Core/Qt5CoreConfig.cmake:15 (message):
The imported target "Qt5::Core" references the file
"/usr/local/lib/qt5/bin/qmake"
but this file does not exist. Possible reasons include:
* The file was deleted, renamed, or moved to another location.
* An install or uninstall procedure did not complete successfully.
* The installation package was faulty and contained
"/usr/local/lib/cmake/Qt5Core/Qt5CoreConfigExtras.cmake"
but not all the files it references.
Call Stack (most recent call first):
/usr/local/lib/cmake/Qt5Core/Qt5CoreConfigExtras.cmake:6 (_qt5_Core_check_file_exists)
/usr/local/lib/cmake/Qt5Core/Qt5CoreConfig.cmake:178 (include)
/usr/local/lib/cmake/Qt5Gui/Qt5GuiConfig.cmake:93 (find_package)
/usr/local/lib/cmake/Qt5Widgets/Qt5WidgetsConfig.cmake:93 (find_package)
openhantek/CMakeLists.txt:3 (find_package)
-- Configuring incomplete, errors occurred!
See also "/wrkdirs/usr/ports/science/openhantek6022/work/.build/CMakeFiles/CMakeOutput.log".
*** Error code 1
Stop.
make: stopped in /usr/ports/science/openhantek6022
build of science/openhantek6022 | openhantek6022-2.15 ended at Mon Oct 14 20:25:22 MSK 2019
build time: 00:00:14
!!! build failure encountered !!!
Created attachment 208399 [details]
Portsfiles
addressing comments from 2019-10-14
Sorry for the inconvenience this is my first try of a contribution to the FreeBSD ports tree. I've tried to address all comments with a new version of the shar, tried a build on a clean system (should work now) and modified the port to not require any GIDs. Just one question (if this is allowed on this bugtracker) - if a port requires a GID should one also propose a patch for /usr/ports/GIDs or is this auto generated from somewhere elese? > Sorry for the inconvenience this is my first try of a contribution to the FreeBSD ports tree. No probs, I'm happy to explain. > I've tried to address all comments with a new version of the shar It's better now, here's the next bunch: - Please use 2 tabs indentation - it's more common and expected and doesn't waste as much free space, while there's only a single variable that doesn't fit which still doesn't justify third tab - As mentioned, you don't need BUILD_DEPENDS=cmake because it's added by USES=cmake - Depending on qmake via BUILD_DEPENDS is incorrect (and in fact breaks the build), you should use USE_QT for all Qt dependencies. Here, you need to add `qmake_build buildtools_build linguisttools_build` to it. Note that most cmake+qt using projects require at least these because these are used in cmake qt detection scripts. - (side note) you don't need to STRIP binaries installed by INSTALL_PROGRAM, as it already performs stripping as needed - As mentioned, the framework already provides do-install target which calls upstream installation procedures, and you should generally not override it. Here, it installs the binary and handles stripping - However, you can extend it, and here you may install devd config in post-install:. That's the only target code you need > tried a build on a clean system (should work now) Please do consider installing poudriere. Even clean system is not capable of catching a lot of port problems. > and modified the port to not require any GIDs. Since you still install devd config, so it does require a group. > Just one question (if this is allowed on this bugtracker) - if a port requires a GID should one also propose a patch for /usr/ports/GIDs or is this auto generated from somewhere elese? A patch is needed. You can take any free GIDs entry, but it also makes sense to add comment to corresponding UIDs entry and note that the GID is taken, see `grep 'GID used' /usr/ports/UIDs`. Any progress? Timeout |
