<div dir="ltr"><div dir="ltr">Well, on my installation I have:<div><br></div><div><div>brew --prefix gnu-getopt</div><div>/Users/jlaffaye/homebrew/opt/gnu-getopt</div></div><div><br></div><div>So I think we should still use brew --prefix but only if it is not found on standard locations.</div><div><br></div><div>I also want to suggest adding an environment variable to specify the location to not run this discovery step, so people in my situation can avoid calling brew each time they call pass.</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 23, 2018 at 6:06 PM <<a href="mailto:pass-maillinglist@artursterz.de">pass-maillinglist@artursterz.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><br><div><blockquote type="cite"><div><span style="font-family:sans-serif">The only thing that annoys me with the code above is that we always execute </span><code bgcolor="#F7F7F7" style="background-color:rgb(247,247,247);border-top-left-radius:3px;border-top-right-radius:3px;border-bottom-right-radius:3px;border-bottom-left-radius:3px;margin:0px;padding:0px 0.4em">brew</code><span style="font-family:sans-serif">, though technically the overhead is the same or less than guarding it with a </span><code bgcolor="#F7F7F7" style="background-color:rgb(247,247,247);border-top-left-radius:3px;border-top-right-radius:3px;border-bottom-right-radius:3px;border-bottom-left-radius:3px;margin:0px;padding:0px 0.4em">command -v brew</code><span style="font-family:sans-serif"> statement (less for people who have the executable).</span></div></blockquote></div><br><div>Okay, I see your point. Maybe we can ditch this all together? As far as I can see from the documentation (<a href="https://docs.brew.sh/Installation" target="_blank">https://docs.brew.sh/Installation</a>), there are about no chances that the brew prefix is not<font face="Menlo"> /usr/local</font>.</div><div>It requires similar steps as for macports (<a href="https://www.macports.org/install.php" target="_blank">https://www.macports.org/install.php</a>) to install brew somewhere else then in its default path. But for macports,<font face="Menlo"> pass </font>assumes the default path anyway, so we could do this for brew, could we?</div><div>Additionally, this is a somewhat failsafe version. If neither of the three options is available the user will have trouble using<font face="Menlo"> pass </font>anyways, so the<font face="Menlo"> test -x </font>check should be enough.</div><div><br></div><div>A patch based on Allans suggestions is appended.</div><div><br></div><div><br></div><div><div><font face="Menlo">diff --git a/src/platform/darwin.sh b/src/platform/darwin.sh</font></div><div><font face="Menlo">index 342ecce..a984bec 100644</font></div><div><font face="Menlo">--- a/src/platform/darwin.sh</font></div><div><font face="Menlo">+++ b/src/platform/darwin.sh</font></div><div><font face="Menlo">@@ -43,6 +43,10 @@ qrcode() {</font></div><div><font face="Menlo">        fi</font></div><div><font face="Menlo"> }</font></div><div><font face="Menlo"><br></font></div><div><font face="Menlo">-GETOPT="$(brew --prefix gnu-getopt 2>/dev/null || { which port &>/dev/null && echo /opt/local; } || echo /usr/local)/bin/getopt"</font></div><div><font face="Menlo">+for prefix in /usr/local/opt/gnu-getopt /opt/local /usr/local; do</font></div><div><font face="Menlo">+    GETOPT="${prefix}/bin/getopt"</font></div><div><font face="Menlo">+    test -x "$GETOPT" && break</font></div><div><font face="Menlo">+done</font></div><div><font face="Menlo">+</font></div><div><font face="Menlo"> SHRED="srm -f -z"</font></div><div><font face="Menlo"> BASE64="openssl base64"</font></div></div></div>_______________________________________________<br>
Password-Store mailing list<br>
<a href="mailto:Password-Store@lists.zx2c4.com" target="_blank">Password-Store@lists.zx2c4.com</a><br>
<a href="https://lists.zx2c4.com/mailman/listinfo/password-store" rel="noreferrer" target="_blank">https://lists.zx2c4.com/mailman/listinfo/password-store</a><br>
</blockquote></div>