From nobody Tue Aug 29 08:40:17 2023 X-Original-To: dev-commits-ports-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4RZgq561h7z4rXGp; Tue, 29 Aug 2023 08:40:17 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4RZgq55Yj8z4Tw3; Tue, 29 Aug 2023 08:40:17 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1693298417; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=q3C5Ixk0i2YxqBGtQFcU/qCmWCCobQaHyjsB8RWvpAw=; b=eqQ3+/C+OFMsVIX5rHOxeysF+lab6fQ06k/eDF+LI9Ey3ldDAu6xOsUx0ZNh1FWSdrsbBd CIEFRre6QxNvrjY8oxfLkxJ5AVMW5AjrYIdJKdP5ho9bS7ChZ/2nd69DYO2IMsHJMMbcVM Wu3Z5pERQaIQcZaru9xviPEWQqNKTsN4kVvHjlgB0MRQd3+BM9q7P6dHauR0+9jzZbBHTq 8V1Uc75Hgp0rWK9cw0E+7NpionXQbqCj0fsKJ2e7yGYvnsjHH3qFuBZFSPx5tKBQZEujuB j2fKsgSbYyqSsn6GxOATwnmZP8rabd6sP+c7qINqMUp79IrvBOsPPM/ZHKvGyw== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1693298417; a=rsa-sha256; cv=none; b=XSsmBzzheLm0kD4djv2Olv+al6+zJ31KPyPU9lYR99CSMr5AcLhFb1szmxIFlO0xNUASq5 zBFETPH84JwSwHlYRCzXb6MiQKs0oJKXVFKQDBEIBmM8m3Sr0DIcRdsL2j+oY+rOWDkSbQ Exi224u+d1xbcIindPN+91/u9M9xW2XBOke0zU7vZur+osHCr8yViJjfCX1auADhZpV7z3 ft1wx/4ltFTt4Yet5HTOkpY9WIswX7zWNXzmWuT6UTrYL7foWAtYkn3aThNnO3uJX9z0/f wMCvhXesbtJdRQkk/H+Nou1ibt2nGE8ZKbSH4+JENb4gGw1jR4XoPZHQhl300A== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1693298417; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=q3C5Ixk0i2YxqBGtQFcU/qCmWCCobQaHyjsB8RWvpAw=; b=SOC2GblKV1e7skocljtEt/loF9QqECn++V2AbY4HLKYzZ9LO2/+AbxreOLL2pDUaE5n5OH q7F9RW11PWeqcQfBrZekQxPoQkezcmzDa50qhbHyDfhVpvIaSolwuLKmlvqx/jirqNLJPp FbB2DBAARLQ41iWB72zo9DwgfqS+Y/nuoYAFKRlayowxXCUfZWN8+7lvrqOTMudKBXWHhB KT0WjuL9Sz2TMrq+XHWDuciNVmHXu/Qd599JdRPYxO2vphM9iQb6UzukcFZpxA0Cl6311m iGBprnX7+qkt7s6kQUrLY6NmjmlLpsL8VKlfXmdhi4Agke3m14G367DcZbDV/w== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4RZgq54dCZzkhb; Tue, 29 Aug 2023 08:40:17 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.17.1/8.17.1) with ESMTP id 37T8eHb0084226; Tue, 29 Aug 2023 08:40:17 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.17.1/8.17.1/Submit) id 37T8eHoY084218; Tue, 29 Aug 2023 08:40:17 GMT (envelope-from git) Date: Tue, 29 Aug 2023 08:40:17 GMT Message-Id: <202308290840.37T8eHoY084218@gitrepo.freebsd.org> To: ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-branches@FreeBSD.org From: Yuri Victorovich Subject: git: 8e04bdc72dc1 - 2023Q3 - devel/RStudio: Fix bug that RStudio-server didn't have /usr/local/bin in PATH List-Id: Commit messages for all branches of the ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-all@freebsd.org X-BeenThere: dev-commits-ports-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: yuri X-Git-Repository: ports X-Git-Refname: refs/heads/2023Q3 X-Git-Reftype: branch X-Git-Commit: 8e04bdc72dc12204468b207a2e2ec4c0198a9014 Auto-Submitted: auto-generated The branch 2023Q3 has been updated by yuri: URL: https://cgit.FreeBSD.org/ports/commit/?id=8e04bdc72dc12204468b207a2e2ec4c0198a9014 commit 8e04bdc72dc12204468b207a2e2ec4c0198a9014 Author: Yuri Victorovich AuthorDate: 2023-08-29 08:37:27 +0000 Commit: Yuri Victorovich CommitDate: 2023-08-29 08:40:15 +0000 devel/RStudio: Fix bug that RStudio-server didn't have /usr/local/bin in PATH This was previously fixed in post-install but the previous fix relied on the other path that changed, which eroded this fix. The new fix adds /usr/local/bin to PATH in the C++ code. (cherry picked from commit 9884a43067a47d9f4088306ad29d1ec373b25eee) --- devel/RStudio/Makefile | 12 +++++------ .../patch-src_cpp_session_modules_SessionGit.cpp | 24 ++++++++++++++++++++++ 2 files changed, 29 insertions(+), 7 deletions(-) diff --git a/devel/RStudio/Makefile b/devel/RStudio/Makefile index dcc39863ecb0..a6df10b46383 100644 --- a/devel/RStudio/Makefile +++ b/devel/RStudio/Makefile @@ -4,7 +4,7 @@ PORTNAME= RStudio DISTVERSIONPREFIX= v DISTVERSION= 2022.12.0+353 -PORTREVISION= 4 +PORTREVISION= 5 CATEGORIES= devel math java MASTER_SITES= https://s3.amazonaws.com/rstudio-buildtools/dictionaries/:dictionaries \ https://s3.amazonaws.com/rstudio-buildtools/:buildtools @@ -117,6 +117,10 @@ post-patch: @${REINPLACE_CMD} -e ' \ s|||' \ ${WRKSRC}/src/gwt/build.xml + # add $PREFIX/bin to PATH for the server daemon +.if ${FLAVOR:U} == server + @${REINPLACE_CMD} -e 's|%%PREFIX%%|${PREFIX}|' ${WRKSRC}/src/cpp/session/modules/SessionGit.cpp +.endif pre-build: @${CP} ${FILESDIR}/global-setenv.h ${WRKSRC}/src/cpp/desktop/ @@ -143,12 +147,6 @@ post-install: # Some functions expect the pandoc symlink. @${MKDIR} ${STAGEDIR}${PREFIX}/lib/${INSTALL_SUBDIR}/bin/pandoc @cd ${STAGEDIR}${PREFIX}/lib/${INSTALL_SUBDIR}/bin/pandoc && ${LN} -s ../../../../bin/pandoc - # Add $LOCALBASE/bin to PATH to allow RStudio-server to run gmake (R_HOME/etc/Renviron.site is installed, see https://stat.ethz.ch/R-manual/R-devel/library/base/html/Startup.html) -.if ${FLAVOR:U} == server - #@${MKDIR} ${STAGEDIR}${LOCALBASE}/lib/R/etc - #@${ECHO} "PATH=\"${LOCALBASE}/bin:\$$PATH\"" > ${STAGEDIR}${LOCALBASE}/lib/R/etc/Renviron.site # it should be this way, but this fails: https://github.com/rstudio/rstudio/issues/9815 - @${LN} -s ${LOCALBASE}/bin/${GMAKE} ${STAGEDIR}${PREFIX}/lib/rstudio-server/bin/postback/${GMAKE} # hack, should be as above, but at least gmake is in the path now -.endif # There is a variability in .js file names due to use of random numbers, so we use the automatic plist. @${SETENV} ${CO_ENV} ${SH} ${SCRIPTSDIR}/check-stagedir.sh makeplist | ${GREP} -v ^\/ | ${SED} -e 's|%%WWWDIR%%|www/rstudio| ; s|%%CMAKE_BUILD_TYPE%%|${CMAKE_BUILD_TYPE:tl}|' > ${TMPPLIST} diff --git a/devel/RStudio/files/patch-src_cpp_session_modules_SessionGit.cpp b/devel/RStudio/files/patch-src_cpp_session_modules_SessionGit.cpp new file mode 100644 index 000000000000..c72a605a439a --- /dev/null +++ b/devel/RStudio/files/patch-src_cpp_session_modules_SessionGit.cpp @@ -0,0 +1,24 @@ +- add path %%PREFIX%%/bin to PATH so that gmake and cmake executable would be found + +--- src/cpp/session/modules/SessionGit.cpp.orig 2023-08-29 01:47:30 UTC ++++ src/cpp/session/modules/SessionGit.cpp +@@ -130,6 +130,9 @@ core::system::ProcessOptions procOptions() + FilePath postbackDir = session::options().rpostbackPath().getParent(); + core::system::addToPath(&childEnv, postbackDir.getAbsolutePath()); + ++ // add $PREFIX/bin to path ++ core::system::addToPath(&childEnv, "%%PREFIX%%/bin"); ++ + options.workingDir = projects::projectContext().directory(); + + #ifdef _WIN32 +@@ -3370,6 +3373,9 @@ core::Error initialize() + } + + core::system::addToPath(postbackDir.getAbsolutePath()); ++ ++ // add $PREFIX/bin to path ++ core::system::addToPath("%%PREFIX%%/bin"); + + // add suspend/resume handler + addSuspendHandler(SuspendHandler(boost::bind(onSuspend, _2), onResume));