Skip to content

install_github() error #16

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

Closed
dajmcdon opened this issue Feb 24, 2022 · 5 comments
Closed

install_github() error #16

dajmcdon opened this issue Feb 24, 2022 · 5 comments
Labels
bug Something isn't working

Comments

@dajmcdon
Copy link
Contributor

I can't currently install the package. I get the following error:

remotes::install_github("cmu-delphi/delphi-epidata-r")
The downloaded source packages are in
	‘/private/var/folders/fr/49035_g925n4f5g72mjntmth0000gn/T/RtmpvjwfKK/downloaded_packages’
✓  checking for file ‘/private/var/folders/fr/49035_g925n4f5g72mjntmth0000gn/T/RtmpvjwfKK/remotes7e0d69325f3d/cmu-delphi-delphi-epidata-r-5160ad9/DESCRIPTION’ ...
─  preparing ‘delphi.epidata’:
✓  checking DESCRIPTION meta-information ...
   Warning in grepl(e, files, perl = TRUE, ignore.case = TRUE) :
     PCRE pattern compilation error
   	'unrecognized character follows \'
   	at 'Makefile$'
   Error in grepl(e, files, perl = TRUE, ignore.case = TRUE) : 
     invalid regular expression '^\Makefile$'
   Execution halted
Error: Failed to install 'delphi.epidata' from GitHub:
  System command 'R' failed, exit status: 1, stdout & stderr were printed
@krivard krivard added the bug Something isn't working label Feb 24, 2022
@krivard
Copy link
Contributor

krivard commented Feb 24, 2022

Thanks for the report! This package is alpha and still has some build problems; I don't think we've even attempted to install using remotes yet.

@dshemetov
Copy link
Contributor

Hm, unable to reproduce on my Ubuntu machine, remotes installs fine.

@dajmcdon
Copy link
Contributor Author

I don't "need" it necessarily, but it's currently imported by Ryan's epiprocess. Maybe he has a different version on his machine (I'll check).

@krivard
Copy link
Contributor

krivard commented Feb 28, 2022

This may have been fixed by #14 -- would you try again?

@dajmcdon
Copy link
Contributor Author

dajmcdon commented Feb 28, 2022

Yup! Works now. A got a few warnings that I'll paste here in case its useful for testing, but otherwise it's good.

> devtools::install_github("cmu-delphi/delphi-epidata-r")
Downloading GitHub repo cmu-delphi/delphi-epidata-r@HEAD
✓  checking for file ‘/private/var/folders/fr/49035_g925n4f5g72mjntmth0000gn/T/RtmpNb9Sjf/remotes17d1b7bd6fe6/cmu-delphi-delphi-epidata-r-067fcf8/DESCRIPTION’ ...
─  preparing ‘delphi.epidata’:
✓  checking DESCRIPTION meta-information ...
─  checking for LF line-endings in source and make files and shell scripts
─  checking for empty or unneeded directories
   Omitted ‘LazyData’ from DESCRIPTION
─  creating default NAMESPACE file
─  building ‘delphi.epidata_1.0.0.tar.gz’
   
* installing *source* package ‘delphi.epidata’ ...
** using staged installation
** R
** byte-compile and prepare package for lazy loading
Warning: replacing previous import ‘jsonlite::unbox’ by ‘rlang::unbox’ when loading ‘delphi.epidata’
Warning: replacing previous import ‘jsonlite::flatten’ by ‘rlang::flatten’ when loading ‘delphi.epidata’
** help
No man pages found in package  ‘delphi.epidata’ 
*** installing help indices
** building package indices
** installing vignettes
** testing if installed package can be loaded from temporary location
Warning: replacing previous import ‘jsonlite::unbox’ by ‘rlang::unbox’ when loading ‘delphi.epidata’
Warning: replacing previous import ‘jsonlite::flatten’ by ‘rlang::flatten’ when loading ‘delphi.epidata’
** testing if installed package can be loaded from final location
Warning: replacing previous import ‘jsonlite::unbox’ by ‘rlang::unbox’ when loading ‘delphi.epidata’
Warning: replacing previous import ‘jsonlite::flatten’ by ‘rlang::flatten’ when loading ‘delphi.epidata’
** testing if installed package keeps a record of temporary installation path
* DONE (delphi.epidata)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants