Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] support/testing: set $USER in rust tests
@ 2018-04-28 22:14 Eric Le Bihan
  2018-04-29  7:58 ` Thomas Petazzoni
  2018-05-01  7:32 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Eric Le Bihan @ 2018-04-28 22:14 UTC (permalink / raw)
  To: buildroot

When the run-time tests to build rust and rust-bin packages are run via Docker,
the $USER environment variable is not set, which makes cargo fail when
initializing the test project.

So add it to make cargo happy.

Signed-off-by: Eric Le Bihan <eric.le.bihan.dev@free.fr>
---
 support/testing/tests/package/test_rust.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/support/testing/tests/package/test_rust.py b/support/testing/tests/package/test_rust.py
index e6c0de2214..2dc814f99d 100644
--- a/support/testing/tests/package/test_rust.py
+++ b/support/testing/tests/package/test_rust.py
@@ -21,6 +21,7 @@ class TestRustBase(infra.basetest.BRTest):
     def build_test_prog(self):
         hostdir = os.path.join(self.builddir, 'host')
         env = os.environ.copy()
+        env["USER"] = "br-user"
         env["PATH"] = "{}:".format(os.path.join(hostdir, 'bin')) + env["PATH"]
         env["CARGO_HOME"] = os.path.join(hostdir, 'usr', 'share', 'cargo')
         env["RUST_TARGET_PATH"] = os.path.join(hostdir, 'etc', 'rustc')
-- 
2.14.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH 1/1] support/testing: set $USER in rust tests
  2018-04-28 22:14 [Buildroot] [PATCH 1/1] support/testing: set $USER in rust tests Eric Le Bihan
@ 2018-04-29  7:58 ` Thomas Petazzoni
  2018-05-01  7:32 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2018-04-29  7:58 UTC (permalink / raw)
  To: buildroot

Hello,

On Sun, 29 Apr 2018 00:14:13 +0200, Eric Le Bihan wrote:
> When the run-time tests to build rust and rust-bin packages are run via Docker,
> the $USER environment variable is not set, which makes cargo fail when
> initializing the test project.
> 
> So add it to make cargo happy.
> 
> Signed-off-by: Eric Le Bihan <eric.le.bihan.dev@free.fr>
> ---
>  support/testing/tests/package/test_rust.py | 1 +
>  1 file changed, 1 insertion(+)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH 1/1] support/testing: set $USER in rust tests
  2018-04-28 22:14 [Buildroot] [PATCH 1/1] support/testing: set $USER in rust tests Eric Le Bihan
  2018-04-29  7:58 ` Thomas Petazzoni
@ 2018-05-01  7:32 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-05-01  7:32 UTC (permalink / raw)
  To: buildroot

>>>>> "Eric" == Eric Le Bihan <eric.le.bihan.dev@free.fr> writes:

 > When the run-time tests to build rust and rust-bin packages are run via Docker,
 > the $USER environment variable is not set, which makes cargo fail when
 > initializing the test project.

 > So add it to make cargo happy.

 > Signed-off-by: Eric Le Bihan <eric.le.bihan.dev@free.fr>

Committed to 2018.02.x, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-05-01  7:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-04-28 22:14 [Buildroot] [PATCH 1/1] support/testing: set $USER in rust tests Eric Le Bihan
2018-04-29  7:58 ` Thomas Petazzoni
2018-05-01  7:32 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox