Skip to content
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

Update api.cpp for ANDROID #796

Merged
merged 1 commit into from
Jan 6, 2018
Merged

Conversation

kendonB
Copy link
Contributor

@kendonB kendonB commented Jan 6, 2018

People are compiling Rcpp on Android so they can work on the bus (and the toilet):

termux/termux-packages#250 (comment)

This patch should help with installing Rcpp on that platform. Credit @its-pointless:
https://github.com/its-pointless/gcc_termux/blob/master/R-patches/Rcpp/api.cpp.patch

People are compiling Rcpp on Android so they can work on the bus (and the toilet):

termux/termux-packages#250 (comment)

This patch should help with install Rcpp on that platform. Credit @its-pointless:
https://github.com/its-pointless/gcc_termux/blob/master/R-patches/Rcpp/api.cpp.patch
@eddelbuettel
Copy link
Member

I wondered about that. I followed that end-of-December post and also installed the basic Debian image on my phone, but I got disenchanted when I could not upgrade through to Debian testing. However, on that platform everything already seems to work with the plain source. One of the other branches of the #define must be covering this.

I'll just wait for the Travis check but I think I can merge this as is as I see no possible harm from the PR and potential upside -- so thumbs up and thanks!

@eddelbuettel
Copy link
Member

I actually have termux installed too. Guess I might not have known about its apt capabilities.

Are you running R there? When I do pkg search r-base nothing comes back as found.

@eddelbuettel eddelbuettel merged commit dfe552b into RcppCore:master Jan 6, 2018
@kendonB
Copy link
Contributor Author

kendonB commented Jan 6, 2018

pkg search r-cran. Install info is scattered through termux/termux-packages#250 and https://github.com/its-pointless/gcc_termux

@eddelbuettel
Copy link
Member

Still nuttin'. Using https://termux.net stable InRelease. Do I need to switch?

Also, if you have an email address you'd like me to use in ChangeLog please send (or DM) it (or reply here). Else the (useless) default from GH goes in...

Gottan run an errand now...

@kendonB
Copy link
Contributor Author

kendonB commented Jan 6, 2018

You likely have to add the its-pointless repo: see the readme here: https://github.com/its-pointless/gcc_termux

@eddelbuettel
Copy link
Member

I guess I am even fiurther behind as I have no /etc/apt/. Hm. Will read up -- thanks for the pointers!

@kendonB
Copy link
Contributor Author

kendonB commented Jan 6, 2018

Mine is in ~/../usr/etc/apt

@eddelbuettel
Copy link
Member

Oh wow that worked and I even see r-cran-rquantlib, the fattest of all fat packages ...

@eddelbuettel
Copy link
Member

Yay. R prompt. Rcpp installed. [ Annoying that he renames the package, and also mungles the depends on libicu. ]

Many thanks for this. Very cool.

eddelbuettel added a commit that referenced this pull request Jan 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants