<font face="'Helvetica Neue', 'Helvetica Neue', Helvetica, Arial, sans-serif"><span style=" font-size:14px; font-family:'Helvetica Neue', 'Helvetica Neue', Helvetica, Arial, sans-serif;">Kevin may have forgotten a reply all, but he proposed a POSIX compatible fix, rather than relying on undocumented behavior.<br><br>–<div>Kyle Marek-Spartz</div><br><br><br><span id="forwardBeginsHere"></span>On May 5, 2014, 5:00:31 PM, Kevin Cox <kevincox@kevincox.ca> wrote:<hr><br>On 05/05/14 16:21, Kyle Marek-Spartz wrote: <br><blockquote type="cite">Hello, <br> <br>A recent change in pass.zsh-completion broke autocompletion when using <br>BSD sed. I’ve made the relevant sed expression compatible with GNU and <br>BSD sed. <br> <br>Previous change with regression: <br>http://lists.zx2c4.com/pipermail/password-store/2014-April/000773.html <br> <br>http://git.zx2c4.com/password-store/commit/src/completion/pass.zsh-completion?id=f82e9d6cf3bc3a12bdfce89bf319d76f79e66efc <br> <br></blockquote> <br>I tested out this fix and it works on GNU sed.  The -E option is not <br>documented in GNU sed but it is an alias of the -r extended regular <br>expressions option.  It may be slightly better to use the -r option <br>instead as it is documented in both implementations (the BSD man page <br>says it is an alias of -E). <br> <br>So while this fix works it might be worthwhile to stick posix basic (as <br>that is what is used throughout pass and since extended is not part of <br>posix sed (AFAIK).  To continue with posix basic the `\?` can be <br>replaced with `\{0,1\}` which should work on all compliant seds. <br> <br>Cheers, <br>Kevin <br> <br><hr>                 <div>- signature.asc, 303 bytes</div></span></font>