Page MenuHomeFreeBSD

Mk/Scripts/qa.sh: Add eigen[23] and py-qt5 in proxydeps_suggest_uses
Needs ReviewPublic

Authored by lbartoletti on Nov 16 2020, 6:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 17 2024, 3:36 PM
Unknown Object (File)
Dec 22 2023, 11:32 PM
Unknown Object (File)
Dec 2 2023, 12:58 AM
Unknown Object (File)
Jun 25 2023, 6:22 PM
Unknown Object (File)
May 4 2023, 8:07 AM
Unknown Object (File)
Apr 22 2023, 1:41 PM
Subscribers

Details

Reviewers
tcberner
Group Reviewers
O5: Ports Framework(Owns No Changed Paths)
portmgr
Summary

Hello,

kde@ would like to add eigen and py-qt5 in sugges_uses qa.sh

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 34833
Build 31864: arc lint + arc unit

Event Timeline

Mk/Scripts/qa.sh
580

hm, eigen:* likely are not working as it only walks through the shared libraries getting linked against.

Try the following for eigen

Index: Mk/bsd.sanity.mk
===================================================================
--- Mk/bsd.sanity.mk    (revision 555776)
+++ Mk/bsd.sanity.mk    (working copy)
@@ -106,6 +106,12 @@
 DEV_ERROR+=    "All LIB_DEPENDS should use the new format and start out with lib.  \(libfoo.so vs foo.so\)"
 .endif

+.for _eigen_ver in 2 3
+.  if ((defined(BUILD_DEPENDS) && ${BUILD_DEPENDS:M*eigen${_eigen_ver}}) || (defined(RUN_DEPENDS) && ${RUN_DEPENDS:M*eigen${_eigen_ver}})) && (defined(USES) && empty(USES:Meigen*))
+DEV_WARNING+=  "Use USES=eigen:${_eigen_ver} to depend on math/eigen${_eigen_ver}"
+.  endif
+.endfor
+
 .if defined(LICENSE)
 .if ${LICENSE:MBSD}
 DEV_WARNING+=  "LICENSE must not contain BSD, instead use BSD[234]CLAUSE"