<html>

  <head>

    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">

  </head>

  <body>

    <div class="moz-cite-prefix">On 4/6/22 10:26, Dietmar Maurer wrote:<br>

    </div>

    <blockquote type="cite"

      cite="mid:1074870458.1383.1649233576312@webmail.proxmox.com">

      <pre class="moz-quote-pre" wrap="">

</pre>

      <blockquote type="cite">

        <pre class="moz-quote-pre" wrap="">On 04/04/2022 6:19 PM Dylan Whyte <a class="moz-txt-link-rfc2396E" href="mailto:d.whyte@proxmox.com"><d.whyte@proxmox.com></a> wrote:

Prior to this, during an interactive restore, if a directory was matched

via a pattern match or selection, only the empty directory would be

restored, and not its contents. 

</pre>

      </blockquote>

      <pre class="moz-quote-pre" wrap="">

Why not simply use "**" if you want to restore a whole tree?

</pre>

    </blockquote>

    <p>I had originally thought about this, but there are some good

      reasons for the patch:</p>

    <ul>

      <li>I believe there is an expectation when selecting a directory

        for restore, that you would like for the entire directory to be

        restored (unless any sub-directory is explicitly excluded).</li>

      <li>The 'select' command doesn't do pattern matching, so it

        wouldn't be able to use '**' to restore the directory. This

        point doesn't apply to 'find' and 'restore --pattern'.<br>

      </li>

      <li>With the current implementation, '**' won't restore empty

        sub-directories of a matched directory, in spite of the fact

        that they appear in the match list.</li>

    </ul>

  </body>

</html>