Closed benz0li closed 1 year ago
@nx10 For your information:
Package ‘httpgd’ was removed from the CRAN repository.
That's unfortunate I provided a patch weeks ago which was rejected due to false positives in BH ( https://github.com/eddelbuettel/bh/issues/88 ) I will re-upload momentarily.
Sorry for the inconveniences everybody I moved to the US from Europe last week and still have somewhat limited internet access.
I will re-upload momentarily.
Thank you.
Sorry for the inconveniences everybody
No worries.
Just submitted it. Hope it will be up in a couple of days.
Archived once again?!? https://cran.r-project.org/incoming/archive/
Yes, I just got these comments
Please add \value to .Rd files regarding exported methods and explain
the functions results in the documentation. Please write about the
structure of the output (class) and also what the output means. (If a
function does not return a value, please document that too, e.g.
\value{No return value, called for side effects} or similar)
Missing Rd-tags:
hgd_test_pattern.Rd: \value
\dontrun{} should only be used if the example really cannot be executed
(e.g. because of missing additional software, missing API keys, ...) by
the user. That's why wrapping examples in \dontrun{} adds the comment
("# Not run:") as a warning for the user.
Does not seem necessary.
Please unwrap the examples if they are executable in < 5 sec, or replace
\dontrun{} with \donttest{}.
Please ensure that your functions do not write by default or in your
examples/vignettes/tests in the user's home filespace (including the
package directory and getwd()). This is not allowed by CRAN policies.
Please omit any default path in writing functions. In your
examples/vignettes/tests you can write to tempdir().
Please always make sure to reset to user's options(), working directory
or par() after you changed it in examples and vignettes and demos.
e.g.:
oldpar <- par(mfrow = c(1,2))
...
par(oldpar)
Please fix and resubmit.
I am not sure this is entirely accurate (and it was fine on CRAN for the last 2 years), but will take time time to make all the changes they want.
Edit: For reference this is the patch/1.3.1 branch https://github.com/nx10/httpgd/tree/bh-hotfix
I think I can help with roxygen comments and test modifications.
@eitsupi Any help is very appreciated! Just make sure to work on the branch I linked.
I will also be able to fully work on httpgd again soon.
I am installing httpgd 1.3.0 from archive and staying with BH 1.78.0-0 until this issue is resolved.
I did fix roxygen comments on #129. However, I could not figure out where the problem of writing temporary files was.
I could not figure out where the problem of writing temporary files was.
This is what I was hinting at with "I am not sure [the CRAN review] is entirely accurate". I do not think httpgd ever writes files as a side effect.
Thanks @eitsupi for the help! I will prepare everthing and resubmit later today.
We are up again. CRAN usually takes 1-3 days to build windows and Mac binaries: https://CRAN.R-project.org/package=httpgd
Wonderful!
By the way, why don't you set up R-universe? I think R-universe is more convenient than installing directly from GitHub as a workaround in such cases.
Yes R universe and anaconda are definitely planned but I have kept pushing them off for the 2.0 release. There is not much left for that and it will change all of the build logic.
Describe the bug
Package installation currently fails for v1.3.0: httpgd-1.3.0_install-log.txt
To Reproduce
Expected behavior
Installation succeeding.
Screenshots
none
Environment
Additional context