Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Laine Stump
On 11/3/23 8:41 AM, Andrea Bolognani wrote: On Fri, Nov 03, 2023 at 11:17:03AM +, Daniel P. Berrangé wrote: On Fri, Nov 03, 2023 at 04:13:12AM -0700, Andrea Bolognani wrote: On Fri, Nov 03, 2023 at 10:47:11AM +, Daniel P. Berrangé wrote: On Fri, Nov 03, 2023 at 10:41:51AM +,

Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Laine Stump
On 11/3/23 6:11 AM, Andrea Bolognani wrote: On Fri, Nov 03, 2023 at 12:00:30AM -0400, Laine Stump wrote: [*] There may be a better way to suppress this environment setting (maybe something done to prevent it from ever being added to the environment in the first place?), and that would

Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Andrea Bolognani
On Fri, Nov 03, 2023 at 11:17:03AM +, Daniel P. Berrangé wrote: > On Fri, Nov 03, 2023 at 04:13:12AM -0700, Andrea Bolognani wrote: > > On Fri, Nov 03, 2023 at 10:47:11AM +, Daniel P. Berrangé wrote: > > > On Fri, Nov 03, 2023 at 10:41:51AM +, Daniel P. Berrangé wrote: > > > > On Fri,

Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Daniel P . Berrangé
On Fri, Nov 03, 2023 at 04:13:12AM -0700, Andrea Bolognani wrote: > On Fri, Nov 03, 2023 at 10:47:11AM +, Daniel P. Berrangé wrote: > > On Fri, Nov 03, 2023 at 10:41:51AM +, Daniel P. Berrangé wrote: > > > On Fri, Nov 03, 2023 at 03:19:17AM -0700, Andrea Bolognani wrote: > > > > ld:

Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Andrea Bolognani
On Fri, Nov 03, 2023 at 10:47:11AM +, Daniel P. Berrangé wrote: > On Fri, Nov 03, 2023 at 10:41:51AM +, Daniel P. Berrangé wrote: > > On Fri, Nov 03, 2023 at 03:19:17AM -0700, Andrea Bolognani wrote: > > > ld: warning: ignoring duplicate libraries: '-lxml2' > > > > There's a similarish

Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Daniel P . Berrangé
On Fri, Nov 03, 2023 at 10:41:51AM +, Daniel P. Berrangé wrote: > On Fri, Nov 03, 2023 at 03:19:17AM -0700, Andrea Bolognani wrote: > > On Fri, Nov 03, 2023 at 03:11:07AM -0700, Andrea Bolognani wrote: > > > On Fri, Nov 03, 2023 at 12:00:30AM -0400, Laine Stump wrote: > > > > [*] There may be

Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Daniel P . Berrangé
On Fri, Nov 03, 2023 at 03:19:17AM -0700, Andrea Bolognani wrote: > On Fri, Nov 03, 2023 at 03:11:07AM -0700, Andrea Bolognani wrote: > > On Fri, Nov 03, 2023 at 12:00:30AM -0400, Laine Stump wrote: > > > [*] There may be a better way to suppress this environment setting > > > (maybe something

Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Andrea Bolognani
On Fri, Nov 03, 2023 at 03:11:07AM -0700, Andrea Bolognani wrote: > On Fri, Nov 03, 2023 at 12:00:30AM -0400, Laine Stump wrote: > > [*] There may be a better way to suppress this environment setting > > (maybe something done to prevent it from ever being added to the > > environment in

Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Andrea Bolognani
On Fri, Nov 03, 2023 at 12:00:30AM -0400, Laine Stump wrote: > [*] There may be a better way to suppress this environment setting > (maybe something done to prevent it from ever being added to the > environment in the first place?), and that would be fine too. This > patch does work

Re: [PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-03 Thread Daniel P . Berrangé
On Fri, Nov 03, 2023 at 12:00:30AM -0400, Laine Stump wrote: > This environment variable is supposedly set according to the contents > of ~/.CFUserTextEncoding, and certainly on MacOS 14 (Sonoma) it shows > up in the environment, causing commandtest to fail. However, the value > that is shown in

[PATCH] tests: ignore $__CF_USER_TEXT_ENCODING in env during commandtest

2023-11-02 Thread Laine Stump
This environment variable is supposedly set according to the contents of ~/.CFUserTextEncoding, and certainly on MacOS 14 (Sonoma) it shows up in the environment, causing commandtest to fail. However, the value that is shown in $__CF_USER_TEXT_ENCODING during the test 1) is not in the environment