<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 24, 2014 at 4:50 AM, Chris Down <span dir="ltr"><<a href="mailto:chris@chrisdown.name" target="_blank">chris@chrisdown.name</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Sure thing, thanks. One thing is that package maintainers need to make<br>
sure they install this alongside pass for 1.6, or there will be little<br>
point in merging.<br></blockquote><div><br></div><div>Also, some folks check contrib directories manually looking for goodies.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Also, I think the [ $? -eq 0 ] check you added (without mentioning,<br>
which is a little odd...) is undesirable.  Since this script uses bash,<br>
we might as well use `((', even if that does happen. I'm also not<br>
convinced this error check is necessary -- the only normal way that<br>
dmenu could return a value != 0 is if dmenu is manually terminated by a<br>
signal. In that (extremely rare) case, it doesn't even matter --<br>
$password will be empty. Programming for something that both isn't an<br>
issue when it happens, and something that happens extraordinarily<br>
rarely, is something I would really like to avoid.</blockquote><div><br></div><div>Thanks. This is something I was playing with myself and didn't actually mean to merge it. Anyway, I fixed it to the desired intent: <<a href="http://git.zx2c4.com/password-store/commit/?id=f7764184104315b620ddb39be691145863e089ab">http://git.zx2c4.com/password-store/commit/?id=f7764184104315b620ddb39be691145863e089ab</a>>.</div>
</div></div></div>