public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Maciej Barć" <xgqt@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: dev-lang/crystal/files/, dev-lang/crystal/
Date: Thu, 30 Jan 2025 17:44:13 +0000 (UTC)	[thread overview]
Message-ID: <1738259048.091bfc623c5ab135fab330145aaafcc822ca746e.xgqt@gentoo> (raw)

commit:     091bfc623c5ab135fab330145aaafcc822ca746e
Author:     Alfred Wingate <parona <AT> protonmail <DOT> com>
AuthorDate: Thu Jan 30 13:46:17 2025 +0000
Commit:     Maciej Barć <xgqt <AT> gentoo <DOT> org>
CommitDate: Thu Jan 30 17:44:08 2025 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=091bfc62

dev-lang/crystal: remove tests that don't work in the sandbox

Tests cannot remove SANDBOX_ variables from the environment.

Signed-off-by: Alfred Wingate <parona <AT> protonmail.com>
Signed-off-by: Maciej Barć <xgqt <AT> gentoo.org>

 dev-lang/crystal/crystal-1.15.0-r1.ebuild          |  1 +
 ...l-1.15.0-remove-enviroment-clearing-tests.patch | 28 ++++++++++++++++++++++
 2 files changed, 29 insertions(+)

diff --git a/dev-lang/crystal/crystal-1.15.0-r1.ebuild b/dev-lang/crystal/crystal-1.15.0-r1.ebuild
index 46a09f05b368..92214799d3c1 100644
--- a/dev-lang/crystal/crystal-1.15.0-r1.ebuild
+++ b/dev-lang/crystal/crystal-1.15.0-r1.ebuild
@@ -53,6 +53,7 @@ RDEPEND="
 PATCHES=(
 	"${FILESDIR}/${PN}-0.27.0-gentoo-tests-long-unix.patch"
 	"${FILESDIR}/${PN}-0.27.0-gentoo-tests-long-unix-2.patch"
+	"${FILESDIR}/${PN}-1.15.0-remove-enviroment-clearing-tests.patch"
 )
 
 src_prepare() {

diff --git a/dev-lang/crystal/files/crystal-1.15.0-remove-enviroment-clearing-tests.patch b/dev-lang/crystal/files/crystal-1.15.0-remove-enviroment-clearing-tests.patch
new file mode 100644
index 000000000000..11667b0da28a
--- /dev/null
+++ b/dev-lang/crystal/files/crystal-1.15.0-remove-enviroment-clearing-tests.patch
@@ -0,0 +1,28 @@
+Sandbox won't allow removing sandbox related environment variables.
+
+diff --git a/spec/std/process_spec.cr b/spec/std/process_spec.cr
+index 965ed1431..c32b00537 100644
+--- a/spec/std/process_spec.cr
++++ b/spec/std/process_spec.cr
+@@ -281,20 +281,6 @@ describe Process do
+     end
+ 
+     describe "environ" do
+-      it "clears the environment" do
+-        value = Process.run(*print_env_command, clear_env: true) do |proc|
+-          proc.output.gets_to_end
+-        end
+-        value.should eq("")
+-      end
+-
+-      it "clears and sets an environment variable" do
+-        value = Process.run(*print_env_command, clear_env: true, env: {"FOO" => "bar"}) do |proc|
+-          proc.output.gets_to_end
+-        end
+-        value.should eq("FOO=bar#{newline}")
+-      end
+-
+       it "sets an environment variable" do
+         value = Process.run(*print_env_command, env: {"FOO" => "bar"}) do |proc|
+           proc.output.gets_to_end
+


             reply	other threads:[~2025-01-30 17:44 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-30 17:44 Maciej Barć [this message]
  -- strict thread matches above, loose matches on Subject: below --
2023-02-12  0:48 [gentoo-commits] repo/gentoo:master commit in: dev-lang/crystal/files/, dev-lang/crystal/ Maciej Barć
2022-09-09  9:24 Sam James
2022-04-24 14:41 Michał Górny
2021-07-15  9:28 Sergei Trofimovich
2020-04-07 21:47 Sergei Trofimovich
2019-01-31  7:21 Sergei Trofimovich
2018-12-23 14:23 Sergei Trofimovich
2018-12-23 13:26 Sergei Trofimovich
2018-09-02 10:53 Sergei Trofimovich
2018-08-09 22:34 Sergei Trofimovich
2017-12-21 10:13 Sergei Trofimovich
2017-12-11 20:03 Michał Górny
2017-11-25 19:58 Sergei Trofimovich
2017-09-16 10:54 Sergei Trofimovich
2017-06-30  7:08 Sergei Trofimovich
2017-04-23 21:39 Sergei Trofimovich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1738259048.091bfc623c5ab135fab330145aaafcc822ca746e.xgqt@gentoo \
    --to=xgqt@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox