<div dir="ltr"><div class="gmail_default" style="font-family:verdana,sans-serif">Hi Raphael,</div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif">The systemd changes mostly looks good (patch 002).   I like the practice of not using /etc/default/snmpd (I did not use this when I "systemd-itized" net snmp).    </div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif">Other then that, my "After" was set to "networking.service syslog.service" rather then "network.target" because I find the latter does not mean the network is actually up and usable.   Also, we opted to not start snmpd  </div><div class="gmail_default" style="font-family:verdana,sans-serif">by default (we make users consciously choose to "enable" and "start" (in debian/rules file).</div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif">I haven't looked at the first patch (diffs of diffs give me a headache...I just applied all the patches (via git apply) but I will look at it later this week.   Patches 3 and 4 look good but I haven't tested them yet.</div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif">Regards,</div><div class="gmail_default" style="font-family:verdana,sans-serif">Sam Tannous</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 16, 2016 at 8:56 AM, Raphael Hertzog <span dir="ltr"><<a href="mailto:hertzog@debian.org" target="_blank">hertzog@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Control: tag -1 + patch<br>
<br>
Hello people,<br>
<br>
please find attached two patches to apply to the pkg-net-snmp git<br>
repository. The first one is the inclusion of the last NMU by Steinar<br>
Gundersson. The second one adds two systemd service files with the<br>
required packaging changes.<br>
<br>
Hideki, you seem to be the only active maintainer lately, can you apply<br>
those patches and make an upload or would you prefer an non-maintainer<br>
upload?<br>
<br>
Following systemd best practices, I opted to not load<br>
/etc/default/snmpd in the environment. Any customization shall be made<br>
with override files and I documented this in the "*.default" files.<br>
<br>
Thank you.<br>
<br>
PS: I would have gladly committed my work in your git repository,<br>
unfortunately the pkg-net-snmp repository is not writable to all Debian<br>
developers. The package does not seem to be very actively maintained...<br>
maybe it would be a good idea to drop from Uploaders the people who<br>
are not doing anything and possibly move the package to collab-maint<br>
so that it's easier for drive-by contributors like me to help you?<br>
<br>
PPS: Thomas, Tom, Sam, Mathieu, feel free to test this patch and to report<br>
back any problem that you might have discovered. While I ensured that<br>
the daemon start properly, I'm not an actual net-snmp user and would<br>
welcome a confirmation by real users that it works as expected.<br>
<span class="HOEnZb"><font color="#888888">--<br>
Raphaël Hertzog ◈ Debian Developer<br>
<br>
Support Debian LTS: <a href="http://www.freexian.com/services/debian-lts.html" rel="noreferrer" target="_blank">http://www.freexian.com/services/debian-lts.html</a><br>
Learn to master Debian: <a href="http://debian-handbook.info/get/" rel="noreferrer" target="_blank">http://debian-handbook.info/get/</a><br>
</font></span></blockquote></div><br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div style="color:rgb(0,0,0);font-family:Times;font-size:14px"><span style="font-family:arial,helvetica,sans-serif;font-size:small"><span style="color:rgb(0,90,149)"><span><br></span></span></span></div></div></div>
</div></div>