<br><br><div class="gmail_quote">On Sat, Sep 12, 2009 at 12:37 PM, Nick Treleaven <span dir="ltr"><<a href="mailto:nick.treleaven@btinternet.com" target="_blank">nick.treleaven@btinternet.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">


<div>On Fri, 11 Sep 2009 14:06:38 -0400<br>
Erik Southworth <<a href="mailto:erik.southworth@gmail.com" target="_blank">erik.southworth@gmail.com</a>> wrote:<br>
<br>
> To be clear, if we don't have ``which`` it shouldn't error about a c++<br>
> compiler. Also, (1) if we must have ``which``, configure should check for it<br>
> before trying to use it. (2) If we need to check a prog use AC_CHECK_PROG or<br>
> AC_PATH_PROG.<br>
><br>
> AFAIK,<br>
> > which is a coreutils utility, so it should work on all systems capable of<br>
> > executing a shell script.<br>
><br>
><br>
> No. It's not part of coreutils.<br>
><br>
><br>
> > 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>
> ><br>
><br>
> It's that Geany is built in a clean chroot build environment along with only<br>
> the compilers and libs required. ``which`` is not a necessary build<br>
> requirement IMHO. We should have a portable, standard, ``configure`` that<br>
> can build Geany, not a system utility script that calls any arbitrary prog<br>
> and forces users to figure out which ones are needed.<br>
><br>
><br>
> > Chances are that your $CXX is not defined to a correct variable.<br>
> ><br>
><br>
> > ><br>
> > > How about just using a shell builtin, ``hash``, ``type -P`` or just<br>
> > ``eval<br>
> > > $CXX --version`` as a test instead?<br>
> > Does $CXX --version really exist with all C++ compilers?<br>
> ><br>
><br>
> Bottom line: we should use portable autoconf macros not random system<br>
> utilities.<br>
<br>
</div>If you provide a patch that does it portably we would likely apply it.<br>
<br></blockquote><div>Here is a patch against Rev 4190 in Trunk.<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;">
Regards,<br>
<font color="#888888">Nick<br>
</font><div><div></div><div>_______________________________________________<br>
Geany-devel mailing list<br>
<a href="mailto:Geany-devel@uvena.de" target="_blank">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>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>-Erik S<br>