search

OpenDDS / OpenDDS-Android

An OpenDDS Build Matrix for Android
8 stars 3 forks source link

Test also r13b with API 21 which is a valid combination which should work #1

Closed jwillemsen closed 5 years ago

jwillemsen commented 5 years ago 
* shell_matrix.sh:
jwillemsen commented 5 years ago

Would it be something to add a new repo also to the DOCGroup for ACE/TAO so that we can use our scripts to also compile there with Android? Maybe use a travis-ci weekly schedule, no need to do it on each PR, that would increase build times much more

iguessthislldo commented 5 years ago

That would probably be a good idea if you think Android cross API and NDK compatibility should be a larger priority. It should be noted that this repo wasn't made for long term development for ACE/TAO/OpenDDS Android compatibility. It was made for quick overview of what was broken and to get results for the footprint study of OpenDDS on various platforms that we were doing.

jwillemsen commented 5 years ago

Would be good when we can do this for ace/tao/opendds regularly, would reduce risks for all users. Maybe move this to azure devops, there we get 10 parallel jobs for free

iguessthislldo commented 5 years ago

The build is failing for the combination you added, is this combination working on you machine?

jwillemsen commented 5 years ago

I think it is an error in the OpenDDS code, see https://github.com/objectcomputing/OpenDDS/pull/976. Probably for portability OpenDDS could use the ACE wrappers for this functionality. Probably ACE_HAS_POSIX_SEM and ACE_HAS_POSIX_SEM_TIMEOUT have to be defined from some NDK/API level but not sure which ones exactly

jwillemsen commented 5 years ago

After the recent OpenDDS changes this now compiles with all versions