[Ohrrpgce] SVN: teeemcee/8705 Add Slice.{Pos, ScreenPos, Size}, Frame.size, Surface.size, MouseInfo.pos,

Thu May 18 10:03:38 PDT 2017

On Thu, May 18, 2017 at 9:57 AM, Ralph Versteegen <teeemcee at gmail.com>
wrote:

> Ah. Did you have to fix the last problems you had by downgrading
> something? It's hard to have compatible versions of everything. I think we
> do support the latest NDK though?
>

I got the SDK fixed on my work laptop by uninstalling Android Studio, and
restoring a month-old backup copy of my entire SDK folder.

But I don't remember if I ever fixed the issues on my home laptop, which is
the nightly build one (and if I remember correctly, its issues were
different. I don't think I ever installed Android Studio there)

>
> I looked at the release notes. I see that a future NDK version will remove
> GCC. It's already deprecated. FB does not support compiling with clang yet,
> but I think it's likely that clang will eventually add support for
> whichever gcc extension/quirks FB requires.
>
> Also, the latest NDK targets android 2.3 at a minimum, but the next
> release, within the next month, will support 4.0 at a minimum.
> (Of course, this doesn't mean we can't support compiling with that NDK)
>

When I set up the VM, I'll probably just stick with whatever NDK works for
me right now (which is r8, I think?)

---
James

>
> On 19 May 2017 at 04:40, James Paige <Bob at hamsterrepublic.com> wrote:
>
>> Because If it is in a VM, I won't accidentally break it when I mess with
>> my Android SDK for unrelated projects :)
>>
>> (I could probably set up a temporary non-VM nightly build, but I have to
>> double check whether or not my Android SDK is *already* broken on my
>> nightly build machine from the last time I upgraded something Java related)
>>
>> On Thu, May 18, 2017 at 9:34 AM, Ralph Versteegen <teeemcee at gmail.com>
>> wrote:
>>
>>> Why would you need a VM for that?
>>>
>>> On 19 May 2017 at 04:09, James Paige <Bob at hamsterrepublic.com> wrote:
>>>
>>>> I haven't had time yet, but sooner or later I will get an Android
>>>> nightly build VM up and running so we can spot such things quicker.
>>>>
>>>> On Thu, May 18, 2017 at 9:05 AM, Ralph Versteegen <teeemcee at gmail.com>
>>>> wrote:
>>>>
>>>>> Fixed. The NDK r8 toolchain uses gcc 4.6. I was going to say it
>>>>> doesn't support that c++11 feature, but actually looks more like a compiler
>>>>> bug.
>>>>>
>>>>> On 19 May 2017 at 02:31, James Paige <Bob at hamsterrepublic.com> wrote:
>>>>>
>>>>>> Double Opps!
>>>>>>
>>>>>> 8704 is the last revision that worked on android.
>>>>>> 8705 was broken on other platforms
>>>>>> 8706 fixed other platforms, but is still broken on Android.
>>>>>>
>>>>>> Here is the place where it fails:
>>>>>>
>>>>>> Compile++ thumb  : application <= surface.cpp
>>>>>> /home/james/misc/android-ndk-r8d/toolchains/arm-linux-androi
>>>>>> deabi-4.6/prebuilt/linux-x86/bin/arm-linux-androideabi-g++ -MMD -MP
>>>>>> -MF ./obj/local/armeabi/objs-debug/application/ohrrpgce/tmp/surface.o.d
>>>>>> -fpic -ffunction-sections -funwind-tables -fstack-protector
>>>>>> -D__ARM_ARCH_5__ -D__ARM_ARCH_5T__ -D__ARM_ARCH_5E__ -D__ARM_ARCH_5TE__
>>>>>> -no-canonical-prefixes -march=armv5te -mtune=xscale -msoft-float
>>>>>> -fno-exceptions -fno-rtti -mthumb -Os -g -DNDEBUG -fomit-frame-pointer
>>>>>> -fno-strict-aliasing -finline-limit=64 -O0 -UNDEBUG -marm
>>>>>> -fno-omit-frame-pointer -Ijni/../jni/application/ohrrpgce/tmp
>>>>>> -Ijni/../jni/application/../sdl-1.2/include
>>>>>> -Ijni/../jni/application/../sdl_mixer/include
>>>>>> -Ijni/../jni/application/../fb/include -I/home/james/misc/android-ndk
>>>>>> -r8d/sources/cxx-stl/gnu-libstdc++/4.6/include
>>>>>> -I/home/james/misc/android-ndk-r8d/sources/cxx-stl/gnu-libstdc++/4.6/libs/armeabi/include
>>>>>> -Ijni/../jni/application -DANDROID -O3 -DFBCVERSION=1060 --std=c99
>>>>>> -Wa,--noexecstack --std=c++0x -g -Wall -Wno-non-virtual-dtor  -g
>>>>>> -I/home/james/misc/android-ndk-r8d/platforms/android-4/arch-arm/usr/include
>>>>>> -c  jni/../jni/application/ohrrpgce/tmp/surface.cpp -o
>>>>>> ./obj/local/armeabi/objs-debug/application/ohrrpgce/tmp/surface.o
>>>>>> cc1plus: warning: command line option '-std=c99' is valid for C/ObjC
>>>>>> but not for C++ [enabled by default]
>>>>>> jni/../jni/application/ohrrpgce/tmp/surface.cpp: In function 'void
>>>>>> shrinkrow(RGBcolor*, int, RGBcolor*, int, int, fixedpoint)':
>>>>>> jni/../jni/application/ohrrpgce/tmp/surface.cpp:157:85: error:
>>>>>> cannot convert '<brace-enclosed initializer list>' to 'uint32_t {aka
>>>>>> unsigned int}' in initialization
>>>>>> make: *** [obj/local/armeabi/objs-debug/
>>>>>> application/ohrrpgce/tmp/surface.o] Error 1
>>>>>> make: *** Waiting for unfinished jobs....
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> On Thu, May 18, 2017 at 7:12 AM, James Paige <Bob at hamsterrepublic.com
>>>>>> > wrote:
>>>>>>
>>>>>>> Opps!
>>>>>>>
>>>>>>> It looks like revision 8705 broke the Android build. It fails with
>>>>>>> the following:
>>>>>>>
>>>>>>> fbc -c gfx_sdl.bas -o build/game-gfx_sdl.o -mt -exx -g -O 2 -target
>>>>>>> linux -arch 686 -d DATAFILES='"/usr/share/games/ohrrpgce"' -d
>>>>>>> IS_GAME -m game
>>>>>>> /home/james/src/ohr/wip/surface.bi(37) error 14: Expected
>>>>>>> identifier, found 'XYPair' in 'size as XYPair'
>>>>>>> scons: *** [build/game-gfx_sdl.o] Error 1
>>>>>>> scons: building terminated because of errors.
>>>>>>>
>>>>>>>
>>>>>>> Not sure why this works on other platforms but fails on Android
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Mon, May 1, 2017 at 9:29 AM, <subversion at hamsterrepublic.com>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> teeemcee
>>>>>>>> 2017-05-01 09:29:43 -0700 (Mon, 01 May 2017)
>>>>>>>> 165
>>>>>>>> Add Slice.{Pos,ScreenPos,Size}, Frame.size, Surface.size,
>>>>>>>> MouseInfo.pos, which are in union with the existing members
>>>>>>>>
>>>>>>>> Start to use these aliases around the codebase
>>>>>>>> ---
>>>>>>>> U   wip/allmodex.bas
>>>>>>>> U   wip/allmodex.bi
>>>>>>>> U   wip/common.rbas
>>>>>>>> U   wip/scriptcommands.bas
>>>>>>>> U   wip/slices.bas
>>>>>>>> U   wip/slices.bi
>>>>>>>> U   wip/surface.bi
>>>>>>>> _______________________________________________
>>>>>>>> Ohrrpgce mailing list
>>>>>>>> ohrrpgce at lists.motherhamster.org
>>>>>>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherh
>>>>>>>> amster.org
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>> _______________________________________________
>>>>>> Ohrrpgce mailing list
>>>>>> ohrrpgce at lists.motherhamster.org
>>>>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherh
>>>>>> amster.org
>>>>>>
>>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Ohrrpgce mailing list
>>>>> ohrrpgce at lists.motherhamster.org
>>>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>>>>
>>>>>
>>>>
>>>> _______________________________________________
>>>> Ohrrpgce mailing list
>>>> ohrrpgce at lists.motherhamster.org
>>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>>>
>>>>
>>>
>>> _______________________________________________
>>> Ohrrpgce mailing list
>>> ohrrpgce at lists.motherhamster.org
>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>>
>>>
>>
>> _______________________________________________
>> Ohrrpgce mailing list
>> ohrrpgce at lists.motherhamster.org
>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>
>>
>
> _______________________________________________
> Ohrrpgce mailing list
> ohrrpgce at lists.motherhamster.org
> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.motherhamster.org/pipermail/ohrrpgce-motherhamster.org/attachments/20170518/5cdca6a2/attachment-0001.htm>