Code review comment for lp:~chmouel/nova/lp684657

Revision history for this message
Thierry Carrez (ttx) wrote :

Right, this covers in fact two bugs, one of which is invalid, and the other is Low prio. We should reject this branch and wait for a version that only fixes the bashisms.

Chmouel: in all cases, could you file a separate bug for the "doesn't run in bash" issue so that we don't lose track of that ?

« Back to merge proposal