Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 729828 - app-admin/syslog-ng-3.28.1 automagically depends on net-analyzer/net-snmp
Summary: app-admin/syslog-ng-3.28.1 automagically depends on net-analyzer/net-snmp
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Tomáš Mózes
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2020-06-27 09:28 UTC by Jeroen Roovers (RETIRED)
Modified: 2020-06-29 04:59 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jeroen Roovers (RETIRED) gentoo-dev 2020-06-27 09:28:41 UTC
While working on bug #729806 I discovered a related problem.

Currently the ebuild does this in src_configure():

        $(use_enable snmp snmp-dest)

Yet the configure script actually does only these:

 --enable-afsnmp Enable afsnmp module (default: auto)
[...]
  --with-net-snmp=path      use path to net-snmp-config script

The latter should probably be found based on path, but the former should be explicitly enabled or disabled:

--- a/app-admin/syslog-ng/syslog-ng-3.28.1-r1.ebuild
+++ b/app-admin/syslog-ng/syslog-ng-3.28.1-r1.ebuild
@@ -126,7 +126,7 @@ src_configure() {
                $(use_enable python)
                $(use_enable redis)
                $(use_enable smtp)
-               $(use_enable snmp snmp-dest)
+               $(use_enable snmp afsnmp)
                $(use_enable spoof-source)
                $(use_enable systemd)
                $(use_enable tcpd tcp-wrapper)
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2020-06-27 09:30:02 UTC
 --enable-snmp-dest Enable snmp destination (default: auto)

is apparently a holdover from 3.26.1.
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2020-06-28 07:17:07 UTC
ping
Comment 3 Tomáš Mózes 2020-06-28 19:32:44 UTC
Please commit this fix too, thanks!
Comment 4 Larry the Git Cow gentoo-dev 2020-06-28 20:45:07 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1348a904272e79f177519434af7fc9dc9036b369

commit 1348a904272e79f177519434af7fc9dc9036b369
Author:     Jeroen Roovers <jer@gentoo.org>
AuthorDate: 2020-06-28 20:43:52 +0000
Commit:     Jeroen Roovers <jer@gentoo.org>
CommitDate: 2020-06-28 20:45:03 +0000

    app-admin/syslog-ng: Fix USE=snmp configure switch
    
    Package-Manager: Portage-2.3.103, Repoman-2.3.23
    Closes: https://bugs.gentoo.org/729828
    Signed-off-by: Jeroen Roovers <jer@gentoo.org>

 .../{syslog-ng-3.28.1-r1.ebuild => syslog-ng-3.28.1-r2.ebuild}          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 5 Tomáš Mózes 2020-06-29 04:59:40 UTC
Thank you Jeroen. I don't use snmp at all, thanks for improving this for all.