Skip to content

Commit eddc8ac

Browse files
committed
rstudio: wrap quarto with pandoc instead of patching file paths
1 parent b1b43d3 commit eddc8ac

File tree

2 files changed

+9
-54
lines changed

2 files changed

+9
-54
lines changed

pkgs/applications/editors/rstudio/default.nix

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
{
22
lib,
33
stdenv,
4+
runCommand,
45
fetchzip,
56
fetchFromGitHub,
67
replaceVars,
@@ -53,6 +54,13 @@ let
5354
d: !(lib.hasAttr "dictFileName" d && lib.elem d.dictFileName (map (d: d.dictFileName) largeDicts))
5455
) hunspellDictionaries;
5556
dictionaries = largeDicts ++ otherDicts;
57+
58+
# rstudio assumes quarto bundles pandoc into bin/tools/
59+
quartoWrapper = runCommand "quarto-wrapper" { } ''
60+
mkdir -p $out/bin/tools
61+
ln -s ${lib.getExe quarto} $out/bin/quarto
62+
ln -s ${lib.getExe pandoc} $out/bin/tools/pandoc
63+
'';
5664
in
5765
stdenv.mkDerivation rec {
5866
pname = "RStudio";
@@ -130,7 +138,6 @@ stdenv.mkDerivation rec {
130138
./fix-resources-path.patch
131139
./ignore-etc-os-release.patch
132140
./dont-yarn-install.patch
133-
./dont-assume-pandoc-in-quarto.patch
134141
./boost-1.86.patch
135142
];
136143

@@ -170,15 +177,11 @@ stdenv.mkDerivation rec {
170177
done
171178
done
172179
173-
ln -s ${quarto} dependencies/quarto
180+
ln -s ${quartoWrapper} dependencies/quarto
174181
175182
# version in dependencies/common/install-mathjax
176183
ln -s ${mathJaxSrc} dependencies/mathjax-27
177184
178-
# version in CMakeGlobals.txt (PANDOC_VERSION)
179-
mkdir -p dependencies/pandoc/2.18
180-
ln -s ${lib.getBin pandoc}/bin/* dependencies/pandoc/2.18
181-
182185
# version in CMakeGlobals.txt (RSTUDIO_INSTALLED_NODE_VERSION)
183186
mkdir -p dependencies/common/node
184187
ln -s ${nodejs} dependencies/common/node/18.20.3

pkgs/applications/editors/rstudio/dont-assume-pandoc-in-quarto.patch

Lines changed: 0 additions & 48 deletions
This file was deleted.

0 commit comments

Comments
 (0)