public inbox for glibc-cvs@sourceware.org
help / color / mirror / Atom feed
From: Joseph Myers <jsm28@sourceware.org>
To: glibc-cvs@sourceware.org
Subject: [glibc] Fix locale/tst-locale-locpath cross-testing when sshd sets LANG.
Date: Fri, 24 Jan 2020 17:23:00 -0000	[thread overview]
Message-ID: <20200124172324.31701.qmail@sourceware.org> (raw)

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=ef02e3c476ce76a1205744cdd396ac1eda1013d6

commit ef02e3c476ce76a1205744cdd396ac1eda1013d6
Author: Joseph Myers <joseph@codesourcery.com>
Date:   Fri Jan 24 17:23:07 2020 +0000

    Fix locale/tst-locale-locpath cross-testing when sshd sets LANG.
    
    The locale/tst-locale-locpath test unsets LANG, then runs a test with
    test_wrapper_env and expects LANG to remain unset for that test.  This
    does not work for cross-testing with cross-test-ssh.sh when sshd (on
    the system specified as an argument to cross-test-ssh.sh) is
    configured to have a default LANG setting.
    
    The general design used in cross testing, after commit
    8540f6d2a74fe9d67440535ebbcfa252180a3172 ("Don't require test wrappers
    to preserve environment variables, use more consistent environment.",
    6 June 2014), is that environment settings required by tests should be
    passed explicitly to $(test-wrapper-env).  This patch changes
    tst-locale-locpath.sh to pass an explicit LANG= rather than expecting
    "unset LANG" to be in effect for the program run under
    test_wrapper_env.  Note that this does slightly change the environment
    in which the test is run natively (empty LANG instead of unset LANG)
    but that difference does not appear relevant to what it is trying to
    test.
    
    Tested for Arm that this fixes the failure seen for that test in
    cross-testing.

Diff:
---
 locale/tst-locale-locpath.sh | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/locale/tst-locale-locpath.sh b/locale/tst-locale-locpath.sh
index bafdf91..f37d452 100644
--- a/locale/tst-locale-locpath.sh
+++ b/locale/tst-locale-locpath.sh
@@ -34,10 +34,8 @@ trap cleanup 0
 rm -rf "$testroot"
 mkdir -p $testroot
 
-unset LANG
-
 ${test_wrapper_env} \
-${run_program_env} LC_ALL=invalid-locale LOCPATH=does-not-exist \
+${run_program_env} LANG= LC_ALL=invalid-locale LOCPATH=does-not-exist \
 ${common_objpfx}elf/ld.so --library-path "$LIBPATH" \
   "${common_objpfx}locale/locale" \
   > "$testroot/stdout" 2> "$testroot/stderr"


                 reply	other threads:[~2020-01-24 17:23 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=20200124172324.31701.qmail@sourceware.org \
    --to=jsm28@sourceware.org \
    --cc=glibc-cvs@sourceware.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).