Linux standalone: Add back the LOCPATH=/dev/null hack to avoid the system locale-archive being read.

Version mismatches between the system locale-archive and the glibc in the
bundle have been observed to cause git crashes.

Unfortunately, this causes locales to not be used in the linux standalone
bundle, as was the case until version 6.20160419.

glibc hardcodes the path to /usr/lib/locale/locale-archive and does not
let an environment variable cause a different locale-archive file to be used.

The only other option to include locales in the bundle would be to include
exploded locale definition directories in the bundle for a number of
locales, generated by localedef. But these take at least 300 kb per locale,
and there are a great many locales; it would be hundreds of megabytes to
include them all.

(Hmm, we could include localdef in the bundle, and check LANG in runshell
and compile the locale directories on the fly. This would need
/usr/share/i18n/ and /usr/lib/locale-archive to be included in the bundle.
It's.. doable.)

I know this is going to once again cause users of the bundle to complain
that eg, ls doesn't show their unicode filenames right. Better than strange
crashes though.
This commit is contained in:
Joey Hess 2016-10-04 12:52:36 -04:00
parent 9c35905bbe
commit c079811226
No known key found for this signature in database
GPG key ID: C910D9222512E3C7
4 changed files with 27 additions and 14 deletions

View file

@ -5,8 +5,6 @@
- Licensed under the GNU GPL version 3 or higher.
-}
{-# LANGUAGE CPP #-}
module Main where
import System.Environment
@ -48,6 +46,8 @@ mklibs top = do
writeFile (top </> "libdirs") (unlines libdirs)
writeFile (top </> "gconvdir")
(parentDir $ Prelude.head $ filter ("/gconv/" `isInfixOf`) glibclibs)
writeFile (top </> "localedir")
(parentDir $ Prelude.head $ filter ("/locale/" `isInfixOf`) glibclibs)
let linker = Prelude.head $ filter ("ld-linux" `isInfixOf`) libs'
mapM_ (installLinkerShim top linker) exes
@ -73,18 +73,6 @@ installLinkerShim top linker exe = do
[ "#!/bin/sh"
, "GIT_ANNEX_PROGRAMPATH=\"$0\""
, "export GIT_ANNEX_PROGRAMPATH"
#ifdef MIN_VERSION_GLASGOW_HASKELL
#if ! MIN_VERSION_GLASGOW_HASKELL(7,10,0,0)
#define NEED_LOCPATH_WORKAROUND
#endif
#else
#define NEED_LOCPATH_WORKAROUND
#endif
#ifdef NEED_LOCPATH_WORKAROUND
-- workaround for https://ghc.haskell.org/trac/ghc/ticket/7695
, "LOCPATH=/dev/null"
, "export LOCPATH"
#endif
, "exec \"$GIT_ANNEX_DIR/" ++ exelink ++ "\" --library-path \"$GIT_ANNEX_LD_LIBRARY_PATH\" \"$GIT_ANNEX_DIR/shimmed/" ++ base ++ "/" ++ base ++ "\" \"$@\""
]
modifyFileMode exe $ addModes executeModes

View file

@ -14,6 +14,12 @@ git-annex (6.20160924) UNRELEASED; urgency=medium
external special remotes, because running multiple git-annex commands
at the same time could already start multiple processes for the same
external special remotes.
* Linux standalone: Add back the LOCPATH=/dev/null hack to avoid
the system locale-archive being read. Version mismatches between the
system locale-archive and the glibc in the bundle have been observed
to cause git crashes. Unfortunately, this causes locales to not be
used in the linux standalone bundle, reverting back to pre-6.20160419
behavior.
-- Joey Hess <id@joeyh.name> Mon, 26 Sep 2016 16:46:19 -0400

View file

@ -0,0 +1,14 @@
[[!comment format=mdwn
username="joey"
subject="""comment 4"""
date="2016-10-04T16:23:12Z"
content="""
NFS was a red herring.
We've tracked this down to the linux standalone tarball not including
locale-archive, so it uses the system one, which may have a different
format and so breaks glibc.
I'm committing a fix that adds locale-archive to the linux standalone
tarball.
"""]]

View file

@ -100,6 +100,11 @@ export ORIG_GCONV_PATH
GCONV_PATH="$base/$(cat "$base/gconvdir")"
export GCONV_PATH
ORIG_LOCPATH="$LOCPATH"
export ORIG_LOCPATH
LOCPATH=/dev/null
export LOCPATH
ORIG_GIT_EXEC_PATH="$GIT_EXEC_PATH"
export ORIG_GIT_EXEC_PATH
GIT_EXEC_PATH="$base/git-core"