Control: tag -1 + pending

Hi Helmut,

Quoting Helmut Grohne (2023-01-15 17:41:32)
> we recently talked about mmdebstrap --include with an apt pattern containing
> a comma being broken. An example invocation is:
> 
>     mmdebstrap --variant=apt 
> '--include=?or(?exact-name(linux-image-cloud-amd64),?exact-name(linux-image-amd64))'
>  unstable
> 
> I figured that I could also come up with the fixing patch and am
> attaching it to your convenience. If you want to include test cases,
> please keep in mind that this is fixed twice, once for --variant=extract and
> once for --variant=apt.

thank you for your bug report and the patch! Though I think that the splitting
can be completely omitted and the code can rely on the splitting being done in
the argument parser. I fixed this here:

https://gitlab.mister-muffin.de/josch/mmdebstrap/commit/f27ed490d6d3799e0a51fa33e68e6832d0fc4eb5

Two new test cases make sure that this works for --variant=extract as well as
for the normal variants and includes an apt pattern very similar to your
example to make sure that this works.

Thanks!

cheers, josch

Reply via email to