Bug 5175

Created attachment 3914 [details] [review]
patch to font-util's configure.ac & Makefile.am to make mapdir configurable

Created attachment 3937 [details] [review]
update patch so configure displays default mapdir path

I have now tested this patch against CVS, following the directions at
http://xorg.freedesktop.org/wiki/ModularDevelopersGuide, performing both make
install & make distcheck.

After discussing this bug with mharris on IRC, he suggested changing font-util's
mapdir to use $datadir instead of $libdir, since the encodings files are
architecture-independent data files and therefore according to the FHS belong in
$datadir.  Instead of adding a --with-mapdir configure option,
X.org/distributors would be free to set datadir=libdir to get the old behavior.

mharris also proposed making encodings & util parallel dirs to fonts instead of
subdirectories.  So the structure would look like
$datadir/X11/{encodings,fonts,util} instead of
$datadir/X11/fonts/{encodings,util}.  This way the fonts directory would only
contain fonts and distributors wouldn't have to do ugly hacks to ignore special
subdirectories in fonts.

Implementing this change requires patching three modules: libfontenc, encodings,
and font-util.  I will be attaching those patches now.

The only concern is for those distributors that will be upset about moving the
default location of these files, but it seems like the switch to modular X.org
would be the best time to cause such a break, and the distributors would be free
to patch the sources to get the old behavior.

The only remaining question I have is if the directory for font-util's map files
should be called util or maps.  Calling it maps would clearly denote its purpose
and hopefully prevent any abuse that might occurs with a generic directory name
such as util, which could end up with lots of random things in it from many
different packages.  But if the util directory were reserved specifically for
font-util, then I would agree to using util for storing the map files (& any
other architecture-independent data relevant to font-util).

Created attachment 3955 [details] [review]
patch to encodings to move to datadir & toplevel encodings dir

Created attachment 3956 [details] [review]
patch to font-util to move to datadir & toplevel util dir

Created attachment 3957 [details] [review]
patch to libfontenc to move to datadir & toplevel encodings dir

Created attachment 3972 [details] [review]
update patch to encodings to also adjust large encodingsdir

I would think there should be a way to get the encodingsdirs set in Makefile.am
in the large subdirectory to be based off that in the top-level Makefile.am,
but I don't know the correct autofoo to do that, so this patch just hardcodes
it.

Comment on attachment 3914 [details] [review]
patch to font-util's configure.ac & Makefile.am to make mapdir configurable

Requesting a commit on the original patch, as it's less drastic and shouldn't
require much thought or discussion.

Comment on attachment 3914 [details] [review]
patch to font-util's configure.ac & Makefile.am to make mapdir configurable

Removing request, as the pkgconfig file also needs to be changed from mapdir to
MAPDIR.

Created attachment 4051 [details] [review]
configurable-mapdir.patch

Update from comment #4; also fix fontutil.pc.in so other packages can detect
mapdir.

Comment on attachment 4051 [details] [review]
configurable-mapdir.patch

Fix pkgconfig file as well.

I've checked in the patches to make the dirs configurable.  Thanks!
We can discuss changing the default locations of the font files for 7.1.