Emulator build failed

More
18 Jan 2017 16:49 - 18 Jan 2017 16:50 #58131 by vlad_vy
Emulator build failed was created by vlad_vy
After PR #98 "faster byte reverse for xn297 routines" I can't build emulators with my Windows build environment, I have the error:
...
 + Compiling 'protocol/spi/nrf24l01.c'
protocol/spi/nrf24l01.c:306:30: error: "__ARM_ARCH_ISA_THUMB" is not defined [-Werror=undef]
cc1.exe: some warnings being treated as errors

make: *** [objs/emu_devo8-w32/nrf24l01.o] Error 1
What is wrong with my build environment?
Last edit: 18 Jan 2017 16:50 by vlad_vy.

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 04:35 #58163 by vlad_vy
Replied by vlad_vy on topic Emulator build failed
Will be fixed in Nightly builds, if pull request will be approved.

'protocol/spi/nrf24l01.c'
-#if ( defined (__GNUC__) && (__ARM_ARCH_ISA_THUMB==2) )
+#if ( defined (__GNUC__) && defined (__ARM_ARCH_ISA_THUMB) && (__ARM_ARCH_ISA_THUMB == 2) )

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 05:19 #58166 by victzh
Replied by victzh on topic Emulator build failed
Your compiler is GNU C, but __ARM_ARCH_ISA_THUMB is not defined. I'll fix it in a sec.

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 07:32 - 19 Jan 2017 07:33 #58167 by vlad_vy
Replied by vlad_vy on topic Emulator build failed
After PR #105 "Filesystem for the emu_devo12e" I can't build emulator for devo12e with my Windows build environment, I have infinite number of 3 lines:
...
print() on closed filehandle $fh at ../utils/font/bdf_to_font.pl line 232.
print() on closed filehandle $fh at ../utils/font/bdf_to_font.pl line 233.
print() on closed filehandle $fh at ../utils/font/bdf_to_font.pl line 234.
...
Last edit: 19 Jan 2017 07:33 by vlad_vy.

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 07:58 #58169 by FDR
Replied by FDR on topic Emulator build failed
Strange, because I could build it yesterday on Windows as well.
I'll take a look when I'll get home.

Without that mod the 12e filesystem only contained the models directory...

I've got my 12E just before Xmas, but only now started to care about it, because I had no time for that.
Yesterday I loaded deviation on it, but realised, that the filesystem is missing the language directory.
It behaved strangely on the main page config, so I wanted to try that in the emulator, when I saw that it misses almost the whole filesystem.
I built with my changes, and I could start the emulator, however the screen was too small, which you've already corrected, but the buttons used are wrong too, so it is still not usable...

Vlad, do you have a 12E too?

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 08:01 #58170 by vlad_vy
Replied by vlad_vy on topic Emulator build failed
No, but emulator buttons fixed also. See my next pull request.

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 08:02 #58171 by FDR
Replied by FDR on topic Emulator build failed
Thank! :)
I'll try that too...

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 08:43 - 19 Jan 2017 08:45 #58173 by vlad_vy
Replied by vlad_vy on topic Emulator build failed
With my proposed changes for PR #105 "Filesystem for the emu_devo12e" the devo12e emulator builds and works fine. I think it's the right way, all other emulators do the same.
Last edit: 19 Jan 2017 08:45 by vlad_vy.

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 14:04 #58189 by vlad_vy
Replied by vlad_vy on topic Emulator build failed
Pull request with fix has been sent.

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 16:51 #58195 by FDR
Replied by FDR on topic Emulator build failed

vlad_vy wrote: No, but emulator buttons fixed also. See my next pull request.

Merged.

Please Log in or Create an account to join the conversation.

More
19 Jan 2017 16:52 #58196 by FDR
Replied by FDR on topic Emulator build failed

vlad_vy wrote: Pull request with fix has been sent.

Merged too...

Please Log in or Create an account to join the conversation.

Time to create page: 0.046 seconds
Powered by Kunena Forum