Package: mmdebstrap
Version: 0.5.1-4
Severity: wishlist

Currently --components allows separation by commas OR spaces.
Currently --include allows separation by commas BUT NOT spaces:

    $ mmdebstrap sid delete-me --components='main contrib non-free' 
--include='amd64-microcode,intel-microcode,iucode-tool'    # works
    $ mmdebstrap sid delete-me --components='main contrib non-free' 
--include='amd64-microcode intel-microcode iucode-tool'    # fails
    [...]
    E: Unable to locate package amd64-microcode intel-microcode iucode-tool
    [...]

Please make --include behave like --components.

This will make it easier for me to create --include from a great big
bash array with inline comments:

    includes+=(
        baz                         # for chromium
        quux
        quuux                       # because of international terrorism
        # [...]
    )

Because then I can simply say

    "--include=${includes[*]}"

instead of having to mess around e.g.

    OLDIFS=$IFS IFS=,
    "--include=${includes[*]}"
    IFS=$OLDIFS


My perl-fu is pretty stale, but based on

    my @comps = split /[,\s]+/, $comp;

I think this patch is "good enough":

    --- /rsync:root@not-omega:/tmp/bootstrap/mmdebstrap
    +++ #<buffer mmdebstrap-patched-2>
    @@ -1196,7 +1196,7 @@

         my %pkgs_to_install;
         if (defined $options->{include}) {
    -   for my $pkg (split /,/, $options->{include}) {
    +   for my $pkg (split /[,\s]+/, $options->{include}) {
                $pkgs_to_install{$pkg} = ();
            }
         }

Reply via email to