<div dir="ltr">Hi,<div><br></div><div style>  This should be fixed now by 655afb183bd8abac0b60307645c9b43ff37b3082. Could you try it out ?</div><div style><br></div><div style>           Zoltan</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, May 29, 2013 at 6:00 PM, Jeremy Bell <span dir="ltr"><<a href="mailto:bell.jeremy@gmail.com" target="_blank">bell.jeremy@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div><div>I'm not sure if this is the best way to fix the issue, but I've submitted a pull request with a small fix:<br><a href="https://github.com/mono/mono/pull/650" target="_blank">https://github.com/mono/mono/pull/650</a><br>

<br></div>This is my first mono pull request, so please let me know if there are any contrib guidelines I missed.<br><br></div>Regards,<br>Jeremy<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br>
<br><div class="gmail_quote">On Thu, May 23, 2013 at 2:25 PM, Jeremy Bell <span dir="ltr"><<a href="mailto:bell.jeremy@gmail.com" target="_blank">bell.jeremy@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><br>These:<br>export SYSROOT=$NDK/platforms/android-14/arch-arm<br>export PATH=$NDK_STANDALONE/bin:$PATH<br>

export CC=arm-linux-androideabi-gcc<br>export CXX=arm-linux-androideabi-g++<br>export AR=arm-linux-androideabi-ar<br>
export AS=arm-linux-androideabi-as<br>export CPP=arm-linux-androideabi-cpp<br>export LD=arm-linux-androideabi-ld<br>export RANLIB=arm-linux-androideabi-ranlib<br>export STRIP=arm-linux-androideabi-strip<br>./autogen.sh --build=i686-pc-linux-gnu --host=arm-linux-androideabi --target=arm-linux-androideabi --enable-nls=no --with-mcs-docs=no --with-mcs-build=no CFLAGS="-DARM_FPU_NONE=1" CXXFLAGS="-DARM_FPU_NONE=1" --prefix=$PREFIX<br>


<br></div><div>Same issue with the armv7-a build:<br>export SYSROOT=$NDK/platforms/android-14/arch-arm<br>export PATH=$NDK_STANDALONE/bin:$PATH<br>export CC=arm-linux-androideabi-gcc<br>export CXX=arm-linux-androideabi-g++<br>


export AR=arm-linux-androideabi-ar<br>export AS=arm-linux-androideabi-as<br>export CPP=arm-linux-androideabi-cpp<br>export LD=arm-linux-androideabi-ld<br>export RANLIB=arm-linux-androideabi-ranlib<br>export STRIP=arm-linux-androideabi-strip<br>


./autogen.sh --build=i686-pc-linux-gnu  --host=armv7-a-linux-androideabi --target=armv7-a-linux-androideabi --enable-nls=no --with-mcs-docs=no --with-mcs-build=no CFLAGS="-DARM_FPU_VFP=1" CXXFLAGS="-DARM_FPU_VFP" --prefix=$INSTALL_PREFIX<br>


<br></div><div><br></div>My system:<br>Ubuntu 13.04<br><br></div>Thanks,<br>Jeremy<br></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 23, 2013 at 1:39 PM, Zoltan Varga <span dir="ltr"><<a href="mailto:vargaz@gmail.com" target="_blank">vargaz@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div> buildver.h is always built unless some configure flag disables it. What configure arguments are you using ?</div>


<span><font color="#888888"><div><br></div><div>          Zoltan</div></font></span></div><div class="gmail_extra">
<br><br><div class="gmail_quote"><div><div>On Thu, May 23, 2013 at 5:01 PM, Jeremy Bell <span dir="ltr"><<a href="mailto:bell.jeremy@gmail.com" target="_blank">bell.jeremy@gmail.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
<div dir="ltr"><div><div><div><div><div>At some point between branch mono-2-10-9 and branch master, a change was made to /mono/mini/main.c:<br><br></div>branch mono-2-10-9:<br><br></div>main.c:<br>#include <config.h><br>




#include "mini.h"<br>#ifndef HOST_WIN32<br>#include "buildver.h"<br>#endif<br><br><br></div>branch master:<br>#include <config.h><br>#include "mini.h"<br>#ifndef HOST_WIN32<br>#ifdef HAVE_SGEN_GC<br>




#include "buildver-sgen.h"<br>#else<br>#include "buildver.h"<br>#endif<br>#endif<br><br></div>This makes main.c impossible to compile when buildver-sgen.h is generated and not buildver.h. First of all, HAVE_SGEN_GC is never defined for files in /mini as far as I can tell, so main.c always attempts to include buildver.h, which does not exist when buildver-sgen.h is generated instead.<br>




<br>However, even if you explicitly define HAVE_SGEN_GC in CFLAGS, etc... then you will still get an error, in mini.h, because it believes it is an error to have either HAVE_SGEN_GC or HAVE_BOEHM_GC defined when mini.h is included, as /mini code should not have dependencies on the GC being used, so it says:<br>




<br></div><div>mini.h:<br>/*<br> * The mini code should not have any compile time dependencies on the GC being used, so the same object file from mini/<br> * can be linked into both mono and mono-sgen.<br> */<br>#if defined(HAVE_BOEHM_GC) || defined(HAVE_SGEN_GC)<br>




#error "The code in mini/ should not depend on these defines."<br>#endif<br><br><br></div><div>So, either way, main.c won't compile without modification. Is the error in /mono/mini/mini.h no longer valid? Or was the change to /mono/mini/main.c to depend on the HAVE_SGEN_GC define a regression?<br>




<br></div><div>Thanks,<br>Jeremy<br></div></div>
<br></div></div><div>_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
<br></div></blockquote></div><br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>