search

tidyverse / hms

A simple class for storing time-of-day values
https://hms.tidyverse.org/
Other
138 stars 25 forks source link

couldn't install hms #11

Closed Myfanwy closed 8 years ago

Myfanwy commented 8 years ago

Hi,

when I ran devtools::install_github('krlmlr/hms'), I got the following readout:

Downloading GitHub repo krlmlr/hms@master from URL https://api.github.com/repos/krlmlr/hms/zipball/master Installing hms Downloading GitHub repo jimhester/covr@v2 from URL https://api.github.com/repos/jimhester/covr/zipball/v2 Error: Could not find build tools necessary to build covr

I went to his repo page, but I'm not really sure where to start to build covr. Any help would be greatly appreciated, thanks!

krlmlr commented 8 years ago

@jimhester: Why is devtools attempting to grab a "remote" package that is listed in the Suggests?

krlmlr commented 8 years ago

Thanks. Could you please try

devtools::install_github("krlmlr/hms@feature/11-covr")

If you confirm this works, I'll merge, and installation without the @ suffix will work.

Myfanwy commented 8 years ago

@jimhester @krlmlr ok, thanks! Will try. I was also running an old version of devtools - but got the same error message after package update.

Myfanwy commented 8 years ago

devtools::install_github("krlmlr/hms@feature/11-covr") worked!

krlmlr commented 8 years ago

Just out of curiosity -- does the error also occur with the bleeding-edge version of devtools?

Myfanwy commented 8 years ago

is the bleeding edge version devtools::install_github("hadley/devtools") ? Because if so, I just tried to run that and got the same error I did with covr:

Downloading GitHub repo hadley/devtools@master from URL https://api.github.com/repos/hadley/devtools/zipball/master Error: Could not find build tools necessary to build devtools

Myfanwy commented 8 years ago

must be my XCode...?

krlmlr commented 8 years ago

Got it sorted out: It's fixed in https://github.com/hadley/devtools/pull/1067, not merged yet. About time ;-)

I can confirm that this version of devtools doesn't attempt installing covr.

jimhester commented 8 years ago

FWIW covr v2 has been merged to master and released to CRAN.

github-actions[bot] commented 3 years ago

This old thread has been automatically locked. If you think you have found something related to this, please open a new issue and link to this old issue if necessary.