Skip to content

Commit

Permalink
rstudio: don't use libsForQt5.callPackage
Browse files Browse the repository at this point in the history
  • Loading branch information
TomaSajt committed Dec 5, 2024
1 parent 77872ee commit 12c54fe
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 15 deletions.
23 changes: 9 additions & 14 deletions pkgs/applications/editors/rstudio/default.nix
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
{
lib,
stdenv,
mkDerivation,
fetchurl,
fetchFromGitHub,
makeDesktopItem,
Expand All @@ -11,11 +10,7 @@
zlib,
openssl,
R,
qtbase,
qtxmlpatterns,
qtsensors,
qtwebengine,
qtwebchannel,
libsForQt5,
quarto,
libuuid,
hunspellDicts,
Expand All @@ -29,7 +24,6 @@
soci,
postgresql,
nodejs,
qmake,
server ? false, # build server version
sqlite,
pam,
Expand Down Expand Up @@ -74,7 +68,7 @@ let

description = "Set of integrated tools for the R language";
in
(if server then stdenv.mkDerivation else mkDerivation) (
stdenv.mkDerivation (
rec {
inherit
pname
Expand All @@ -97,6 +91,7 @@ in
]
++ lib.optionals (!server) [
copyDesktopItems
libsForQt5.wrapQtAppsHook
];

buildInputs =
Expand All @@ -119,11 +114,11 @@ in
]
else
[
qtbase
qtxmlpatterns
qtsensors
qtwebengine
qtwebchannel
libsForQt5.qtbase
libsForQt5.qtxmlpatterns
libsForQt5.qtsensors
libsForQt5.qtwebengine
libsForQt5.qtwebchannel
]
);

Expand All @@ -139,7 +134,7 @@ in
"-DCMAKE_INSTALL_PREFIX=${placeholder "out"}/lib/rstudio"
]
++ lib.optionals (!server) [
"-DQT_QMAKE_EXECUTABLE=${qmake}/bin/qmake"
"-DQT_QMAKE_EXECUTABLE=${libsForQt5.qmake}/bin/qmake"
];

# Hack RStudio to only use the input R and provided libclang.
Expand Down
2 changes: 1 addition & 1 deletion pkgs/top-level/all-packages.nix
Original file line number Diff line number Diff line change
Expand Up @@ -15491,7 +15491,7 @@ with pkgs;
backend = "wayland";
};

rstudio = libsForQt5.callPackage ../applications/editors/rstudio {
rstudio = callPackage ../applications/editors/rstudio {
jdk = jdk8;
};

Expand Down

0 comments on commit 12c54fe

Please sign in to comment.