[PATCH xorg-gtest 2/3] Install configuration file in /etc/X11 instead of /ush/share/xorg/gtest
Jamey Sharp
jamey at minilop.net
Mon Jan 30 11:27:21 PST 2012
This should be fixed by commit ead968a4300c0adeff89b9886e888b6d284c75cc,
already on master--right?
Jamey
On Mon, Jan 30, 2012 at 03:32:19PM -0200, Daniel d'Andrada wrote:
> This is another enabler for making it possible to run Xorg without
> root privileges.
>
> The -config parameter in Xorg doesn't accept files in paths outside
> Xorg's standard search path for config files when run as non-root.
> See -config in "man Xorg" for more info.
>
> You have to run configure with "--sysconfdir=/etc" so that our conf
> file lands in Xorg's standard path for configuration files, namely /etc/X11
>
> Signed-off-by: Daniel d'Andrada <daniel.dandrada at canonical.com>
>
> diff --git a/Makefile.am b/Makefile.am
> index d10bca8..24f6f7b 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -20,13 +20,12 @@ library_include_HEADERS = \
> include/xorg/gtest/process.h \
> include/xorg/gtest/test.h
>
> -library_datadir = $(datadir)/xorg/gtest
> -library_data_DATA = conf/dummy.conf
> +library_datadir = $(sysconfdir)/X11
> +library_data_DATA = conf/gtest-dummy.conf
>
> libxorg_gtest_main_la_CPPFLAGS = \
> $(AM_CPPFLAGS) \
> - $(GTEST_CPPFLAGS) \
> - -DDUMMY_CONF_PATH="\"$(library_datadir)/dummy.conf\""
> + $(GTEST_CPPFLAGS)
>
> libxorg_gtest_la_LDFLAGS = $(X11_LIBS)
> libxorg_gtest_main_la_LDFLAGS = $(X11_LIBS)
> diff --git a/configure.ac b/configure.ac
> index 9d7b36b..d7d1682 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -36,7 +36,7 @@ AS_IF([test "x$ac_cv_lib_gtest_main" != xyes],
>
> AC_SUBST([GTEST_CPPFLAGS])
>
> -AC_SUBST(DUMMY_CONF_PATH, "$datadir/xorg/gtest/dummy.conf")
> +AC_SUBST(DUMMY_CONF_PATH, "$sysconfdir/X11/gtest-dummy.conf")
>
> AC_CONFIG_FILES([Makefile
> xorg-gtest.pc])
> diff --git a/src/main.cpp b/src/main.cpp
> index eb6e728..2d45468 100644
> --- a/src/main.cpp
> +++ b/src/main.cpp
> @@ -26,6 +26,7 @@
> #include "xorg/gtest/environment.h"
>
> #define DEFAULT_XORG_LOGFILE "/tmp/Xorg.GTest.log"
> +#define DEFAULT_XORG_CONFIG "gtest-dummy.conf"
>
> namespace {
>
> @@ -50,7 +51,7 @@ const struct option longopts[] = {
>
> int main(int argc, char *argv[]) {
> /* Default Xorg dummy conf path. */
> - std::string xorg_conf_path(DUMMY_CONF_PATH);
> + std::string xorg_conf_path(DEFAULT_XORG_CONFIG);
>
> std::string xorg_logfile_path(DEFAULT_XORG_LOGFILE);
>
> @@ -102,7 +103,8 @@ int main(int argc, char *argv[]) {
> std::cout << "\nAdditional options:\n";
> std::cout << " --no-dummy-server: Use the currently running
> X server "
> "for testing\n";
> - std::cout << " --xorg-conf: Path to xorg dummy configuration
> file\n";
> + std::cout << " --xorg-conf: Path to xorg dummy configuration
> file. See -config in \"man Xorg\".\n"
> + " Its default value is
> "DEFAULT_XORG_CONFIG".\n";
> std::cout << " --server: Path to X server executable\n";
> std::cout << " --xorg-display: xorg dummy display port\n";
> std::cout << " --xorg-logfile: xorg logfile filename. See
> -logfile in \"man Xorg\".\n"
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120130/60fddb4c/attachment-0001.pgp>
More information about the xorg-devel
mailing list