New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support building for Android #106

Closed
jeffdiclemente opened this Issue Jul 15, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@jeffdiclemente
Contributor

jeffdiclemente commented Jul 15, 2016

The use of certain C++11 string functions, such as std::to_string, are not supported in the latest Android NDKs.

@jeffdiclemente jeffdiclemente self-assigned this Jul 15, 2016

@saschazelzer

This comment has been minimized.

Show comment
Hide comment
@saschazelzer

saschazelzer Jul 24, 2016

Member

Could you elaborate a little more about the general state of Android support? Is that something you were looking at, and if yes what else would be missing? Is it something we should support officially (by listing it as a supported platform in the readme file)?

Member

saschazelzer commented Jul 24, 2016

Could you elaborate a little more about the general state of Android support? Is that something you were looking at, and if yes what else would be missing? Is it something we should support officially (by listing it as a supported platform in the readme file)?

@jeffdiclemente

This comment has been minimized.

Show comment
Hide comment
@jeffdiclemente

jeffdiclemente Jul 25, 2016

Contributor

We support Android, iOS and linux-arm. Our requirement is that CppMicroServices does need to build and run on these platforms.

The general state of CppMicroServices support was that it was working for our usage up until the introduction of std::to_string (which is perfectly fine and should not hinder the use of C++11 string conversion functions). Currently only Android is broken.
We've dealt with the same situation before (i.e. missing C/C++ functions in Android). The typical solution is to implement the missing function and bring it into the std:: namespace such that clients don't know the difference.

I think its worth officially stating that Android and iOS are supported.
We haven't done extensive testing of CppMicroServices on these platforms yet, so I think that's worth mentioning in any statement of support (e.g. not heavily tested in production environment, etc...).

FYI; We have a short term cmake solution to perform cross platform builds of CppMicroServices for these platforms. It simply involves using the correct resource compiler binary for the host build platform. We haven't had the bandwidth to refine the solution and submit it yet. That's coming soon.

Contributor

jeffdiclemente commented Jul 25, 2016

We support Android, iOS and linux-arm. Our requirement is that CppMicroServices does need to build and run on these platforms.

The general state of CppMicroServices support was that it was working for our usage up until the introduction of std::to_string (which is perfectly fine and should not hinder the use of C++11 string conversion functions). Currently only Android is broken.
We've dealt with the same situation before (i.e. missing C/C++ functions in Android). The typical solution is to implement the missing function and bring it into the std:: namespace such that clients don't know the difference.

I think its worth officially stating that Android and iOS are supported.
We haven't done extensive testing of CppMicroServices on these platforms yet, so I think that's worth mentioning in any statement of support (e.g. not heavily tested in production environment, etc...).

FYI; We have a short term cmake solution to perform cross platform builds of CppMicroServices for these platforms. It simply involves using the correct resource compiler binary for the host build platform. We haven't had the bandwidth to refine the solution and submit it yet. That's coming soon.

@kevinleeMW

This comment has been minimized.

Show comment
Hide comment
@kevinleeMW

kevinleeMW Aug 3, 2016

Contributor

closed with PR #111

Contributor

kevinleeMW commented Aug 3, 2016

closed with PR #111

@kevinleeMW kevinleeMW closed this Aug 3, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment