You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "HyukjinKwon (via GitHub)" <gi...@apache.org> on 2025/12/26 00:20:29 UTC
[PR] Minor: [R] Add the explicit mirror to install R dependencies [arrow]
HyukjinKwon opened a new pull request, #48649:
URL: https://github.com/apache/arrow/pull/48649
### Rationale for this change
Otherwise, it complains that the mirror has to be manually set in, e.g., R profile.
Should be best to make it self-contained.
### What changes are included in this PR?
This PR fixes the R dev guides (https://arrow.apache.org/docs/4.0/r/articles/developing.html) to provide an explicit mirror when downloading related dependencies.
### Are these changes tested?
Manually tested when setting up the dev.
### Are there any user-facing changes?
No, dev-only.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: [R] Add the explicit mirror to install R dependencies [arrow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #48649:
URL: https://github.com/apache/arrow/pull/48649#issuecomment-3694068821
Revision: 2476bc319c8fc0297f089ba2ba65fbba0724ed29
Submitted crossbow builds: [ursacomputing/crossbow @ actions-c6e5649cbb](https://github.com/ursacomputing/crossbow/branches/all?query=actions-c6e5649cbb)
|Task|Status|
|----|------|
|test-r-devdocs|[](https://github.com/ursacomputing/crossbow/actions/runs/20541337980/job/59006051695)|
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: [R] Add the explicit mirror to install R dependencies [arrow]
Posted by "conbench-apache-arrow[bot] (via GitHub)" <gi...@apache.org>.
conbench-apache-arrow[bot] commented on PR #48649:
URL: https://github.com/apache/arrow/pull/48649#issuecomment-3733524867
After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 7fe39d470f6b1fb803772dbb6dccfb4d26af54fe.
There weren't enough matching historic benchmark results to make a call on whether there were regressions.
The [full Conbench report](https://github.com/apache/arrow/runs/60004908994) has more details.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: [R] Add the explicit mirror to install R dependencies [arrow]
Posted by "jonkeane (via GitHub)" <gi...@apache.org>.
jonkeane merged PR #48649:
URL: https://github.com/apache/arrow/pull/48649
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Minor: [R] Add the explicit mirror to install R dependencies [arrow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #48649:
URL: https://github.com/apache/arrow/pull/48649#issuecomment-3691846048
<!--
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->
Thanks for opening a pull request!
If this is not a [minor PR](https://github.com/apache/arrow/blob/main/CONTRIBUTING.md#Minor-Fixes). Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose
Opening GitHub issues ahead of time contributes to the [Openness](http://theapacheway.com/open/#:~:text=Openness%20allows%20new%20users%20the,must%20happen%20in%20the%20open.) of the Apache Arrow project.
Then could you also rename the pull request title in the following format?
GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}
or
MINOR: [${COMPONENT}] ${SUMMARY}
See also:
* [Other pull requests](https://github.com/apache/arrow/pulls/)
* [Contribution Guidelines - Contributing Overview](https://arrow.apache.org/docs/developers/overview.html)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: [R] Add the explicit mirror to install R dependencies [arrow]
Posted by "jonkeane (via GitHub)" <gi...@apache.org>.
jonkeane commented on code in PR #48649:
URL: https://github.com/apache/arrow/pull/48649#discussion_r2649181676
##########
r/vignettes/developers/setup.Rmd:
##########
@@ -189,7 +189,7 @@ checkout:
```{bash, save=run}
popd # To go back to the root directory of the project, from cpp/build
pushd r
-R -e "install.packages('remotes'); remotes::install_deps(dependencies = TRUE)"
+R -e 'install.packages("remotes", repos="https://cloud.r-project.org"); remotes::install_deps(dependencies = TRUE)'
Review Comment:
We might want to pull this out into the text. For R users / developers they will have a preferred repo source configured already. For example, if someone is on linux + pointed at Posit Public Package manager for binaries, this will override that + be much slower for them.
I've also triggered a devdocs CI job to see if this impacts that too.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: [R] Add the explicit mirror to install R dependencies [arrow]
Posted by "jonkeane (via GitHub)" <gi...@apache.org>.
jonkeane commented on PR #48649:
URL: https://github.com/apache/arrow/pull/48649#issuecomment-3694033862
@github actions crossbow submit test-r-devdocs
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: [R] Add the explicit mirror to install R dependencies [arrow]
Posted by "jonkeane (via GitHub)" <gi...@apache.org>.
jonkeane commented on PR #48649:
URL: https://github.com/apache/arrow/pull/48649#issuecomment-3694067703
@github-actions crossbow submit test-r-devdocs
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: [R] Add the explicit mirror to install R dependencies [arrow]
Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon commented on code in PR #48649:
URL: https://github.com/apache/arrow/pull/48649#discussion_r2649194504
##########
r/vignettes/developers/setup.Rmd:
##########
@@ -189,7 +189,7 @@ checkout:
```{bash, save=run}
popd # To go back to the root directory of the project, from cpp/build
pushd r
-R -e "install.packages('remotes'); remotes::install_deps(dependencies = TRUE)"
+R -e 'install.packages("remotes", repos="https://cloud.r-project.org"); remotes::install_deps(dependencies = TRUE)'
Review Comment:
Ah that's fine! I am fine with closing this too. I'll leave this to you 🫡
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org