On Feb 7, 2017, at 6:59 AM, Jean-Paul Calderone exarkun@twistedmatrix.com wrote:
On Fri, Jan 13, 2017 at 1:13 AM, Glyph Lefkowitz <glyph@twistedmatrix.com mailto:glyph@twistedmatrix.com> wrote: Thanks for highlighting those. I've put the link in the other direction as well.
Craig seems eager to go ahead with reverting this change in behavior.
https://github.com/twisted/twisted/pull/672#issuecomment-275956265 https://github.com/twisted/twisted/pull/672#issuecomment-275956265
As far as I can tell, no one has weighed in on the other side. So I'm inclined to go along with the reversion.
My 2¢ for the other side is: if trial does this, but twist and twistd don't, then it will be possible to get a passing test run for a plugin that doesn't get loaded. I think it would be simpler and easier to debug to leave these consistent.
(I really don't think we should put this behavior back into twist or twistd, because "cd malware; twist web --path ." should not pwn your machine.)
-glyph