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

RSQlite master requires Rcpp master as well. #79

Merged
merged 1 commit into from
Feb 4, 2015

Conversation

bbroeksema
Copy link
Contributor

Description says it all. I wasn't able to install RSQlite without using Rcpp from master.

@jjallaire
Copy link
Contributor

Okay, got it. I'm not able to merge this however I'm sure one of the owners will soon.

hadley added a commit that referenced this pull request Feb 4, 2015
RSQlite master requires Rcpp master as well.
@hadley hadley merged commit 991c1ab into r-dbi:master Feb 4, 2015
@hadley
Copy link
Member

hadley commented Feb 4, 2015

@jjallaire it's a bit of a pain that we have to repeat install instructions in the README and in .travis.yml. I think the only way to fix this is add some extra metadata to the DESCRIPTION that devtools::install_deps() can use

troels pushed a commit to troels/RSQLite that referenced this pull request Nov 27, 2019
troels pushed a commit to troels/RSQLite that referenced this pull request Nov 27, 2019
- Improve DBI compatibility (r-dbi#24, r-dbi#41, r-dbi#52, r-dbi#53, r-dbi#61, r-dbi#62, r-dbi#70, r-dbi#79, r-dbi#104, r-dbi#121).
- Define `timegm` for Windows and add missing function (r-dbi#136).
- The time zone is set by executing `SET TIMEZONE='UTC'` instead of setting an environment variable (r-dbi#136).
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants