[PATCH 1 of 7 V2] [pyramid/ctl] pyramid command will always run in foreground from now on

Denis Laxalde denis.laxalde at logilab.fr
Tue Sep 24 15:31:07 CEST 2019


Laurent Peuch a écrit :
> Turns out it is actually fixed in this series in changeset
> c1d201581b16 "[cubicweb-ctl] backport --loglevel option of pyramid to
> all instance commands", the merge_options was missing in pyramid.
> 
> Dunno if you want a specific patch for that one?
> 
> The other error on ^C for pyramid (when pyramid command returned 1) is
> also solve in the pdb related changeset.

Yes, it's not that obvious to me which patch to pick to solve this issue
from the last series you sent (i.e. I'm lost).

So if you could send a series of patches to solve this issue first and
independently of any features, that'd be really nice.

On the other hand, I'm not sure the proposed solution (merge_options) is
the way to go. Some times ago, I questioned the usefulness of the --pdb
flag for /all/ instance commands. Specifically, what's the point of
having this for the "pyramid" command?



More information about the cubicweb-devel mailing list