[{"id":1766620,"web_url":"http://patchwork.ozlabs.org/comment/1766620/","msgid":"<8d8bb3d5-6ea9-94b4-1a53-a57909b8bbc6@mind.be>","list_archive_url":null,"date":"2017-09-11T23:08:52","subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","submitter":{"id":11809,"url":"http://patchwork.ozlabs.org/api/people/11809/","name":"Arnout Vandecappelle","email":"arnout@mind.be"},"content":"On 08-09-17 15:45, Julien Floret wrote:\n> Since commit be8e32d585f3 (\"netsnmp: configurable MIB modules\"),\n> the list of MIB modules can be selected with a configuration option.\n> \n> However, there was still an hardcoded list of MIB files to exclude from\n> the target filesystem.\n> Since it is complicated to know which MIB files are necessary according\n> to the configuration, let's install all of them.\n> \n> Cc: przemyslaw <przemyslaw.wrzos@calyptech.com>\n> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>\n> Signed-off-by: Julien Floret <julien.floret@6wind.com>\n\n Looks good to me, except: shouldn't the default of\nBR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES so that all the previous BLOAT_MIBS are\nthere?\n\n Regards,\n Arnout","headers":{"Return-Path":"<buildroot-bounces@busybox.net>","X-Original-To":["incoming@patchwork.ozlabs.org","buildroot@lists.busybox.net"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","buildroot@osuosl.org"],"Authentication-Results":"ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=busybox.net\n\t(client-ip=140.211.166.133; helo=hemlock.osuosl.org;\n\tenvelope-from=buildroot-bounces@busybox.net;\n\treceiver=<UNKNOWN>)","Received":["from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xrkBm6gg7z9s7B\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 12 Sep 2017 09:09:04 +1000 (AEST)","from localhost (localhost [127.0.0.1])\n\tby hemlock.osuosl.org (Postfix) with ESMTP id 1DAF588B50;\n\tMon, 11 Sep 2017 23:09:00 +0000 (UTC)","from hemlock.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id YtgVdLaocgi6; Mon, 11 Sep 2017 23:08:58 +0000 (UTC)","from ash.osuosl.org (ash.osuosl.org [140.211.166.34])\n\tby hemlock.osuosl.org (Postfix) with ESMTP id 46F5A88B77;\n\tMon, 11 Sep 2017 23:08:58 +0000 (UTC)","from silver.osuosl.org (smtp3.osuosl.org [140.211.166.136])\n\tby ash.osuosl.org (Postfix) with ESMTP id 448EF1C25D4\n\tfor <buildroot@lists.busybox.net>;\n\tMon, 11 Sep 2017 23:08:57 +0000 (UTC)","from localhost (localhost [127.0.0.1])\n\tby silver.osuosl.org (Postfix) with ESMTP id 37E392014B\n\tfor <buildroot@lists.busybox.net>;\n\tMon, 11 Sep 2017 23:08:57 +0000 (UTC)","from silver.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id s6vWe775yX+h for <buildroot@lists.busybox.net>;\n\tMon, 11 Sep 2017 23:08:56 +0000 (UTC)","from exchange.essensium.com (220.77.144.195.ipv4.evonet.be\n\t[195.144.77.220])\n\tby silver.osuosl.org (Postfix) with ESMTP id C588B2013C\n\tfor <buildroot@buildroot.org>; Mon, 11 Sep 2017 23:08:55 +0000 (UTC)","from [10.8.0.14] (10.3.7.11) by beleexch01.local.ess-mail.com\n\t(10.3.7.8) with Microsoft SMTP Server (TLS) id 15.0.847.32;\n\tTue, 12 Sep 2017 01:08:54 +0200"],"X-Virus-Scanned":["amavisd-new at osuosl.org","amavisd-new at osuosl.org"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","To":"Julien Floret <julien.floret@6wind.com>, <buildroot@buildroot.org>","References":"<20170908134548.23645-1-julien.floret@6wind.com>","From":"Arnout Vandecappelle <arnout@mind.be>","Organization":"Essensium/Mind","Message-ID":"<8d8bb3d5-6ea9-94b4-1a53-a57909b8bbc6@mind.be>","Date":"Tue, 12 Sep 2017 01:08:52 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170908134548.23645-1-julien.floret@6wind.com>","Content-Language":"en-GB","X-Originating-IP":"[10.3.7.11]","X-ClientProxiedBy":"beleexch01.local.ess-mail.com (10.3.7.8) To\n\tbeleexch01.local.ess-mail.com (10.3.7.8)","Subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","X-BeenThere":"buildroot@busybox.net","X-Mailman-Version":"2.1.18-1","Precedence":"list","List-Id":"Discussion and development of buildroot <buildroot.busybox.net>","List-Unsubscribe":"<http://lists.busybox.net/mailman/options/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=unsubscribe>","List-Archive":"<http://lists.busybox.net/pipermail/buildroot/>","List-Post":"<mailto:buildroot@busybox.net>","List-Help":"<mailto:buildroot-request@busybox.net?subject=help>","List-Subscribe":"<http://lists.busybox.net/mailman/listinfo/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=subscribe>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"buildroot-bounces@busybox.net","Sender":"\"buildroot\" <buildroot-bounces@busybox.net>"}},{"id":1766771,"web_url":"http://patchwork.ozlabs.org/comment/1766771/","msgid":"<CAHfJR7jjCn8FdGqhKOq=7_pAgwObLNSQUdzzdbzOi+ijyvJLCw@mail.gmail.com>","list_archive_url":null,"date":"2017-09-12T08:48:10","subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","submitter":{"id":68113,"url":"http://patchwork.ozlabs.org/api/people/68113/","name":"Julien Floret","email":"julien.floret@6wind.com"},"content":"Hello Arnout,\n\n2017-09-12 1:08 GMT+02:00 Arnout Vandecappelle <arnout@mind.be>:\n>\n>\n> On 08-09-17 15:45, Julien Floret wrote:\n>> Since commit be8e32d585f3 (\"netsnmp: configurable MIB modules\"),\n>> the list of MIB modules can be selected with a configuration option.\n>>\n>> However, there was still an hardcoded list of MIB files to exclude from\n>> the target filesystem.\n>> Since it is complicated to know which MIB files are necessary according\n>> to the configuration, let's install all of them.\n>>\n>> Cc: przemyslaw <przemyslaw.wrzos@calyptech.com>\n>> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>\n>> Signed-off-by: Julien Floret <julien.floret@6wind.com>\n>\n>  Looks good to me, except: shouldn't the default of\n> BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES so that all the previous BLOAT_MIBS are\n> there?\n\nHum, maybe we could parse\nBR2_PACKAGE_NETSNMP_{WITH,WITHOUT}_MIB_MODULES to deduce BLOAT_MIBS,\nbut to me it does not seem trivial nor elegant...\nWhat about adding a new config option BR2_NETSNMP_WITHOUT_MIB_FILES\nwhose default value is the content of BLOAT_MIBS?","headers":{"Return-Path":"<buildroot-bounces@busybox.net>","X-Original-To":["incoming@patchwork.ozlabs.org","buildroot@lists.busybox.net"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","buildroot@osuosl.org"],"Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=busybox.net\n\t(client-ip=140.211.166.133; helo=hemlock.osuosl.org;\n\tenvelope-from=buildroot-bounces@busybox.net;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=6wind-com.20150623.gappssmtp.com\n\theader.i=@6wind-com.20150623.gappssmtp.com\n\theader.b=\"Qc425Are\"; dkim-atps=neutral"],"Received":["from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xrz3908ZJz9s0g\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 12 Sep 2017 18:48:20 +1000 (AEST)","from localhost (localhost [127.0.0.1])\n\tby hemlock.osuosl.org (Postfix) with ESMTP id 6D99989471;\n\tTue, 12 Sep 2017 08:48:16 +0000 (UTC)","from hemlock.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id FAbqFEl7dBSN; Tue, 12 Sep 2017 08:48:14 +0000 (UTC)","from ash.osuosl.org (ash.osuosl.org [140.211.166.34])\n\tby hemlock.osuosl.org (Postfix) with ESMTP id 3ACF389427;\n\tTue, 12 Sep 2017 08:48:14 +0000 (UTC)","from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133])\n\tby ash.osuosl.org (Postfix) with ESMTP id 01B881BF238\n\tfor <buildroot@lists.busybox.net>;\n\tTue, 12 Sep 2017 08:48:13 +0000 (UTC)","from localhost (localhost [127.0.0.1])\n\tby hemlock.osuosl.org (Postfix) with ESMTP id E52CD89426\n\tfor <buildroot@lists.busybox.net>;\n\tTue, 12 Sep 2017 08:48:12 +0000 (UTC)","from hemlock.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id jmRwwWnnKBRB for <buildroot@lists.busybox.net>;\n\tTue, 12 Sep 2017 08:48:11 +0000 (UTC)","from mail-io0-f178.google.com (mail-io0-f178.google.com\n\t[209.85.223.178])\n\tby hemlock.osuosl.org (Postfix) with ESMTPS id CF9FA89421\n\tfor <buildroot@buildroot.org>; Tue, 12 Sep 2017 08:48:11 +0000 (UTC)","by mail-io0-f178.google.com with SMTP id k101so436771iod.0\n\tfor <buildroot@buildroot.org>; Tue, 12 Sep 2017 01:48:11 -0700 (PDT)","by 10.168.70.143 with HTTP; Tue, 12 Sep 2017 01:48:10 -0700 (PDT)"],"X-Virus-Scanned":["amavisd-new at osuosl.org","amavisd-new at osuosl.org"],"X-Greylist":"from auto-whitelisted by SQLgrey-1.7.6","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=6wind-com.20150623.gappssmtp.com; s=20150623;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=E8jYkeHIPUCiH5JCjvJymsSJgNaRMxO5vDRTwCM7/0s=;\n\tb=Qc425Are5bgYcRRfL5XSOlwlrK44wb1hofr36TnrJgAoG0MUWV6rnVG2+f7ZhNd8xw\n\tsPaMK3gUV+pO2Ai/+12mPV3MSf2rp/QM2kdBjqadUfh9rZmXlV6uhHJjFhasf2XCo5DS\n\tltTnGNuUWTIsrSq5omX93/GFj2QCyXdnIbV1xz9Db/0iIIt+97Jy3bigh8+ziUKrhaxi\n\te8G4Gk27M0QK03aRgklDG67RbMeBh3Es8pJUXcgawmBkg+WJntkfrxAMiq0xZAVHUfRg\n\tRvwiUkrDF6fq6Cf6YRCGja52DqP7DP3Vhjfyk6GSvsBP3VG/fla35MhPh1036iz6H5b3\n\tmdGQ==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=E8jYkeHIPUCiH5JCjvJymsSJgNaRMxO5vDRTwCM7/0s=;\n\tb=RJCW1vXxzU7arFlXL5sDvyOiViSFerpHL/VODdXbFOCbF9RLT0d/wJQ0ZJiWfoe+qv\n\t/352Ws+j3nB6ugzPqkp6UTr2AQEEmf9gi+pGCJ0Yo6QcEWw6obBeLII6jUOi/BuPzmM0\n\tpjOSinmnIo2xREZ/zZc4V54m3j5uIoMM58iJ+lxSVhcIPbQoaNACeXQ3Ml6evftx4ux8\n\tBXzY7gwZKCqFXgZpNCOTq5ndAKALfyE92OqvasBnLX311yg5OD9yFRAWzK6f5FW8s8ED\n\tPI8+R0yl9Lfcm9Z2KU8UixInqojWs8PAn56Q1F0REjMIt8Y3X7mBTlcvbrR0htpeXl07\n\toTtg==","X-Gm-Message-State":"AHPjjUgRxo6DqxEeio3U2iDqEvomP95624HviDgj7aReFXwdwUznZLuh\n\tENKTT4bMWeoFDCHSbnO5h145PT+3ZlFXNHc=","X-Google-Smtp-Source":"AOwi7QDU9cMDfnyqvQ64C3dTCQdzrsJLJMIRZtqgCze4pItBbYr8ScY0Z4mb42em4FbvseSm5W+5Yzc/ly4/9503oDI=","X-Received":"by 10.202.168.140 with SMTP id\n\tr134mr13483423oie.232.1505206090835; \n\tTue, 12 Sep 2017 01:48:10 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<8d8bb3d5-6ea9-94b4-1a53-a57909b8bbc6@mind.be>","References":"<20170908134548.23645-1-julien.floret@6wind.com>\n\t<8d8bb3d5-6ea9-94b4-1a53-a57909b8bbc6@mind.be>","From":"Julien Floret <julien.floret@6wind.com>","Date":"Tue, 12 Sep 2017 10:48:10 +0200","Message-ID":"<CAHfJR7jjCn8FdGqhKOq=7_pAgwObLNSQUdzzdbzOi+ijyvJLCw@mail.gmail.com>","To":"Arnout Vandecappelle <arnout@mind.be>","Cc":"buildroot <buildroot@buildroot.org>","Subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","X-BeenThere":"buildroot@busybox.net","X-Mailman-Version":"2.1.18-1","Precedence":"list","List-Id":"Discussion and development of buildroot <buildroot.busybox.net>","List-Unsubscribe":"<http://lists.busybox.net/mailman/options/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=unsubscribe>","List-Archive":"<http://lists.busybox.net/pipermail/buildroot/>","List-Post":"<mailto:buildroot@busybox.net>","List-Help":"<mailto:buildroot-request@busybox.net?subject=help>","List-Subscribe":"<http://lists.busybox.net/mailman/listinfo/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=subscribe>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"buildroot-bounces@busybox.net","Sender":"\"buildroot\" <buildroot-bounces@busybox.net>"}},{"id":1766923,"web_url":"http://patchwork.ozlabs.org/comment/1766923/","msgid":"<d60e9f05-f4d2-3927-ee7f-148d6db4c22c@mind.be>","list_archive_url":null,"date":"2017-09-12T11:44:22","subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","submitter":{"id":11809,"url":"http://patchwork.ozlabs.org/api/people/11809/","name":"Arnout Vandecappelle","email":"arnout@mind.be"},"content":"On 12-09-17 10:48, Julien Floret wrote:\n> Hello Arnout,\n> \n> 2017-09-12 1:08 GMT+02:00 Arnout Vandecappelle <arnout@mind.be>:\n>>\n>>\n>> On 08-09-17 15:45, Julien Floret wrote:\n>>> Since commit be8e32d585f3 (\"netsnmp: configurable MIB modules\"),\n>>> the list of MIB modules can be selected with a configuration option.\n>>>\n>>> However, there was still an hardcoded list of MIB files to exclude from\n>>> the target filesystem.\n>>> Since it is complicated to know which MIB files are necessary according\n>>> to the configuration, let's install all of them.\n>>>\n>>> Cc: przemyslaw <przemyslaw.wrzos@calyptech.com>\n>>> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>\n>>> Signed-off-by: Julien Floret <julien.floret@6wind.com>\n>>\n>>  Looks good to me, except: shouldn't the default of\n>> BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES so that all the previous BLOAT_MIBS are\n>> there?\n> \n> Hum, maybe we could parse\n> BR2_PACKAGE_NETSNMP_{WITH,WITHOUT}_MIB_MODULES to deduce BLOAT_MIBS,\n> but to me it does not seem trivial nor elegant...\n> What about adding a new config option BR2_NETSNMP_WITHOUT_MIB_FILES\n> whose default value is the content of BLOAT_MIBS?\n\n Let me rephrase my comment.\n\n What this patch does, basically, is to clean up the discrepancy between\nBR2_PACKAGE_NETSNMP_{WITH,WITHOUT}_MIB_MODULES and BLOAT_MIBS. That is\nabsolutely a good thing.\n\n It looks to me like the current BLOAT_MIBS option is removing stuff that is\nalready disabled by the default value of\nBR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES. So that part is fine - your patch is\nnot actually changing anything in the default configuration.\n\n However, it looks to me like the current BLOAT_MIBS is also removing things\nthat are *not* included in the current default value of\nBR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES. Therefore, after this change, we end up\nwith a bigger rootfs (using defaults) than before.\n\n My suggestion is: wouldn't it make sense to extend the default value of\nBR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES? As far as I can see, the current\ndefault removes DISMAN-EVENT and DISMAN-SCHEDULE already. Are there additional\nmodules we can disable, so that also BRIDGE DISMAN-SCRIPT EtherLike RFC-1215\nRFC1155-SMI RFC1213 SCTP and SMUX are gone? And can we do this without loosing\nfunctionality that we did have before?\n\n Probably extending BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES should be a separate\npatch anyway.\n\n Regards,\n Arnout","headers":{"Return-Path":"<buildroot-bounces@busybox.net>","X-Original-To":["incoming@patchwork.ozlabs.org","buildroot@lists.busybox.net"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","buildroot@osuosl.org"],"Authentication-Results":"ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=busybox.net\n\t(client-ip=140.211.166.137; helo=fraxinus.osuosl.org;\n\tenvelope-from=buildroot-bounces@busybox.net;\n\treceiver=<UNKNOWN>)","Received":["from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xs2zD4rpWz9s5L\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 12 Sep 2017 21:45:04 +1000 (AEST)","from localhost (localhost [127.0.0.1])\n\tby fraxinus.osuosl.org (Postfix) with ESMTP id B065F84828;\n\tTue, 12 Sep 2017 11:45:01 +0000 (UTC)","from fraxinus.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id 2oV9jR4zEczA; Tue, 12 Sep 2017 11:44:59 +0000 (UTC)","from ash.osuosl.org (ash.osuosl.org [140.211.166.34])\n\tby fraxinus.osuosl.org (Postfix) with ESMTP id 08798841E7;\n\tTue, 12 Sep 2017 11:44:59 +0000 (UTC)","from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133])\n\tby ash.osuosl.org (Postfix) with ESMTP id 9362A1BF238\n\tfor <buildroot@lists.busybox.net>;\n\tTue, 12 Sep 2017 11:44:57 +0000 (UTC)","from localhost (localhost [127.0.0.1])\n\tby hemlock.osuosl.org (Postfix) with ESMTP id 8E1D58885A\n\tfor <buildroot@lists.busybox.net>;\n\tTue, 12 Sep 2017 11:44:57 +0000 (UTC)","from hemlock.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id UHLHQIrzkASL for <buildroot@lists.busybox.net>;\n\tTue, 12 Sep 2017 11:44:56 +0000 (UTC)","from exchange.essensium.com (220.77.144.195.ipv4.evonet.be\n\t[195.144.77.220])\n\tby hemlock.osuosl.org (Postfix) with ESMTP id D712188856\n\tfor <buildroot@buildroot.org>; Tue, 12 Sep 2017 11:44:54 +0000 (UTC)","from [10.8.0.14] (10.3.7.11) by beleexch01.local.ess-mail.com\n\t(10.3.7.8) with Microsoft SMTP Server (TLS) id 15.0.847.32;\n\tTue, 12 Sep 2017 13:44:22 +0200"],"X-Virus-Scanned":["amavisd-new at osuosl.org","amavisd-new at osuosl.org"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","To":"Julien Floret <julien.floret@6wind.com>","References":"<20170908134548.23645-1-julien.floret@6wind.com>\n\t<8d8bb3d5-6ea9-94b4-1a53-a57909b8bbc6@mind.be>\n\t<CAHfJR7jjCn8FdGqhKOq=7_pAgwObLNSQUdzzdbzOi+ijyvJLCw@mail.gmail.com>","From":"Arnout Vandecappelle <arnout@mind.be>","Organization":"Essensium/Mind","Message-ID":"<d60e9f05-f4d2-3927-ee7f-148d6db4c22c@mind.be>","Date":"Tue, 12 Sep 2017 13:44:22 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<CAHfJR7jjCn8FdGqhKOq=7_pAgwObLNSQUdzzdbzOi+ijyvJLCw@mail.gmail.com>","Content-Language":"en-GB","X-Originating-IP":"[10.3.7.11]","X-ClientProxiedBy":"beleexch01.local.ess-mail.com (10.3.7.8) To\n\tbeleexch01.local.ess-mail.com (10.3.7.8)","Cc":"buildroot <buildroot@buildroot.org>","Subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","X-BeenThere":"buildroot@busybox.net","X-Mailman-Version":"2.1.18-1","Precedence":"list","List-Id":"Discussion and development of buildroot <buildroot.busybox.net>","List-Unsubscribe":"<http://lists.busybox.net/mailman/options/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=unsubscribe>","List-Archive":"<http://lists.busybox.net/pipermail/buildroot/>","List-Post":"<mailto:buildroot@busybox.net>","List-Help":"<mailto:buildroot-request@busybox.net?subject=help>","List-Subscribe":"<http://lists.busybox.net/mailman/listinfo/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=subscribe>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"buildroot-bounces@busybox.net","Sender":"\"buildroot\" <buildroot-bounces@busybox.net>"}},{"id":1767840,"web_url":"http://patchwork.ozlabs.org/comment/1767840/","msgid":"<CAHfJR7jTT1tGBQr-rgZ+JTZQW6rEhagyDJBQAfB0yUPJwh_evw@mail.gmail.com>","list_archive_url":null,"date":"2017-09-13T12:16:26","subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","submitter":{"id":68113,"url":"http://patchwork.ozlabs.org/api/people/68113/","name":"Julien Floret","email":"julien.floret@6wind.com"},"content":"2017-09-12 13:44 GMT+02:00 Arnout Vandecappelle <arnout@mind.be>:\n>\n>\n> On 12-09-17 10:48, Julien Floret wrote:\n>> Hello Arnout,\n>>\n>> 2017-09-12 1:08 GMT+02:00 Arnout Vandecappelle <arnout@mind.be>:\n>>>\n>>>\n>>> On 08-09-17 15:45, Julien Floret wrote:\n>>>> Since commit be8e32d585f3 (\"netsnmp: configurable MIB modules\"),\n>>>> the list of MIB modules can be selected with a configuration option.\n>>>>\n>>>> However, there was still an hardcoded list of MIB files to exclude from\n>>>> the target filesystem.\n>>>> Since it is complicated to know which MIB files are necessary according\n>>>> to the configuration, let's install all of them.\n>>>>\n>>>> Cc: przemyslaw <przemyslaw.wrzos@calyptech.com>\n>>>> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>\n>>>> Signed-off-by: Julien Floret <julien.floret@6wind.com>\n>>>\n>>>  Looks good to me, except: shouldn't the default of\n>>> BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES so that all the previous BLOAT_MIBS are\n>>> there?\n>>\n>> Hum, maybe we could parse\n>> BR2_PACKAGE_NETSNMP_{WITH,WITHOUT}_MIB_MODULES to deduce BLOAT_MIBS,\n>> but to me it does not seem trivial nor elegant...\n>> What about adding a new config option BR2_NETSNMP_WITHOUT_MIB_FILES\n>> whose default value is the content of BLOAT_MIBS?\n>\n>  Let me rephrase my comment.\n>\n>  What this patch does, basically, is to clean up the discrepancy between\n> BR2_PACKAGE_NETSNMP_{WITH,WITHOUT}_MIB_MODULES and BLOAT_MIBS. That is\n> absolutely a good thing.\n>\n>  It looks to me like the current BLOAT_MIBS option is removing stuff that is\n> already disabled by the default value of\n> BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES. So that part is fine - your patch is\n> not actually changing anything in the default configuration.\n>\n>  However, it looks to me like the current BLOAT_MIBS is also removing things\n> that are *not* included in the current default value of\n> BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES. Therefore, after this change, we end up\n> with a bigger rootfs (using defaults) than before.\n>\n>  My suggestion is: wouldn't it make sense to extend the default value of\n> BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES? As far as I can see, the current\n> default removes DISMAN-EVENT and DISMAN-SCHEDULE already. Are there additional\n> modules we can disable, so that also BRIDGE DISMAN-SCRIPT EtherLike RFC-1215\n> RFC1155-SMI RFC1213 SCTP and SMUX are gone? And can we do this without loosing\n> functionality that we did have before?\n\nThanks for the detailed response, now I understand better your comment.\n\nHonestly, I could not tell... netsnmp is quite complicated. Ensuring\nwe don't lose any functionality after changing the default modules\nwould probably require a deeper investigation and knowledge of netsnmp\ncode that I'm not able to do right now. And it seems netsnmp \"make\ninstall\" installs MIB description files, whether or not the module(s)\nusing them are enabled or not.\nWhat's more, if I recall correctly, modules can still work without\ntheir MIB description file - they just cannot translate object IDs\ninto human-readable names.\n\nBut without this patch, netsnmp is unusable for us, because we need\nthe object names and this hardcoded BLOAT_MIBS list removes mandatory\nMIB files whatever the configuration options.\n(By the way, some of the MIBS in BLOAT_MIBS (RFC-1215 and RFC1155-SMI)\ndon't seem to be suppressed, because the suffix added in\nNETSNMP_REMOVE_BLOAT_MIBS (\"-MIB.txt\") doesn't apply for them...)\n\n>  Probably extending BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES should be a separate\n> patch anyway.\n\nYes, probably. I'm not sure these default values are the most common\ncase, however.\nAlso, IMHO having two options {WITH,WITHOUT}_MIB_MODULES makes quite\ndifficult to know which modules are really enabled in the end.\nI'm starting to think that maybe a better approach would be to have\none Config.in option per module... This would be clearer and would\nperhaps also allow removing unused MIB files according to the modules\nthat are enabled or not.\n\nSo do you think this patch is acceptable for now?","headers":{"Return-Path":"<buildroot-bounces@busybox.net>","X-Original-To":["incoming@patchwork.ozlabs.org","buildroot@lists.busybox.net"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","buildroot@osuosl.org"],"Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=busybox.net\n\t(client-ip=140.211.166.137; helo=fraxinus.osuosl.org;\n\tenvelope-from=buildroot-bounces@busybox.net;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=6wind-com.20150623.gappssmtp.com\n\theader.i=@6wind-com.20150623.gappssmtp.com\n\theader.b=\"OB3cvsrx\"; dkim-atps=neutral"],"Received":["from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xsgmG5rlRz9s72\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 13 Sep 2017 22:22:54 +1000 (AEST)","from localhost (localhost [127.0.0.1])\n\tby fraxinus.osuosl.org (Postfix) with ESMTP id 4A58587E46;\n\tWed, 13 Sep 2017 12:22:50 +0000 (UTC)","from fraxinus.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id gvlfD57bH45W; Wed, 13 Sep 2017 12:22:47 +0000 (UTC)","from ash.osuosl.org (ash.osuosl.org [140.211.166.34])\n\tby fraxinus.osuosl.org (Postfix) with ESMTP id 6749687D5B;\n\tWed, 13 Sep 2017 12:22:47 +0000 (UTC)","from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138])\n\tby ash.osuosl.org (Postfix) with ESMTP id 1765C1C3F96\n\tfor <buildroot@lists.busybox.net>;\n\tWed, 13 Sep 2017 12:22:46 +0000 (UTC)","from localhost (localhost [127.0.0.1])\n\tby whitealder.osuosl.org (Postfix) with ESMTP id 0DE95887A9\n\tfor <buildroot@lists.busybox.net>;\n\tWed, 13 Sep 2017 12:22:46 +0000 (UTC)","from whitealder.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id 6fNR3v4+JSIl for <buildroot@lists.busybox.net>;\n\tWed, 13 Sep 2017 12:22:44 +0000 (UTC)","from mail-oi0-f45.google.com (mail-oi0-f45.google.com\n\t[209.85.218.45])\n\tby whitealder.osuosl.org (Postfix) with ESMTPS id CDEBB8875B\n\tfor <buildroot@buildroot.org>; Wed, 13 Sep 2017 12:22:44 +0000 (UTC)","by mail-oi0-f45.google.com with SMTP id l74so327211oih.1\n\tfor <buildroot@buildroot.org>; Wed, 13 Sep 2017 05:22:44 -0700 (PDT)","by 10.168.71.130 with HTTP; Wed, 13 Sep 2017 05:16:26 -0700 (PDT)"],"X-Virus-Scanned":["amavisd-new at osuosl.org","amavisd-new at osuosl.org"],"X-Greylist":"delayed 00:06:16 by SQLgrey-1.7.6","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=6wind-com.20150623.gappssmtp.com; s=20150623;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=ZKGHj8CiGO+510jYFWv05+NYLljjGfK0d4u+m9WgVlo=;\n\tb=OB3cvsrxXAN7C+ZNU4u+RaW0DlLimMr/uYNi2JQEQ0Q4tkUZkWQkceqYbQR/ANfclB\n\tCEhs/ojI9BKARDZuPAHLauk7Lfhn+XeUOAQZj97YutEA+KIwq4e6i9tIPI9cmnslHrI5\n\tS5ZQrdvtf+rp0sBpVH4p6j/kEC6nnmH0vb85g7wmk9BdUCawxfVdIg7BNvpGJlkKDcoF\n\t2rOLtW9unODExf8qhqpFoIW+pI62geIJyUmd+dfRbvZNdFqdTSxXkoSXML+fRZ1c8Ed+\n\tgqVcdF1fk2Q6UXE9PMmHB/IETt0A4KW+2HMVckOgk5Pk5e3g2PNukByKh7oy5FpISJGJ\n\tt3aQ==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=ZKGHj8CiGO+510jYFWv05+NYLljjGfK0d4u+m9WgVlo=;\n\tb=gzJFukVWOaymZE4jjV3MtEA1hAInadmmK0h4zXlAStphwTse2Ys6me2m0PpF4l/1Bt\n\tn/eBkVnOXExpmmFSzYF+sg0xNRGSNvjfsk8IXcoDBkw0cpjKXM7ieD3YFZppxA5PebCT\n\tQwa0l8AAs031+Zi8du5NwnXUi2soRvsxEuAKL1dP2u1zR9FJpK+d8SvTy6yMrdupproV\n\t6o93KYkVwAYe4HGaA+HL2Mm0kme2sh97PzAaqauPqcP417y+anUkQn8s099eUiRy2jC2\n\tAPk9PD0f510Ya8ptr+xh35znB1k5egPxwRSn0eIeIZ4T2+dZTATgimWPqBq0laoF3sl8\n\tUPvg==","X-Gm-Message-State":"AHPjjUjXANKcKUm33eTXA0SrujNgsADRDVFcYOrjc4Nom0N19EHVQkSw\n\tuC2W1wGBL7W6qp646KypA8KcLVCpZPNNWk0yOzw3QA==","X-Google-Smtp-Source":"AOwi7QDjfEBLFYwhi2MsxJxqqX/r6QMYuj3lYd7/Ij4q+3q0qmG3gPEEVXmbWNpe6QKaf21J5QV1vlBrUpzjlmlVq0A=","X-Received":"by 10.202.196.197 with SMTP id\n\tu188mr17307437oif.228.1505304987504; \n\tWed, 13 Sep 2017 05:16:27 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<d60e9f05-f4d2-3927-ee7f-148d6db4c22c@mind.be>","References":"<20170908134548.23645-1-julien.floret@6wind.com>\n\t<8d8bb3d5-6ea9-94b4-1a53-a57909b8bbc6@mind.be>\n\t<CAHfJR7jjCn8FdGqhKOq=7_pAgwObLNSQUdzzdbzOi+ijyvJLCw@mail.gmail.com>\n\t<d60e9f05-f4d2-3927-ee7f-148d6db4c22c@mind.be>","From":"Julien Floret <julien.floret@6wind.com>","Date":"Wed, 13 Sep 2017 14:16:26 +0200","Message-ID":"<CAHfJR7jTT1tGBQr-rgZ+JTZQW6rEhagyDJBQAfB0yUPJwh_evw@mail.gmail.com>","To":"Arnout Vandecappelle <arnout@mind.be>","Cc":"buildroot <buildroot@buildroot.org>","Subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","X-BeenThere":"buildroot@busybox.net","X-Mailman-Version":"2.1.18-1","Precedence":"list","List-Id":"Discussion and development of buildroot <buildroot.busybox.net>","List-Unsubscribe":"<http://lists.busybox.net/mailman/options/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=unsubscribe>","List-Archive":"<http://lists.busybox.net/pipermail/buildroot/>","List-Post":"<mailto:buildroot@busybox.net>","List-Help":"<mailto:buildroot-request@busybox.net?subject=help>","List-Subscribe":"<http://lists.busybox.net/mailman/listinfo/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=subscribe>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"buildroot-bounces@busybox.net","Sender":"\"buildroot\" <buildroot-bounces@busybox.net>"}},{"id":1768192,"web_url":"http://patchwork.ozlabs.org/comment/1768192/","msgid":"<f14a15ee-4bc9-ffc0-38f1-fd15ebe43790@mind.be>","list_archive_url":null,"date":"2017-09-13T21:24:55","subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","submitter":{"id":11809,"url":"http://patchwork.ozlabs.org/api/people/11809/","name":"Arnout Vandecappelle","email":"arnout@mind.be"},"content":"On 13-09-17 14:16, Julien Floret wrote:\n[snip]\n> Honestly, I could not tell... netsnmp is quite complicated. Ensuring\n> we don't lose any functionality after changing the default modules\n> would probably require a deeper investigation and knowledge of netsnmp\n> code that I'm not able to do right now. And it seems netsnmp \"make\n> install\" installs MIB description files, whether or not the module(s)\n> using them are enabled or not.\n> What's more, if I recall correctly, modules can still work without\n> their MIB description file - they just cannot translate object IDs\n> into human-readable names.\n\n OK. Well, the only thing this patch does, in the end, is make the target\nfilesystem a bit bigger. By how much, do you know?\n\n Users can always remove unneeded MIBs in a post-build script.\n\n\n> But without this patch, netsnmp is unusable for us, because we need\n> the object names and this hardcoded BLOAT_MIBS list removes mandatory\n> MIB files whatever the configuration options.\n\n That's a good reason!\n\n> (By the way, some of the MIBS in BLOAT_MIBS (RFC-1215 and RFC1155-SMI)\n> don't seem to be suppressed, because the suffix added in\n> NETSNMP_REMOVE_BLOAT_MIBS (\"-MIB.txt\") doesn't apply for them...)\n\n Yeah, the option is from the very first time the package was added 7 years ago,\nand hasn't been updated through all the version bumps...\n\n\n>>  Probably extending BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES should be a separate\n>> patch anyway.\n> \n> Yes, probably. I'm not sure these default values are the most common\n> case, however.\n\n Never mind that, since it's anyway not really related, as you say.\n\n\n> Also, IMHO having two options {WITH,WITHOUT}_MIB_MODULES makes quite\n> difficult to know which modules are really enabled in the end.\n> I'm starting to think that maybe a better approach would be to have\n> one Config.in option per module... This would be clearer and would\n> perhaps also allow removing unused MIB files according to the modules\n> that are enabled or not.\n\n But that would make for a really long list of options, no? I think that that's\na bit too much.\n\n\n> \n> So do you think this patch is acceptable for now?\n\n I've applied, thanks.\n\n Perhaps you could write up a CHANGES entry that explains that people should\nremove unneeded MIBs in a post-build script?\n\n Regards,\n Arnout","headers":{"Return-Path":"<buildroot-bounces@busybox.net>","X-Original-To":["incoming@patchwork.ozlabs.org","buildroot@lists.busybox.net"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","buildroot@osuosl.org"],"Authentication-Results":"ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=busybox.net\n\t(client-ip=140.211.166.138; helo=whitealder.osuosl.org;\n\tenvelope-from=buildroot-bounces@busybox.net;\n\treceiver=<UNKNOWN>)","Received":["from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xsvp43xVLz9s81\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 14 Sep 2017 07:25:16 +1000 (AEST)","from localhost (localhost [127.0.0.1])\n\tby whitealder.osuosl.org (Postfix) with ESMTP id C173089719;\n\tWed, 13 Sep 2017 21:25:14 +0000 (UTC)","from whitealder.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id 3900hTMBwCO7; Wed, 13 Sep 2017 21:25:13 +0000 (UTC)","from ash.osuosl.org (ash.osuosl.org [140.211.166.34])\n\tby whitealder.osuosl.org (Postfix) with ESMTP id AC1128971B;\n\tWed, 13 Sep 2017 21:25:13 +0000 (UTC)","from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138])\n\tby ash.osuosl.org (Postfix) with ESMTP id 129741C3F94\n\tfor <buildroot@lists.busybox.net>;\n\tWed, 13 Sep 2017 21:25:12 +0000 (UTC)","from localhost (localhost [127.0.0.1])\n\tby whitealder.osuosl.org (Postfix) with ESMTP id 0BC588971B\n\tfor <buildroot@lists.busybox.net>;\n\tWed, 13 Sep 2017 21:25:12 +0000 (UTC)","from whitealder.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id bmEvyXJs3o4U for <buildroot@lists.busybox.net>;\n\tWed, 13 Sep 2017 21:25:11 +0000 (UTC)","from exchange.essensium.com (220.77.144.195.ipv4.evonet.be\n\t[195.144.77.220])\n\tby whitealder.osuosl.org (Postfix) with ESMTP id 0ECAF89719\n\tfor <buildroot@buildroot.org>; Wed, 13 Sep 2017 21:25:11 +0000 (UTC)","from [10.8.0.14] (10.3.7.11) by beleexch01.local.ess-mail.com\n\t(10.3.7.8) with Microsoft SMTP Server (TLS) id 15.0.847.32;\n\tWed, 13 Sep 2017 23:24:57 +0200"],"X-Virus-Scanned":["amavisd-new at osuosl.org","amavisd-new at osuosl.org"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","To":"Julien Floret <julien.floret@6wind.com>","References":"<20170908134548.23645-1-julien.floret@6wind.com>\n\t<8d8bb3d5-6ea9-94b4-1a53-a57909b8bbc6@mind.be>\n\t<CAHfJR7jjCn8FdGqhKOq=7_pAgwObLNSQUdzzdbzOi+ijyvJLCw@mail.gmail.com>\n\t<d60e9f05-f4d2-3927-ee7f-148d6db4c22c@mind.be>\n\t<CAHfJR7jTT1tGBQr-rgZ+JTZQW6rEhagyDJBQAfB0yUPJwh_evw@mail.gmail.com>","From":"Arnout Vandecappelle <arnout@mind.be>","Organization":"Essensium/Mind","Message-ID":"<f14a15ee-4bc9-ffc0-38f1-fd15ebe43790@mind.be>","Date":"Wed, 13 Sep 2017 23:24:55 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<CAHfJR7jTT1tGBQr-rgZ+JTZQW6rEhagyDJBQAfB0yUPJwh_evw@mail.gmail.com>","Content-Language":"en-GB","X-Originating-IP":"[10.3.7.11]","X-ClientProxiedBy":"beleexch01.local.ess-mail.com (10.3.7.8) To\n\tbeleexch01.local.ess-mail.com (10.3.7.8)","Cc":"buildroot <buildroot@buildroot.org>","Subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","X-BeenThere":"buildroot@busybox.net","X-Mailman-Version":"2.1.18-1","Precedence":"list","List-Id":"Discussion and development of buildroot <buildroot.busybox.net>","List-Unsubscribe":"<http://lists.busybox.net/mailman/options/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=unsubscribe>","List-Archive":"<http://lists.busybox.net/pipermail/buildroot/>","List-Post":"<mailto:buildroot@busybox.net>","List-Help":"<mailto:buildroot-request@busybox.net?subject=help>","List-Subscribe":"<http://lists.busybox.net/mailman/listinfo/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=subscribe>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"buildroot-bounces@busybox.net","Sender":"\"buildroot\" <buildroot-bounces@busybox.net>"}},{"id":1768554,"web_url":"http://patchwork.ozlabs.org/comment/1768554/","msgid":"<CAHfJR7gj=RGjP7VHFzwET_5OD6u6XbdpvO1A5uDxaUjKtotdtw@mail.gmail.com>","list_archive_url":null,"date":"2017-09-14T12:22:59","subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","submitter":{"id":68113,"url":"http://patchwork.ozlabs.org/api/people/68113/","name":"Julien Floret","email":"julien.floret@6wind.com"},"content":"2017-09-13 23:24 GMT+02:00 Arnout Vandecappelle <arnout@mind.be>:\n[snip]\n>  OK. Well, the only thing this patch does, in the end, is make the target\n> filesystem a bit bigger. By how much, do you know?\n\nApproximately 400K...\n\n>  Users can always remove unneeded MIBs in a post-build script.\n>\n>\n>> But without this patch, netsnmp is unusable for us, because we need\n>> the object names and this hardcoded BLOAT_MIBS list removes mandatory\n>> MIB files whatever the configuration options.\n>\n>  That's a good reason!\n>\n>> (By the way, some of the MIBS in BLOAT_MIBS (RFC-1215 and RFC1155-SMI)\n>> don't seem to be suppressed, because the suffix added in\n>> NETSNMP_REMOVE_BLOAT_MIBS (\"-MIB.txt\") doesn't apply for them...)\n>\n>  Yeah, the option is from the very first time the package was added 7 years ago,\n> and hasn't been updated through all the version bumps...\n>\n>\n>>>  Probably extending BR2_PACKAGE_NETSNMP_WITHOUT_MIB_MODULES should be a separate\n>>> patch anyway.\n>>\n>> Yes, probably. I'm not sure these default values are the most common\n>> case, however.\n>\n>  Never mind that, since it's anyway not really related, as you say.\n>\n>\n>> Also, IMHO having two options {WITH,WITHOUT}_MIB_MODULES makes quite\n>> difficult to know which modules are really enabled in the end.\n>> I'm starting to think that maybe a better approach would be to have\n>> one Config.in option per module... This would be clearer and would\n>> perhaps also allow removing unused MIB files according to the modules\n>> that are enabled or not.\n>\n>  But that would make for a really long list of options, no? I think that that's\n> a bit too much.\n>\n\nThat's right, that would make maybe 20 options... OK, it's probably\nbetter to stick to the current options.\n\n>\n>>\n>> So do you think this patch is acceptable for now?\n>\n>  I've applied, thanks.\n>\n>  Perhaps you could write up a CHANGES entry that explains that people should\n> remove unneeded MIBs in a post-build script?\n\nOK, I will see what I can do. Thanks!","headers":{"Return-Path":"<buildroot-bounces@busybox.net>","X-Original-To":["incoming@patchwork.ozlabs.org","buildroot@lists.busybox.net"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","buildroot@osuosl.org"],"Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=busybox.net\n\t(client-ip=140.211.166.136; helo=silver.osuosl.org;\n\tenvelope-from=buildroot-bounces@busybox.net;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=6wind-com.20150623.gappssmtp.com\n\theader.i=@6wind-com.20150623.gappssmtp.com\n\theader.b=\"LDOl3Duw\"; dkim-atps=neutral"],"Received":["from silver.osuosl.org (smtp3.osuosl.org [140.211.166.136])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xtHk807hnz9sP1\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 14 Sep 2017 22:23:11 +1000 (AEST)","from localhost (localhost [127.0.0.1])\n\tby silver.osuosl.org (Postfix) with ESMTP id D8B2030FE2;\n\tThu, 14 Sep 2017 12:23:05 +0000 (UTC)","from silver.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id fxZuf2XmLjVH; Thu, 14 Sep 2017 12:23:03 +0000 (UTC)","from ash.osuosl.org (ash.osuosl.org [140.211.166.34])\n\tby silver.osuosl.org (Postfix) with ESMTP id 88E0630CA0;\n\tThu, 14 Sep 2017 12:23:03 +0000 (UTC)","from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138])\n\tby ash.osuosl.org (Postfix) with ESMTP id 413F71C16C1\n\tfor <buildroot@lists.busybox.net>;\n\tThu, 14 Sep 2017 12:23:02 +0000 (UTC)","from localhost (localhost [127.0.0.1])\n\tby whitealder.osuosl.org (Postfix) with ESMTP id 3C7C589889\n\tfor <buildroot@lists.busybox.net>;\n\tThu, 14 Sep 2017 12:23:02 +0000 (UTC)","from whitealder.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id v8AXe3vcZxRE for <buildroot@lists.busybox.net>;\n\tThu, 14 Sep 2017 12:23:00 +0000 (UTC)","from mail-io0-f177.google.com (mail-io0-f177.google.com\n\t[209.85.223.177])\n\tby whitealder.osuosl.org (Postfix) with ESMTPS id D3AC389887\n\tfor <buildroot@buildroot.org>; Thu, 14 Sep 2017 12:23:00 +0000 (UTC)","by mail-io0-f177.google.com with SMTP id m103so512699iod.13\n\tfor <buildroot@buildroot.org>; Thu, 14 Sep 2017 05:23:00 -0700 (PDT)","by 10.168.71.130 with HTTP; Thu, 14 Sep 2017 05:22:59 -0700 (PDT)"],"X-Virus-Scanned":["amavisd-new at osuosl.org","amavisd-new at osuosl.org"],"X-Greylist":"from auto-whitelisted by SQLgrey-1.7.6","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=6wind-com.20150623.gappssmtp.com; s=20150623;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=/+jnOYrwILa8v8fTiCJ7uCb0CrA5vBvFOB/+QM/RqyU=;\n\tb=LDOl3DuwaZaolDOTrFluM8kZeMZcpw6RN91URTUovHhreCWh6iguNiIqwbecQstw26\n\tFEMvikz2beM6BrYpDJltjcikNiTE7jmCGh1U1+KVdrpfqudTmiOTBNGzRP+QV4jLBWra\n\tFZo33/5QWP6V4toxv3AbaUSlprfF90sAeA399b5Xxg139JTMvvTzr3auG6tZBXH5KYfZ\n\tWGvBintqFgXeuYVeK7PtW6vI9M0gIXyyMvRKmumawg0rg170VNJ3Sb5v9FZlCF++ttxe\n\tXNIo4NfdZJM+eFOpDa+j5DbIQnuDmmUUobQZ3CdWrXJvys1W2RWFDRR/j62Tqwq2lFnF\n\tVcow==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=/+jnOYrwILa8v8fTiCJ7uCb0CrA5vBvFOB/+QM/RqyU=;\n\tb=oXTfZqGlwHt1Ee8a1CvriRHe3IFSpr9nnr59W5oN87rqFaPLLHOY7Vi5XZcHaZyxuD\n\tXoPLKRcH4ZaMSqQ1PNGfTR9he0bkH2dBTgGiivAvuP1hh8SNzyb1Enr9ccFX1V2fLWgR\n\tmL4J1GFqsg/f+ppW2MgQUtqLbd/D9PklVRScRfrcsvP1+fsJp6C9eO2hpllcJPpWEpGT\n\tVaasga6iX/heL1NDaldfwgsRjmjb7mTqCBaLylrUtf+zpaYJEAcmEYMAns5oW3IDR5h1\n\tmZ8GxutqmrT8OIQH60yAkiyAyRHXvL5YOlfXd8q05nZC4ixyWScE0wDywFYaosDmkXyr\n\tHAiQ==","X-Gm-Message-State":"AHPjjUh6fagjgBlBW6frPA04wsi7tlC+4Ef3Vmt5V8XVXgw726qZvJJ7\n\t878+1uX3kfYyskI+SoBPOtccdv7oCuhvbKZJtBUs+b8Ei/k=","X-Google-Smtp-Source":"AOwi7QA9q++vR/hOwdTGvy3JeJ0pA50mt8nKOCAL5n3o/R+YIScoc/yOH7pprG/v0zybogfv1V8d9W8P7gUpIdG2f4Q=","X-Received":"by 10.202.4.15 with SMTP id 15mr20158955oie.125.1505391779831;\n\tThu, 14 Sep 2017 05:22:59 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<f14a15ee-4bc9-ffc0-38f1-fd15ebe43790@mind.be>","References":"<20170908134548.23645-1-julien.floret@6wind.com>\n\t<8d8bb3d5-6ea9-94b4-1a53-a57909b8bbc6@mind.be>\n\t<CAHfJR7jjCn8FdGqhKOq=7_pAgwObLNSQUdzzdbzOi+ijyvJLCw@mail.gmail.com>\n\t<d60e9f05-f4d2-3927-ee7f-148d6db4c22c@mind.be>\n\t<CAHfJR7jTT1tGBQr-rgZ+JTZQW6rEhagyDJBQAfB0yUPJwh_evw@mail.gmail.com>\n\t<f14a15ee-4bc9-ffc0-38f1-fd15ebe43790@mind.be>","From":"Julien Floret <julien.floret@6wind.com>","Date":"Thu, 14 Sep 2017 14:22:59 +0200","Message-ID":"<CAHfJR7gj=RGjP7VHFzwET_5OD6u6XbdpvO1A5uDxaUjKtotdtw@mail.gmail.com>","To":"Arnout Vandecappelle <arnout@mind.be>","Cc":"buildroot <buildroot@buildroot.org>","Subject":"Re: [Buildroot] [PATCH 1/1] netsnmp: install all MIB files","X-BeenThere":"buildroot@busybox.net","X-Mailman-Version":"2.1.18-1","Precedence":"list","List-Id":"Discussion and development of buildroot <buildroot.busybox.net>","List-Unsubscribe":"<http://lists.busybox.net/mailman/options/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=unsubscribe>","List-Archive":"<http://lists.busybox.net/pipermail/buildroot/>","List-Post":"<mailto:buildroot@busybox.net>","List-Help":"<mailto:buildroot-request@busybox.net?subject=help>","List-Subscribe":"<http://lists.busybox.net/mailman/listinfo/buildroot>,\n\t<mailto:buildroot-request@busybox.net?subject=subscribe>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"buildroot-bounces@busybox.net","Sender":"\"buildroot\" <buildroot-bounces@busybox.net>"}}]