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)
Fri, Jan 24, 7:59 PM
Unknown Object (File)
Oct 30 2024, 10:00 AM
Unknown Object (File)
Oct 3 2024, 4:21 AM
Unknown Object (File)
Sep 30 2024, 4:31 AM
Unknown Object (File)
Sep 8 2024, 3:18 PM
Unknown Object (File)
Sep 8 2024, 9:07 AM
Unknown Object (File)
Sep 4 2024, 5:07 AM
Unknown Object (File)
Sep 2 2024, 4:25 AM
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"