← Back to team overview

curtin-dev team mailing list archive

Re: [Merge] ~dbungert/curtin:lp-1987236-chzdev into curtin:master

 

> Looks OKish to me. Are there other reasons why chzdev_export might fail?
> Swallowing failure like this is always a bit dodgy.

We could make this more specific and only swallow exit code 8:
       8      Empty selection
https://github.com/hreinecke/s390-tools-1/blob/39382ec5fdaa78c8b9e3534ff9537af7ce130607/zdev/man/chzdev.8#L634
-- 
https://code.launchpad.net/~dbungert/curtin/+git/curtin/+merge/428926
Your team curtin developers is requested to review the proposed merge of ~dbungert/curtin:lp-1987236-chzdev into curtin:master.



References