Skip to content

[GRDM-XXXXX] DESCRIPTIONインストールをremotesベースに変更 (upstreamのcherry-pick)#28

Closed
yacchin1205 wants to merge 6 commits into
RCOSDP:masterfrom
yacchin1205:fix/r-remotes-install
Closed

[GRDM-XXXXX] DESCRIPTIONインストールをremotesベースに変更 (upstreamのcherry-pick)#28
yacchin1205 wants to merge 6 commits into
RCOSDP:masterfrom
yacchin1205:fix/r-remotes-install

Conversation

@yacchin1205

Copy link
Copy Markdown
Collaborator

DESCRIPTION型リポジトリのビルド失敗を修正するPRです。

DESCRIPTIONファイルを持つリポジトリのインストールは devtools::install_local(getwd()) で行われていますが、devtoolsは依存パッケージが多く、デフォルトのR (4.2.1) と最新のCRANスナップショットの組み合わせではPositのバイナリ提供が終了しているためソースビルドに落ち、依存の curl パッケージ等のビルド失敗によりdevtools自体がインストールされず、ビルドが失敗するようになっています(CIの tests/r/r-rspm-description-file が失敗。masterでも同様です)。

upstream (jupyterhub/repo2docker) では、devtools::install_local の廃止(deprecated)対応としてこの経路を依存ゼロの remotes ベースに変更済みのため、該当コミットをcherry-pickします。

途中でpakを経由しているのはupstreamの試行の経緯をそのまま保持しているためで、最終状態は「基本インストールに remotes を追加し、DESCRIPTIONのインストールを remotes::install_local() で行う」となります。remotesは依存パッケージを持たない純Rパッケージのため、バイナリ提供状況の影響を受けません。upstreamのCIでは同一のテストfixtureがこの構成でpassしています。

conflictの解決は1箇所のみです(673e1bda のRコマンドラインの文脈差)。

動作確認: tests/unit/test_r.py 9件のパス、および tests/r/r-rspm-description-file に対する生成Dockerfileが upstream と同一のインストールコマンドになることを確認済みです。CIでは r-rspm-description-file の回復が期待値です(r3.6/r4.0 fixtureの失敗は別問題「CS-rstudio-grdmインストール時のGitHub APIレートリミット」によるflakyなもので、本PRのスコープ外です)。

🤖 Generated with Claude Code

rgaiacs and others added 6 commits July 8, 2026 10:45
devtools::install_local has been deprecated
can't seem to _prevent_ writing cache
it leaves files in /tmp/ and ~/.cache/R/pkgcache/
pak doesn't work if caches are cleaned
@yacchin1205

Copy link
Copy Markdown
Collaborator Author

#27 に統合しました(CIの健全化として一本化)

@yacchin1205 yacchin1205 closed this Jul 8, 2026
@yacchin1205 yacchin1205 deleted the fix/r-remotes-install branch July 8, 2026 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants