On Wed, Mar 28, 2012 at 1:59 PM, Alon Bar-Lev <alon.bar...@gmail.com> wrote:
> 1. Rename resource includes to rch, see[1].
> 2. Use implicit rule for resources.
> 3. Fixup resources build using automake conventions.
> 4. Misc cleanups
>
> [1] http://comments.gmane.org/gmane.comp.gnu.binutils/56459
>
> Signed-off-by: Alon Bar-Lev <alon.bar...@gmail.com>
> ---
>  Makefile.am                |   51 ++++---
>  res/openvpn-gui-res-de.rc  |  343 ------------------------------------------
>  res/openvpn-gui-res-de.rch |  343 ++++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-dk.rc  |  342 ------------------------------------------
>  res/openvpn-gui-res-dk.rch |  342 ++++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-en.rc  |  340 -----------------------------------------
>  res/openvpn-gui-res-en.rch |  340 +++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-es.rc  |  339 -----------------------------------------
>  res/openvpn-gui-res-es.rch |  339 +++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-fi.rc  |  357 
> --------------------------------------------
>  res/openvpn-gui-res-fi.rch |  357 
> ++++++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-fr.rc  |  339 -----------------------------------------
>  res/openvpn-gui-res-fr.rch |  339 +++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-it.rc  |  338 -----------------------------------------
>  res/openvpn-gui-res-it.rch |  338 +++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-jp.rc  |  341 ------------------------------------------
>  res/openvpn-gui-res-jp.rch |  341 ++++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-nl.rc  |  339 -----------------------------------------
>  res/openvpn-gui-res-nl.rch |  339 +++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-no.rc  |  325 ----------------------------------------
>  res/openvpn-gui-res-no.rch |  325 ++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-pl.rc  |  342 ------------------------------------------
>  res/openvpn-gui-res-pl.rch |  342 ++++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-pt.rc  |  327 ----------------------------------------
>  res/openvpn-gui-res-pt.rch |  327 ++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-se.rc  |  336 -----------------------------------------
>  res/openvpn-gui-res-se.rch |  336 +++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res-tr.rc  |  341 ------------------------------------------
>  res/openvpn-gui-res-tr.rch |  341 ++++++++++++++++++++++++++++++++++++++++++
>  res/openvpn-gui-res.rc     |   28 ++--
>  30 files changed, 4791 insertions(+), 4786 deletions(-)
>  create mode 100644 m4/.keep
>  delete mode 100644 res/openvpn-gui-res-de.rc
>  create mode 100644 res/openvpn-gui-res-de.rch
>  delete mode 100644 res/openvpn-gui-res-dk.rc
>  create mode 100644 res/openvpn-gui-res-dk.rch
>  delete mode 100644 res/openvpn-gui-res-en.rc
>  create mode 100644 res/openvpn-gui-res-en.rch
>  delete mode 100644 res/openvpn-gui-res-es.rc
>  create mode 100644 res/openvpn-gui-res-es.rch
>  delete mode 100644 res/openvpn-gui-res-fi.rc
>  create mode 100644 res/openvpn-gui-res-fi.rch
>  delete mode 100644 res/openvpn-gui-res-fr.rc
>  create mode 100644 res/openvpn-gui-res-fr.rch
>  delete mode 100644 res/openvpn-gui-res-it.rc
>  create mode 100644 res/openvpn-gui-res-it.rch
>  delete mode 100644 res/openvpn-gui-res-jp.rc
>  create mode 100644 res/openvpn-gui-res-jp.rch
>  delete mode 100644 res/openvpn-gui-res-nl.rc
>  create mode 100644 res/openvpn-gui-res-nl.rch
>  delete mode 100644 res/openvpn-gui-res-no.rc
>  create mode 100644 res/openvpn-gui-res-no.rch
>  delete mode 100644 res/openvpn-gui-res-pl.rc
>  create mode 100644 res/openvpn-gui-res-pl.rch
>  delete mode 100644 res/openvpn-gui-res-pt.rc
>  create mode 100644 res/openvpn-gui-res-pt.rch
>  delete mode 100644 res/openvpn-gui-res-se.rc
>  create mode 100644 res/openvpn-gui-res-se.rch
>  delete mode 100644 res/openvpn-gui-res-tr.rc
>  create mode 100644 res/openvpn-gui-res-tr.rch
>
> diff --git a/Makefile.am b/Makefile.am
> index c1753e0..349db19 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -18,8 +18,7 @@
>  #  distribution); if not, write to the Free Software Foundation, Inc.,
>  #  59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
>
> -RCCOMPILE = $(WINDRES) $(DEFS) $(DEFAULT_INCLUDES) $(INCLUDES) \
> -    $(AM_CPPFLAGS) $(CPPFLAGS)
> +SUFFIXES = .rc
>
>  AUTOMAKE_OPTIONS = foreign 1.9
>
> @@ -31,6 +30,8 @@ MAINTAINERCLEANFILES = \
>        $(srcdir)/depcomp $(srcdir)/aclocal.m4 \
>        $(srcdir)/config.guess $(srcdir)/config.sub
>
> +EXTRA_DIST = .gitignore
> +
>  bin_PROGRAMS = openvpn-gui
>
>  dist_doc_DATA = \
> @@ -42,21 +43,20 @@ AM_CPPFLAGS = -D_UNICODE
>  AM_CFLAGS = -municode
>
>  openvpn_gui_RESOURCES = \
> -       res/openvpn-gui-res.rc \
> -       res/openvpn-gui-res-de.rc \
> -       res/openvpn-gui-res-dk.rc \
> -       res/openvpn-gui-res-en.rc \
> -       res/openvpn-gui-res-es.rc \
> -       res/openvpn-gui-res-fi.rc \
> -       res/openvpn-gui-res-fr.rc \
> -       res/openvpn-gui-res-it.rc \
> -       res/openvpn-gui-res-jp.rc \
> -       res/openvpn-gui-res-nl.rc \
> -       res/openvpn-gui-res-no.rc \
> -       res/openvpn-gui-res-pl.rc \
> -       res/openvpn-gui-res-pt.rc \
> -       res/openvpn-gui-res-se.rc \
> -       res/openvpn-gui-res-tr.rc \
> +       res/openvpn-gui-res-de.rch \
> +       res/openvpn-gui-res-dk.rch \
> +       res/openvpn-gui-res-en.rch \
> +       res/openvpn-gui-res-es.rch \
> +       res/openvpn-gui-res-fi.rch \
> +       res/openvpn-gui-res-fr.rch \
> +       res/openvpn-gui-res-it.rch \
> +       res/openvpn-gui-res-jp.rch \
> +       res/openvpn-gui-res-nl.rch \
> +       res/openvpn-gui-res-no.rch \
> +       res/openvpn-gui-res-pl.rch \
> +       res/openvpn-gui-res-pt.rch \
> +       res/openvpn-gui-res-se.rch \
> +       res/openvpn-gui-res-tr.rch \
>        res/connected.ico \
>        res/connecting.ico \
>        res/disconnected.ico \
> @@ -64,8 +64,6 @@ openvpn_gui_RESOURCES = \
>        res/reconnecting.ico \
>        res/openvpn-gui.manifest
>
> -EXTRA_DIST = $(openvpn_gui_RESOURCES)
> -
>  openvpn_gui_SOURCES = \
>        main.c main.h \
>        openvpn.c openvpn.h \
> @@ -82,14 +80,21 @@ openvpn_gui_SOURCES = \
>        openvpn_config.c \
>        openvpn_config.h \
>        chartable.h \
> -       openvpn-gui-res.h
> +       openvpn-gui-res.h \
> +       res/openvpn-gui-res.rc \
> +       $(openvpn_gui_RESOURCES)
>
> -openvpn_gui_LDFLAGS = -mwindows
>  openvpn_gui_LDADD = \
> -       openvpn-gui-res.o \
>        $(OPENSSL_CRYPTO_LIBS) \
>        -lws2_32 \
>        -lcomctl32
> +openvpn_gui_LDFLAGS = \
> +       -mwindows
> +
> +$(srcdir)/res/openvpn-gui-res.rc: $(openvpn_gui_RESOURCES) 
> $(srcdir)/openvpn-gui-res.h
> +       touch "$(srcdir)/res/openvpn-gui-res.rc"
>
> -openvpn-gui-res.o: $(openvpn_gui_RESOURCES) $(srcdir)/openvpn-gui-res.h
> +RCCOMPILE = $(WINDRES) $(DEFS) $(DEFAULT_INCLUDES) $(INCLUDES) \
> +       $(AM_CPPFLAGS) $(CPPFLAGS)
> +.rc.o:
>        $(RCCOMPILE) -i $< -o $@

OK, too large for mailing list again...
Full patch is at[1].

Alon

[1] 
https://github.com/alonbl/openvpn-gui/commit/8ceead092ca498906f289042ae23991fb651ebe1

Reply via email to