On Fri, Sep 11, 2009 at 12:34 AM, Chow Loong Jin <span dir="ltr"><<a href="mailto:hyperair@gmail.com">hyperair@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

On Friday 11,September,2009 09:58 AM, Erik Southworth wrote:<br>
> [...]<br>
<div class="im">> I didn't know that the ``which`` utility was considered a compiler. :p<br>
</div>It isn't, but it searches your $PATH for the full path of a utility, $CXX being<br>
the utility it's searching for. (CXX is a variable containing its name). </blockquote><div><br>To be clear, if we don't have ``which`` it shouldn't error about a c++ compiler. Also, (1) if we must have ``which``, configure should check for it before trying to use it. (2) If we need to check a prog use AC_CHECK_PROG or AC_PATH_PROG.<br>

<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">AFAIK,<br>
which is a coreutils utility, so it should work on all systems capable of<br>
executing a shell script. </blockquote><div><br>No. It's not part of coreutils.<br>  <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

In fact, I think there are many system scripts which<br>
use 'which', so if you don't have 'which', your system shouldn't even work.<br></blockquote><div><br>It's that Geany is built in a clean chroot build environment along with only the compilers and libs required. ``which`` is not a necessary build requirement IMHO. We should have a portable, standard, ``configure`` that can build Geany, not a system utility script that calls any arbitrary prog and forces users to figure out which ones are needed.<br>

 <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Chances are that your $CXX is not defined to a correct variable.<br></blockquote><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im"><br>
><br>
> How about just using a shell builtin, ``hash``, ``type -P`` or just ``eval<br>
> $CXX --version`` as a test instead?<br>
</div>Does $CXX --version really exist with all C++ compilers?<br></blockquote><div><br>Bottom line: we should use portable autoconf macros not random system utilities.<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">


<div class="im">><br>
> ------------------------------------------------------------------<br>
> <a href="http://configure.in:18" target="_blank">configure.in:18</a> : which $CXX >/dev/null 2>&1<br>
> <a href="http://configure.in:51" target="_blank">configure.in:51</a> : GIT=`which git 2>/dev/null`<br>
> <a href="http://configure.in:60" target="_blank">configure.in:60</a> : SVN=`which svn 2>/dev/null`<br>
> Found 3 matches for "which".<br>
</div>The last two are required to get the full path of git and svn into $GIT and<br>
$SVN. The first is harmless for reasons mentioned above.<br>
<br>
--<br>
Kind regards,<br>
<font color="#888888">Chow Loong Jin<br>
<br>
</font><br>_______________________________________________<br>
Geany-devel mailing list<br>
<a href="mailto:Geany-devel@uvena.de">Geany-devel@uvena.de</a><br>
<a href="http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel" target="_blank">http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br>-Erik S<br>