[PATCH app-rstart 00/15] Review configuration

Gaetan Nadon memsize at videotron.ca
Sun Jan 23 17:30:09 PST 2011


Replace cpp/sed combination with sed only for simple text substitution.

A number of substitutions were not required as the values are hard coded
in the makefile and have not changed for the last decade.

Replace the huge install FIXME shell code with proper Automake code.

Perform target separation in appropriate subdirs which greatly
simplifies Makefiles

Gaetan Nadon (15):
  config: use a single directory variable to represent $lib/X11/rstart
  config: -DSERVERNAME not required in client.cc and server.cpp
  config: -DBINDIR not required in server.cpp
  config: use sed rather than cpp to substitute directory values
  config: use sed rather than cpp to substitute rsh command value
  config: use sed rather than cpp to substitute config values
  config: -DPACKAGEname and -DENVPREFIX not required in config.in
  config: install commands using Automake
  config: install contexts files using Automake
  config: let the samples directory handle the samples
  config: let the specs directory handle the specs files
  config: add links from x and x11 from @Aliases
  config: move scripts into their own directory
  config: rstartd.real does not require linking to any library
  config: upgrade configure.ac

 .gitignore                           |    3 -
 Makefile.am                          |  165 ++--------------------------------
 client.cpp                           |  124 -------------------------
 commands/ListContexts                |    4 +-
 commands/ListGenericCommands         |    8 +-
 commands/Makefile.am                 |   16 ++++
 config.cpp                           |   34 -------
 configure.ac                         |   31 ++++---
 contexts/.gitignore                  |    1 +
 contexts/Makefile.am                 |   28 ++++++
 contexts/{x11r6 => x11r6.in}         |    4 +-
 samples/Makefile.am                  |   25 +++++
 scripts/.gitignore                   |    3 +
 scripts/Makefile.am                  |   26 ++++++
 contexts/x11r6 => scripts/config.in  |   20 +++--
 scripts/rstart.in                    |  124 +++++++++++++++++++++++++
 contexts/x11r6 => scripts/rstartd.in |   15 ++--
 server.cpp                           |   29 ------
 specs/Makefile.am                    |    8 ++
 19 files changed, 287 insertions(+), 381 deletions(-)
 delete mode 100644 client.cpp
 create mode 100644 commands/Makefile.am
 delete mode 100644 config.cpp
 create mode 100644 contexts/.gitignore
 create mode 100644 contexts/Makefile.am
 copy contexts/{x11r6 => x11r6.in} (95%)
 create mode 100644 samples/Makefile.am
 create mode 100644 scripts/.gitignore
 create mode 100644 scripts/Makefile.am
 copy contexts/x11r6 => scripts/config.in (68%)
 create mode 100644 scripts/rstart.in
 rename contexts/x11r6 => scripts/rstartd.in (76%)
 delete mode 100644 server.cpp
 create mode 100644 specs/Makefile.am



More information about the xorg-devel mailing list