Bug 242237 - [NEW PORT] devel/libsysctlmibinfo2: sysctl MIB info API version 2
Summary: [NEW PORT] devel/libsysctlmibinfo2: sysctl MIB info API version 2
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Tobias C. Berner
URL:
Keywords:
Depends on: 242068
Blocks: 242239
  Show dependency tree / graph
 
Reported: 2019-11-25 22:15 UTC by Alfonso S. Siciliano
Modified: 2019-12-14 14:39 UTC (History)
2 users (show)

See Also:
tobik: maintainer-feedback+

Attachments
new port .shar (3.13 KB, text/plain)
2019-11-25 22:15 UTC, Alfonso S. Siciliano 		no flags Details
new port .shar (mainstream fix) (3.13 KB, patch)
2019-11-25 23:42 UTC, Alfonso S. Siciliano 		asiciliano: maintainer-approval+
Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alfonso S. Siciliano freebsd_committer freebsd_triage 2019-11-25 22:15:04 UTC 
Created attachment 209427 [details]
new port .shar

The sysctlmibinfo2 library provides an API to explore the sysctl MIB and to get the info of an object, it is useful to build a sysctl(8)-like tool.

WWW: https://gitlab.com/alfix/sysctlmibinfo2/

Testing
 * portlint -A
 * port test
 * poudriere testport -j 130Camd64 -p default -o devel/libsysctlmibinfo2
 * poudriere testport -j 120Ramd64 -p default -o devel/libsysctlmibinfo2
 * poudriere testport -j 113Ramd64 -p default -o devel/libsysctlmibinfo2


Thanks,
Alfonso
Comment 1 Alfonso S. Siciliano freebsd_committer freebsd_triage 2019-11-25 23:42:14 UTC 
Created attachment 209431 [details]
new port .shar (mainstream fix)
Comment 2 Tobias C. Berner freebsd_committer freebsd_triage 2019-12-14 11:09:36 UTC 
Committed. Thanks.
Comment 3 commit-hook freebsd_committer freebsd_triage 2019-12-14 11:09:57 UTC 
A commit references this bug:

Author: tcberner
Date: Sat Dec 14 11:09:29 UTC 2019
New revision: 520108
URL: https://svnweb.freebsd.org/changeset/ports/520108

Log:
  [NEW PORT] devel/libsysctlmibinfo2: sysctl MIB info API version 2

  The sysctlmibinfo2 library provides an API to explore the sysctl MIB and to get
  the info of an object, it is useful to build a sysctl(8)-like tool.

  WWW: https://gitlab.com/alfix/sysctlmibinfo2/

  PR:		242237
  Submitted by:	Alfonso S. Siciliano <alfix86@gmail.com>

Changes:
  head/devel/Makefile
  head/devel/libsysctlmibinfo2/
  head/devel/libsysctlmibinfo2/Makefile
  head/devel/libsysctlmibinfo2/distinfo
  head/devel/libsysctlmibinfo2/pkg-descr
Comment 4 Tobias Kortkamp freebsd_committer freebsd_triage 2019-12-14 13:18:28 UTC 
Reopen because of RUN_DEPENDS ambiguity.  In the port the first
RUN_DEPENDS is overwritten by the second one.  As a consequence the
port only depends on sysutils/sysctlbyname-improved-kmod at runtime.
Are both needed?

RUN_DEPENDS=	${KMODDIR}/sysctlinfo.ko:sysutils/sysctlinfo-kmod
RUN_DEPENDS=	${KMODDIR}/sysctlbyname_improved.ko:sysutils/sysctlbyname-improved-kmod
Comment 5 Alfonso S. Siciliano freebsd_committer freebsd_triage 2019-12-14 14:21:37 UTC 
Thank you for your report,

you are right

# make -V RUN_DEPENDS
/boot/modules/sysctlbyname_improved.ko:sysutils/sysctlbyname-improved-kmod

better:

RUN_DEPENDS=	${KMODDIR}/sysctlinfo.ko:sysutils/sysctlinfo-kmod
RUN_DEPENDS+=	${KMODDIR}/sysctlbyname_improved.ko:sysutils/sysctlbyname-improved-kmod

so:

# make -V RUN_DEPENDS
/boot/modules/sysctlinfo.ko:sysutils/sysctlinfo-kmod /boot/modules/sysctlbyname_improved.ko:sysutils/sysctlbyname-improved-kmod

I (and tests) didn't find the problem because sysctlinfo-kmod is installed also via BUILD_DEPENDS
Comment 6 Tobias Kortkamp freebsd_committer freebsd_triage 2019-12-14 14:39:05 UTC 
Thanks. Fixed.
Comment 7 commit-hook freebsd_committer freebsd_triage 2019-12-14 14:39:20 UTC 
A commit references this bug:

Author: tobik
Date: Sat Dec 14 14:38:38 UTC 2019
New revision: 520120
URL: https://svnweb.freebsd.org/changeset/ports/520120

Log:
  devel/libsysctlmibinfo2: Fix RUN_DEPENDS

  Both sysctlbyname-improved-kmod and sysctlinfo-kmod are needed at runtime

  PR:		242237
  Submitted by:	alfix86@gmail.com (maintainer)

Changes:
  head/devel/libsysctlmibinfo2/Makefile