Merge lp:~hyperair/ewns-viewer/autotools into lp:ewns-viewer

[Chow Loong Jin]

This branch replaces the Makefile present with an Autotools build system. This provides all of the Autotools features for free, including:-
 - Install and run the program anywhere -- just configure with a different --prefix
 - Install the program to a separate root (make install DESTDIR=/path), which makes packaging possible with make install.
 - Easy cross-compilation (./configure --host=$arch_triplet)

However, it still does not compile as there appears to be a missing type called SplashScreen, introduced in revision 85. Perhaps you forgot to commit a file?

[Aleksandar Jovanov (ajovanov93)]

I changed the build system completely today so you will need to change a lot of things.
I am gracious that you made this build system.
I have not seen your autotols files but there will be a need for a lot of changes because of the fact that parsers are now loaded from *.so files in ~/.config/ewns-viewer/parsers (and are compiled separately of course).
Do you want to work more on ewns-viewer?
I plan to submit this app to ubuntu software center for a price.
I am not very willing to share the money unless you do something really big.
Hopefully you understand.
If you are still interested in helping me here is a short summary of things needed to get done :
Localization support,
Parsers test (many of them crash),
Better UI for the ewns view (something like transmission's torrent view).
That should be it.Have a nice day.

________________________________
 From: Chow Loong Jin <email address hidden>
To: <email address hidden>
Sent: Monday, July 9, 2012 7:40 AM
Subject: [Merge] lp:~hyperair/ewns-viewer/autotools into lp:ewns-viewer

Chow Loong Jin has proposed merging lp:~hyperair/ewns-viewer/autotools into lp:ewns-viewer.

Requested reviews:
  Aleksandar Jovanov (ajovanov93)

For more details, see:
https://code.launchpad.net/~hyperair/ewns-viewer/autotools/+merge/113899

This branch replaces the Makefile present with an Autotools build system. This provides all of the Autotools features for free, including:-
- Install and run the program anywhere -- just configure with a different --prefix
- Install the program to a separate root (make install DESTDIR=/path), which makes packaging possible with make install.
- Easy cross-compilation (./configure --host=$arch_triplet)

However, it still does not compile as there appears to be a missing type called SplashScreen, introduced in revision 85. Perhaps you forgot to commit a file?
--
https://code.launchpad.net/~hyperair/ewns-viewer/autotools/+merge/113899
You are requested to review the proposed merge of lp:~hyperair/ewns-viewer/autotools into lp:ewns-viewer.

[Chow Loong Jin (hyperair)]

On 10/07/2012 04:16, Aleksandar Jovanov wrote:
> I changed the build system completely today so you will need to change a lot
> of things. I am gracious that you made this build system. I have not seen
> your autotols files but there will be a need for a lot of changes because of
> the fact that parsers are now loaded from *.so files in
> ~/.config/ewns-viewer/parsers (and are compiled separately of course).

That shouldn't be an issue, really. Traditionally, the build system should not
touch $HOME, but only the configured installation prefix. Your method of
installation that installs half the things into $HOME, and the rest into a
global directory gets a little troublesome, because then other users on the same
system will not be able to access the files you installed into $HOME.

> Do you want to work more on ewns-viewer?
> I plan to submit this app to ubuntu software center for a price.

I know. I'm the one packaging it up on MyApps. :-)

> I am not very willing to share the money unless you do something really big.
> Hopefully you understand.

That's fine, I'm doing this on a whim.

> If you are still interested in helping me here is a short summary of things needed to get done :
> Localization support,
> Parsers test (many of them crash),
> Better UI for the ewns view (something like transmission's torrent view).
> That should be it.Have a nice day.

I can help regarding localization support, but that's about it.

--
Kind regards,
Loong Jin

[Aleksandar Jovanov (ajovanov93)]

Actually only ewns-viewer.desktop is installed locally.
Everything else is copied during runtime.
Also I meant I am grateful not gracious (I wrote this late last night).
If you are interested you can work on localization then.
I will change the build system a bit more,so I will merge your commits later.

________________________________
 From: Chow Loong Jin <email address hidden>
To: <email address hidden>
Sent: Monday, July 9, 2012 11:28 PM
Subject: Re: [Merge] lp:~hyperair/ewns-viewer/autotools into lp:ewns-viewer

On 10/07/2012 04:16, Aleksandar Jovanov wrote:
> I changed the build system completely today so you will need to change a lot
> of things. I am gracious that you made this build system. I have not seen
> your autotols files but there will be a need for a lot of changes because of
> the fact that parsers are now loaded from *.so files in
> ~/.config/ewns-viewer/parsers (and are compiled separately of course).

That shouldn't be an issue, really. Traditionally, the build system should not
touch $HOME, but only the configured installation prefix. Your method of
installation that installs half the things into $HOME, and the rest into a
global directory gets a little troublesome, because then other users on the same
system will not be able to access the files you installed into $HOME.

> Do you want to work more on ewns-viewer?
> I plan to submit this app to ubuntu software center for a price.

I know. I'm the one packaging it up on MyApps. :-)

> I am not very willing to share the money unless you do something really big.
> Hopefully you understand.

That's fine, I'm doing this on a whim.

> If you are still interested in helping me here is a short summary of things needed to get done :
> Localization support,
> Parsers test (many of them crash),
> Better UI for the ewns view (something like transmission's torrent view).
> That should be it.Have a nice day.

I can help regarding localization support, but that's about it.

--
Kind regards,
Loong Jin

https://code.launchpad.net/~hyperair/ewns-viewer/autotools/+merge/113899
You are requested to review the proposed merge of lp:~hyperair/ewns-viewer/autotools into lp:ewns-viewer.

[lp:~hyperair/ewns-viewer/autotools]

90. By Chow Loong Jin on 2012-07-10

Merge from lp:ewns-viewer

91. By Chow Loong Jin on 2012-07-10

Update autotools build system

[Aleksandar Jovanov (ajovanov93)]

Ok.I looked at the files.
Update everything to revision 100.
Then send me a message and I will accept this.
Thanks for helping.

Unmerged revisions

91. By Chow Loong Jin on 2012-07-10

Update autotools build system

90. By Chow Loong Jin on 2012-07-10

Merge from lp:ewns-viewer

89. By Chow Loong Jin on 2012-07-09

Autotoolize

88. By Chow Loong Jin on 2012-07-09

Drop executable permissions on desktop file

87. By Chow Loong Jin on 2012-07-09

Fix desktop-file-validate complaints

- Drop hashbang -- desktop files aren't supposed to be scripts.
- Add trailing ; to Categories

=== removed file 'Makefile'
--- Makefile	2012-07-10 09:28:15 +0000
+++ Makefile	1970-01-01 00:00:00 +0000
@@ -1,96 +0,0 @@
-#  Ewns-viewer is a tool for programers helping them fix compile time errors quicker.
-#  Copyright (C) 2012 Aleksandar Jovanov
-
-#  This program is free software: you can redistribute it and/or modify
-#  it under the terms of the GNU General Public License as published by
-#  the Free Software Foundation, either version 3 of the License, or
-#  (at your option) any later version.
-
-#  This program is distributed in the hope that it will be useful,
-#  but WITHOUT ANY WARRANTY; without even the implied warranty of
-#  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-#  GNU General Public License for more details.
-
-#  You should have received a copy of the GNU General Public License
-#  along with this program. If not, see <http://www.gnu.org/licenses/>.
-
-# REMEMBER to add ewns_view.vala after this is released
-
-VALAC = valac-0.18
-PKGS = --pkg gee-1.0 --pkg posix --pkg gtk+-3.0 --pkg gio-2.0 --pkg unity --pkg libnotify --pkg gmodule-2.0
-
-SOURCES = ewns_viewer.vala utils_and_settings.vala \
-widgets/debuging_extension.vala widgets/ewns_viewer_widget.vala \
-widgets/session_widget.vala session_info.vala settings_window.vala \
-application.vala session_config.vala parsers/java_script_parser.vala \
-splash_screen.vala parser_loader.vala
-
-EXECUTABLE=ewns-viewer
-
-BUILD_PARSERS = false
-
-all:
-	# vaskell
-	valac --pkg gee-1.0 --library=vaskell --pkg gee-1.0 -H vaskell.h vaskell.vala -X -fPIC -X -shared -o libvaskell.so --gir vaskell-0.1.gir
-	g-ir-compiler --shared-library=libvaskell.so --output=vaskell-0.1.typelib vaskell-0.1.gir
-	cp vaskell.vapi vaskell.h parsers && clear
-
-	# lib ewns viewer
-
-	valac --pkg gee-1.0 --pkg gtk+-3.0 --library=lib_ewns_viewer --pkg gee-1.0 lib_ewns_viewer.vala -H lib_ewns_viewer.h -X -fPIC -X -shared -o libewnsviewer.so
-	cp lib_ewns_viewer.vapi lib_ewns_viewer.h parsers && clear
-
-	# ewns viewer executable
-
-	$(VALAC) $(PKGS) vaskell.vapi lib_ewns_viewer.vapi --thread $(SOURCES) -X libvaskell.so -X libewnsviewer.so -X -I. -o $(EXECUTABLE) && clear
-
-	# parsers
-
-	cd parsers && make
-debug:
-	$(VALAC) -g $(PKGS) vaskell.vapi lib_ewns_viewer.vapi --thread $(SOURCES) -X libvaskell.so -X libewnsviewer.so -X -I. -o $(EXECUTABLE) && clear
-install:
-	# main
-	sudo mkdir -p /opt/ewns-viewer
-	sudo cp apps.ewns-viewer.gschema.xml /opt/ewns-viewer
-	sudo glib-compile-schemas /opt/ewns-viewer
-	
-	sudo cp ewns-viewer /opt/ewns-viewer
-	sudo cp ewns-viewer.desktop /opt/ewns-viewer
-	cp ewns-viewer.desktop ~/.local/share/applications
-	
-	sudo cp ewns-viewer-logo.png /opt/ewns-viewer
-	sudo cp run /opt/ewns-viewer
-
-	# docs
-	sudo mkdir -p /opt/ewns-viewer/doc
-	sudo cp README /opt/ewns-viewer/doc
-	sudo cp LICENSE /opt/ewns-viewer/doc
-	sudo cp help.html /opt/ewns-viewer/doc
-
-	# config
-	sudo mkdir -p /opt/ewns-viewer/config
-	sudo cp editors.cfg /opt/ewns-viewer/config
-	sudo cp debuggers.cfg /opt/ewns-viewer/config
-	sudo cp sessions.txt /opt/ewns-viewer/config
-	sudo cp build_completions.txt /opt/ewns-viewer/config
-	sudo cp search_providers.txt /opt/ewns-viewer/config
-	sudo cp parsers.cfg /opt/ewns-viewer/config
-
-	sudo cp theme.css /opt/ewns-viewer
-
-	# libraries
-	sudo cp libvaskell.so /opt/ewns-viewer
-	sudo cp libewnsviewer.so /opt/ewns-viewer
-
-	# parsers
-	cd parsers && make install
-remove:
-	sudo rm -f -R /opt/ewns-viewer
-	rm -f -R ~/.config/ewns-viewer
-	rm -f ~/.local/share/applications/ewns-viewer.desktop
-uninstall: remove
-deinstall: remove
-clean:
-	rm -f $(EXECUTABLE) *.c *.h *.gir *.typelib *.so *.vapi
-	cd parsers && make clean
\ No newline at end of file

=== added file 'Makefile.am'
--- Makefile.am	1970-01-01 00:00:00 +0000
+++ Makefile.am	2012-07-10 12:24:30 +0000
@@ -0,0 +1,112 @@
+ACLOCAL_AMFLAGS = -Im4
+
+# Common includes and flags
+AM_CFLAGS = $(EWNS_CFLAGS) -I$(top_srcdir)
+COMMONLIBS = $(EWNS_LIBS)
+
+AM_VALAFLAGS =					\
+	--pkg gtk+-3.0				\
+	--pkg gee-1.0				\
+	--pkg posix				\
+	--pkg gtk+-3.0				\
+	--pkg gio-2.0				\
+	--pkg unity				\
+	--pkg libnotify				\
+	--pkg gmodule-2.0			\
+	--vapidir=$(abs_srcdir)			\
+	--pkg config
+AM_LDFLAGS = -avoid-version
+
+# Define libraries
+pkglib_LTLIBRARIES = libvaskell.la libewnsviewer.la
+
+# libvaskell.so
+libvaskell_la_SOURCES = vaskell.vala
+libvaskell_la_VALAFLAGS =			\
+	$(AM_VALAFLAGS)				\
+	--library vaskell			\
+	-H vaskell.h
+libvaskell_la_LIBADD = $(COMMONLIBS)
+vaskell.vapi: vaskell.h
+
+# libewnsviewer.so
+libewnsviewer_la_SOURCES = lib_ewns_viewer.vala
+libewnsviewer_la_VALAFLAGS =			\
+	$(AM_VALAFLAGS)				\
+	--library lib_ewns_viewer		\
+	-H lib_ewns_viewer.h
+libewnsviewer_la_LIBADD = $(COMMONLIBS)
+lib_ewns_viewer.vapi: lib_ewns_viewer.h
+
+# ewns-viewer executable
+bin_PROGRAMS = ewns-viewer
+ewns_viewer_SOURCES = ewns_viewer.vala utils_and_settings.vala \
+widgets/debuging_extension.vala widgets/ewns_viewer_widget.vala \
+widgets/session_widget.vala session_info.vala settings_window.vala \
+application.vala session_config.vala splash_screen.vala parser_loader.vala \
+lib_ewns_viewer.vapi vaskell.vapi
+ewns_viewer_LDADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+
+# Parsers
+parsersdir = $(pkglibdir)/parsers
+
+parsers_LTLIBRARIES =				\
+	libvala_parser.la			\
+	libcpp_parser.la			\
+	libcsharp_parser.la			\
+	libd_parser.la				\
+	libgo_parser.la				\
+	libhaskell_parser.la			\
+	libjava_parser.la			\
+	libjava_script_parser.la		\
+	libgenie_parser.la
+
+libvala_parser_la_SOURCES = parsers/vala_parser.vala vaskell.vapi lib_ewns_viewer.vapi
+libcpp_parser_la_SOURCES = parsers/cpp_parser.vala vaskell.vapi lib_ewns_viewer.vapi
+libcsharp_parser_la_SOURCES = parsers/csharp_parser.vala  vaskell.vapi lib_ewns_viewer.vapi
+libd_parser_la_SOURCES = parsers/d_parser.vala vaskell.vapi lib_ewns_viewer.vapi
+libgo_parser_la_SOURCES = parsers/go_parser.vala vaskell.vapi lib_ewns_viewer.vapi
+libhaskell_parser_la_SOURCES = parsers/haskell_parser.vala vaskell.vapi lib_ewns_viewer.vapi
+libjava_parser_la_SOURCES = parsers/java_parser.vala vaskell.vapi lib_ewns_viewer.vapi
+libjava_script_parser_la_SOURCES = parsers/java_script_parser.vala vaskell.vapi lib_ewns_viewer.vapi
+libgenie_parser_la_SOURCES = parsers/genie_parser.vala vaskell.vapi lib_ewns_viewer.vapi
+
+libvala_parser_la_LIBADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+libcpp_parser_la_LIBADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+libcsharp_parser_la_LIBADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+libd_parser_la_LIBADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+libgo_parser_la_LIBADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+libhaskell_parser_la_LIBADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+libjava_parser_la_LIBADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+libjava_script_parser_la_LIBADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+libgenie_parser_la_LIBADD = $(COMMONLIBS) libvaskell.la libewnsviewer.la
+
+# Install config files
+configdir = $(pkgdatadir)/config
+dist_config_DATA =				\
+	editors.cfg				\
+	debuggers.cfg				\
+	sessions.txt				\
+	build_completions.txt			\
+	search_providers.txt
+
+# Install desktop file
+desktopdir = $(datadir)/applications
+dist_desktop_DATA = ewns-viewer.desktop
+
+# Install doc files
+dist_pkgdoc_DATA =				\
+	README					\
+	LICENSE					\
+	help.html
+
+# Install gsettings schemas
+@GSETTINGS_RULES@
+gsettings_SCHEMAS = apps.ewns-viewer.gschema.xml
+
+# Install icon
+pixmapsdir = $(datadir)/pixmaps
+dist_pixmaps_DATA = ewns-viewer-logo.png
+
+# Distribute config.vapi
+EXTRA_DIST = config.vapi

=== added file 'autogen.sh'
--- autogen.sh	1970-01-01 00:00:00 +0000
+++ autogen.sh	2012-07-10 12:24:30 +0000
@@ -0,0 +1,12 @@
+#!/bin/sh
+
+set -e
+
+builddir="$(pwd)"
+srcdir="$(dirname "$0")"
+
+cd "$srcdir"
+autoreconf -vfi
+
+cd "$builddir"
+[ -z "$NOCONFIGURE" ] && exec "$srcdir"/configure "$@"

=== added file 'config.vapi'
--- config.vapi	1970-01-01 00:00:00 +0000
+++ config.vapi	2012-07-10 12:24:30 +0000
@@ -0,0 +1,33 @@
+[CCode (cprefix = "", lower_case_cprefix = "", cheader_filename = "config.h")]
+namespace Config {
+    /* Location of default config files */
+    public const string CONFIGDIR;
+
+    /* Name of package */
+    public const string PACKAGE;
+
+    /* Define to the address where bug reports for this package should be
+     * sent. */
+    public const string PACKAGE_BUGREPORT;
+
+    /* Define to the full name of this package. */
+    public const string PACKAGE_NAME;
+
+    /* Define to the full name and version of this package. */
+    public const string PACKAGE_STRING;
+
+    /* Define to the one symbol short name of this package. */
+    public const string PACKAGE_TARNAME;
+
+    /* Define to the home page for this package. */
+    public const string PACKAGE_URL;
+
+    /* Define to the version of this package. */
+    public const string PACKAGE_VERSION;
+
+    /* Location of documentation files */
+    public const string PKGDOCDIR;
+
+    /* Version number of package */
+    public const string VERSION;
+}

=== added file 'configure.ac'
--- configure.ac	1970-01-01 00:00:00 +0000
+++ configure.ac	2012-07-10 12:24:30 +0000
@@ -0,0 +1,32 @@
+#                                               -*- Autoconf -*-
+# Process this file with autoconf to produce a configure script.
+
+AC_PREREQ([2.68])
+AC_INIT([ewns-viewer], [1.0], [https://bugs.launchpad.net/ewns-viewer])
+AC_CONFIG_MACRO_DIR([m4])
+AM_INIT_AUTOMAKE([foreign -Wall])
+LT_INIT
+
+m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])])
+
+AC_CONFIG_HEADER([config.h])
+
+AM_PROG_VALAC([0.16])
+AC_PROG_CC
+AM_PROG_CC_C_O
+PKG_CHECK_MODULES(EWNS, [gee-1.0 gtk+-3.0 gio-2.0 unity libnotify gmodule-2.0])
+
+configdir='${datadir}/${PACKAGE}/config'
+EXPAND_VARIABLE([configdir])
+AC_SUBST([configdir])
+AC_DEFINE_UNQUOTED([CONFIGDIR], ["$expanded_configdir"], [Location of default config files])
+
+pkgdocdir='${docdir}/${PACKAGE}'
+EXPAND_VARIABLE([pkgdocdir])
+AC_SUBST([pkgdocdir])
+AC_DEFINE_UNQUOTED([PKGDOCDIR], ["$expanded_pkgdocdir"], [Location of documentation files])
+
+GLIB_GSETTINGS
+
+AC_CONFIG_FILES([Makefile])
+AC_OUTPUT

=== modified file 'ewns-viewer.desktop' (properties changed: +x to -x)
--- ewns-viewer.desktop	2012-07-06 14:35:14 +0000
+++ ewns-viewer.desktop	2012-07-10 12:24:30 +0000
@@ -1,5 +1,3 @@
-#!/usr/bin/env xdg-open
-
 #  Ewns-viewer is a tool for programers helping them fix compile time errors quicker.
 #  Copyright (C) 2012 Aleksandar Jovanov
 
@@ -25,4 +23,4 @@
 StartupNotify=true
 Comment=Tool used to help programmers with building their apps.
 Name=Errors, warnings and notes viewer
-Categories=Development;Building;Utility
\ No newline at end of file
+Categories=Development;Building;Utility;

=== removed file 'install.sh'
--- install.sh	2012-07-06 12:49:21 +0000
+++ install.sh	1970-01-01 00:00:00 +0000
@@ -1,29 +0,0 @@
-# main
-sudo mkdir -p /opt/ewns-viewer
-sudo cp apps.ewns-viewer.gschema.xml /opt/ewns-viewer
-sudo glib-compile-schemas /opt/ewns-viewer
-
-sudo cp ewns-viewer /opt/ewns-viewer
-sudo cp ewns-viewer.desktop /opt/ewns-viewer
-cp ewns-viewer.desktop ~/.local/share/applications
-
-mkdir -p ~/.local/share/glib-2.0/schemas
-cp apps.ewns-viewer.gschema.xml ~/.local/share/glib-2.0/schemas
-glib-compile-schemas ~/.local/share/glib-2.0/schemas
-
-sudo cp ewns-viewer-logo.png /opt/ewns-viewer
-sudo cp run /opt/ewns-viewer
-
-# docs
-sudo mkdir -p /opt/ewns-viewer/doc
-sudo cp README /opt/ewns-viewer/doc
-sudo cp LICENSE /opt/ewns-viewer/doc
-sudo cp help.html /opt/ewns-viewer/doc
-
-# config
-sudo mkdir -p /opt/ewns-viewer/config
-sudo cp editors.cfg /opt/ewns-viewer/config
-sudo cp debuggers.cfg /opt/ewns-viewer/config
-sudo cp sessions.txt /opt/ewns-viewer/config
-sudo cp build_completions.txt /opt/ewns-viewer/config
-sudo cp search_providers.txt /opt/ewns-viewer/config
\ No newline at end of file

=== added directory 'm4'
=== added file 'm4/expansions.m4'
--- m4/expansions.m4	1970-01-01 00:00:00 +0000
+++ m4/expansions.m4	2012-07-10 12:24:30 +0000
@@ -0,0 +1,17 @@
+AC_DEFUN([EXPAND_VARIABLE],
+[
+    if test "$prefix" = NONE; then
+        prefix="$ac_default_prefix"
+    fi
+    if test "$exec_prefix" = NONE; then
+        exec_prefix="$prefix"
+    fi
+
+    expanded_$1="${$1}"
+    while test "$prev_expanded_$1" != "$expanded_$1"; do
+        prev_expanded_$1="$expanded_$1"
+        expanded_$1=`(eval echo $expanded_$1)`
+    done
+
+    AC_SUBST(expanded_$1)
+])

=== removed file 'parsers/Makefile'
--- parsers/Makefile	2012-07-10 08:52:00 +0000
+++ parsers/Makefile	1970-01-01 00:00:00 +0000
@@ -1,15 +0,0 @@
-PKG_CONFIG := `pkg-config --cflags --libs gee-1.0 glib-2.0 gmodule-2.0 gtk+-3.0`
-FILES = vala_parser cpp_parser csharp_parser d_parser go_parser haskell_parser java_parser java_script_parser genie_parser
-
-all:	
-	for file in $(FILES) ; do \
-		valac vaskell.vapi --pkg gtk+-3.0 --pkg gee-1.0 lib_ewns_viewer.vapi --pkg gmodule-2.0 -C $$file.vala; \
-		gcc -shared -fPIC $(PKG_CONFIG) -o lib$$file.so $$file.c; \
-	done
-install:
-	sudo mkdir -p /opt/ewns-viewer/parsers
-	for file in $(FILES) ; do \
-		sudo cp lib$$file.so /opt/ewns-viewer/parsers; \
-	done
-clean:
-	rm -f *.c *.h *.vapi *.so
\ No newline at end of file

=== modified file 'session_config.vala'
--- session_config.vala	2012-07-09 19:30:12 +0000
+++ session_config.vala	2012-07-10 12:24:30 +0000
@@ -25,7 +25,7 @@
 string load_config_file(string name)
 {
 	var local_path = Environment.get_variable("HOME") + "/.config/ewns-viewer/" + name;
-	var global_path = "/opt/ewns-viewer/config/" + name;
+	var global_path = Path.build_filename(Config.CONFIGDIR, name);
 	bool copy = true;
 
 	if(FileUtils.test(local_path,FileTest.EXISTS))

=== modified file 'utils_and_settings.vala'
--- utils_and_settings.vala	2012-07-10 09:28:15 +0000
+++ utils_and_settings.vala	2012-07-10 12:24:30 +0000
@@ -42,7 +42,7 @@
 Unity.LauncherEntry _icon;
 
 const string WEB_SITE = "https://launchpad.net/ewns-viewer";
-const string VERSION_STRING = "1.0";
+const string VERSION_STRING = Config.PACKAGE_VERSION;
 
 string EDITOR_EXECUTABLE;
 
@@ -238,7 +238,7 @@
 	if(SHOW_NOTIFICATIONS == false)
 		return;
 
-    var notification = new Notify.Notification(title,body,"/opt/ewns-viewer/ewns-viewer-logo.png");
+    var notification = new Notify.Notification(title,body,"ewns-viewer-logo");
 
     notification.show();
 }
@@ -276,7 +276,7 @@
     if(video)
         Process.spawn_command_line_async("xdg-open https://vimeo.com/45350691");
     else
-        Process.spawn_command_line_async("xdg-open /opt/ewns-viewer/doc/help.html");
+        Process.spawn_command_line_async("xdg-open " + Path.build_filename(Config.PKGDOCDIR,"help.html"));
 }
 
 string read_line(Posix.FILE? stream)
@@ -374,4 +374,4 @@
 string convert_color_field(double d)
 {
     return ((int)(d * 255)).to_string();
-}
\ No newline at end of file
+}