Thank you, Grzegorz. I looked over the patch and I don't see why the outer reference counter is required. It deletes the managed socket from the SocketManager.sockets dictionary but why does that need to happen? Would there be a problem if that reference stayed in place?
On Tue, Nov 30, 2010 at 8:23 AM, Grzegorz Nosek <r...@localdomain.pl> wrote: > W dniu 30.11.2010 00:59, Roger Hoover pisze: >> >> Is the patch not based on svn rev 996? I'm getting an error trying to >> apply the 3rd patch. >> >> Roger-Hoovers-MacBook-Pro:supervisor(share_socket) rhoover$ git apply >> /Users/rhoover/Downloads/0003-Expose-reuse_socket-setting-in-config.patch >> error: patch failed: src/supervisor/options.py:775 >> error: src/supervisor/options.py: patch does not apply > > Oops, sorry. It's based on my supervisord repo, which has some more changes > (I sent everything except PAM support to the list). > > Anyway, the patchset is here again, this time based on rev 996. > > BTW, anybody cares about PAM support? It would probably require shipping a > private copy of the pam module but is otherwise infinitely cool ;) > > Best regards, > Grzegorz Nosek > _______________________________________________ Supervisor-users mailing list Supervisor-users@lists.supervisord.org http://lists.supervisord.org/mailman/listinfo/supervisor-users
