Skip to content

Commit 08acbfa

Browse files
committed
Wrapper : Reintroduce LC_NUMERIC=C
I removed this in e323014 after verifying that the OCIO bug it worked around was now fixed properly. But what I didn't realise is that we have our _own_ equivalent bug in ImageEngine/cortex#1338. So I'm reverting the change until we have time to fix properly in Cortex. Fixes GafferHQ#5158
1 parent 84733df commit 08acbfa

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

Changes.md

+7
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,13 @@
11
1.2.x.x (relative to 1.2.0.2)
22
=======
33

4+
Fixes
5+
-----
6+
7+
- Locale :
8+
- Reintroduced `LC_NUMERIC=C` environment variable assignment to wrapper. This was removed in `1.2.0.0` but is still necessary due to a [bug in Cortex](https://github.com/ImageEngine/cortex/issues/1338).
9+
- Fixed serialisation of `.gfr` files in locales with non-default numeric formatting (#5158).
10+
411
Documentation
512
-------------
613

bin/gaffer

+6
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,12 @@ if [[ $1 == -psn_0_* ]] ; then
5050
shift
5151
fi
5252

53+
# Work around https://github.com/ImageEngine/cortex/issues/1338, which
54+
# causes bad serialisations in certain locales.
55+
##########################################################################
56+
57+
export LC_NUMERIC=C
58+
5359
# Find where this script is located, resolving any symlinks that were used
5460
# to invoke it. Set GAFFER_ROOT based on the script location.
5561
##########################################################################

0 commit comments

Comments
 (0)