Rafael Zalamena <rzalam...@gmail.com> writes: > On Sat, Oct 22, 2016 at 10:32:27PM +0200, Rafael Zalamena wrote: >> On Sat, Oct 22, 2016 at 08:14:16PM +0200, Jeremie Courreges-Anglas wrote: >> > Rafael Zalamena <rzalam...@gmail.com> writes: >> > > On Fri, Oct 21, 2016 at 01:26:36PM +0200, Jeremie Courreges-Anglas wrote: >> > >> Rafael Zalamena <rzalam...@gmail.com> writes: >> > >> ---snip--- >> >> Short answer: >> Yes, you are correct to note this and I think now that it is probably better >> to write another diff to solve this problem. I'll get back at this diff >> later. >> >> ---snip--- >> >> Just to clarify: >> I talked with reyk@ about this global env variables in the last hackathon, >> and we reached the conclusion that the best way to handle this is to use >> the ps_env whenever is possible, however since a lot of functions don't >> get access to ps, we must decide what does less changes to the daemon: >> 1) Use a single global variable (look at the httpd(8) commits); >> 2) Keep using the env (relayd(8) case); > > The diff that makes snmpd(8) use only one global env is in, now we can > move on with the fork+exec diff. > > Here is the updated diff. > > ok?
yup -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
