Skip to content

Commit

Permalink
rstudio: add patch for Boost 1.86
Browse files Browse the repository at this point in the history
I’m not going to bother trying to upstream this trivial fix since
they want full copyright assignment and a mailing address; it’s less
work for both parties for them to rederive it from scratch if and
when they get around to updating Boost themselves.
  • Loading branch information
emilazy committed Jan 6, 2025
1 parent eb69d60 commit 933242a
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 2 deletions.
13 changes: 13 additions & 0 deletions pkgs/applications/editors/rstudio/boost-1.86.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
diff --git a/src/cpp/core/json/JsonRpc.cpp b/src/cpp/core/json/JsonRpc.cpp
index d034ffecd7..4b08486517 100644
--- a/src/cpp/core/json/JsonRpc.cpp
+++ b/src/cpp/core/json/JsonRpc.cpp
@@ -193,7 +193,7 @@

bool JsonRpcResponse::hasAfterResponse() const
{
- return afterResponse_;
+ return !afterResponse_.empty();
}


5 changes: 3 additions & 2 deletions pkgs/applications/editors/rstudio/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
fetchFromGitHub,
replaceVars,
cmake,
boost183,
boost,
zlib,
openssl,
R,
Expand Down Expand Up @@ -85,7 +85,7 @@ stdenv.mkDerivation rec {

buildInputs =
[
boost183
boost
zlib
openssl
R
Expand Down Expand Up @@ -131,6 +131,7 @@ stdenv.mkDerivation rec {
./ignore-etc-os-release.patch
./dont-yarn-install.patch
./dont-assume-pandoc-in-quarto.patch
./boost-1.86.patch
];

postPatch = ''
Expand Down

0 comments on commit 933242a

Please sign in to comment.