<div dir="ltr">any updates on this one?<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Oct 11, 2013 at 5:34 AM, Chris Down <span dir="ltr"><<a href="mailto:chris@chrisdown.name" target="_blank">chris@chrisdown.name</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dominik,<br>
<br>
On 2013-10-10 14:53, Dominik Klein wrote:<br>
> I found your tool very useful, but for me, it lacked one feature: It could<br>
> not display more than one passwordfile at once. With every password being<br>
> in one file, this quickly became an issue for me. Here's a patch.<br>
<br>
Nice!<br>
<br>
> I also noticed that some used programs are not installed by default (at<br>
> least on my ubuntu 12.04), so I wrote a quick check for that.<br>
<br>
In my opinion this should be handled by your package manager, and all checks<br>
for program presence should not be in pass (other than to determine whether to<br>
enable/disable optional functionality).<br>
<br>
> I tried not to brake previous behaviour of the program. Let me know if you<br>
> see any problems in including this.<br>
<br>
I only briefly looked over the patch because I'm at work, but your boolean<br>
style could use some work to become aligned with what was accepted as the<br>
default style:<br>
<br>
    foo=false<br>
    if $foo; then<br>
<br>
becomes<br>
<br>
    foo=0<br>
    if (( foo )); then<br>
<br>
I'll look into it in more detail when I'm out of work.<br>
<br>
Thanks for the patch!<br>
</blockquote></div><br></div>