[PATCH 1/2] Use dist_ prefix as applicable

[PATCH 1/2] Use dist_ prefix as applicable

[=?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont]

[-- Attachment #1: Type: text/plain, Size: 1163 bytes --]

---
 Makefile.am |   11 ++++-------
 1 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index f941a19..71365d7 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -25,7 +25,7 @@ local_headers = $(foreach file,$(pkginclude_HEADERS) \
 if DATAFILES
 dbusconfdir = @DBUS_CONFDIR@
 
-dbusconf_DATA = src/ofono.conf
+dist_dbusconf_DATA = src/ofono.conf
 
 if SYSTEMD
 systemdunitdir = @SYSTEMD_UNITDIR@
@@ -35,7 +35,7 @@ endif
 
 confdir = $(sysconfdir)/ofono
 
-conf_DATA =
+dist_conf_DATA =
 
 statedir = $(localstatedir)/lib/ofono
 
@@ -244,7 +244,7 @@ builtin_modules += phonesim
 builtin_sources += plugins/phonesim.c
 
 if DATAFILES
-conf_DATA += plugins/phonesim.conf
+dist_conf_DATA += plugins/phonesim.conf
 endif
 endif
 
@@ -477,10 +477,7 @@ testdir = $(pkglibdir)/test
 test_SCRIPTS = $(test_scripts)
 endif
 
-conf_files = src/ofono.conf plugins/phonesim.conf
-
-EXTRA_DIST = src/genbuiltin $(conf_files) $(udev_files) \
-					$(doc_files) $(test_scripts)
+EXTRA_DIST = src/genbuiltin $(udev_files) $(doc_files) $(test_scripts)
 
 dist_man_MANS = doc/ofonod.8
 
-- 
1.7.1

[PATCH 2/2] Support for pkg-config

[=?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont]

[-- Attachment #1: Type: text/plain, Size: 1540 bytes --]

---
 .gitignore   |    1 +
 Makefile.am  |    3 +++
 configure.ac |    2 +-
 ofono.pc.in  |   10 ++++++++++
 4 files changed, 15 insertions(+), 1 deletions(-)
 create mode 100644 ofono.pc.in

diff --git a/.gitignore b/.gitignore
index 7cfb1d9..59308be 100644
--- a/.gitignore
+++ b/.gitignore
@@ -20,6 +20,7 @@ install-sh
 libtool
 ltmain.sh
 missing
+ofono.pc
 stamp-h1
 autom4te.cache
 
diff --git a/Makefile.am b/Makefile.am
index 71365d7..3b29d19 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -21,6 +21,9 @@ local_headers = $(foreach file,$(pkginclude_HEADERS) \
 				$(nodist_pkginclude_HEADERS), \
 					include/ofono/$(notdir $(file)))
 
+pkgconfigdir = $(libdir)/pkgconfig
+pkgconfig_DATA = ofono.pc
+
 
 if DATAFILES
 dbusconfdir = @DBUS_CONFDIR@
diff --git a/configure.ac b/configure.ac
index b034ff0..d26b64f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -226,4 +226,4 @@ fi
 AC_DEFINE_UNQUOTED(CONFIGDIR, "${configdir}",
 			[Directory for the configuration files])
 
-AC_OUTPUT(Makefile include/version.h src/ofono.service)
+AC_OUTPUT(Makefile include/version.h src/ofono.service ofono.pc)
diff --git a/ofono.pc.in b/ofono.pc.in
new file mode 100644
index 0000000..19f12e6
--- /dev/null
+++ b/ofono.pc.in
@@ -0,0 +1,10 @@
+prefix=@prefix@
+exec_prefix=@exec_prefix@
+libdir=@libdir@
+plugindir=${libdir}/@PACKAGE@/plugins
+includedir=@includedir@
+
+Name: oFono
+Description: oFono - Open Source Telephony
+Version: @VERSION@
+Cflags: -I${includedir}
-- 
1.7.1

Re: [PATCH 2/2] Support for pkg-config

[Marcel Holtmann]

[-- Attachment #1: Type: text/plain, Size: 2338 bytes --]

Hi Remi,

>  .gitignore   |    1 +
>  Makefile.am  |    3 +++
>  configure.ac |    2 +-
>  ofono.pc.in  |   10 ++++++++++
>  4 files changed, 15 insertions(+), 1 deletions(-)
>  create mode 100644 ofono.pc.in
> 
> diff --git a/.gitignore b/.gitignore
> index 7cfb1d9..59308be 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -20,6 +20,7 @@ install-sh
>  libtool
>  ltmain.sh
>  missing
> +ofono.pc
>  stamp-h1
>  autom4te.cache

please put ofono.pc on top of include/ofono line.
 
> diff --git a/Makefile.am b/Makefile.am
> index 71365d7..3b29d19 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -21,6 +21,9 @@ local_headers = $(foreach file,$(pkginclude_HEADERS) \
>  				$(nodist_pkginclude_HEADERS), \
>  					include/ofono/$(notdir $(file)))
>  
> +pkgconfigdir = $(libdir)/pkgconfig
> +pkgconfig_DATA = ofono.pc
> +
>  
>  if DATAFILES
>  dbusconfdir = @DBUS_CONFDIR@
> diff --git a/configure.ac b/configure.ac
> index b034ff0..d26b64f 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -226,4 +226,4 @@ fi
>  AC_DEFINE_UNQUOTED(CONFIGDIR, "${configdir}",
>  			[Directory for the configuration files])
>  
> -AC_OUTPUT(Makefile include/version.h src/ofono.service)
> +AC_OUTPUT(Makefile include/version.h src/ofono.service ofono.pc)
> diff --git a/ofono.pc.in b/ofono.pc.in
> new file mode 100644
> index 0000000..19f12e6
> --- /dev/null
> +++ b/ofono.pc.in
> @@ -0,0 +1,10 @@
> +prefix=@prefix@
> +exec_prefix=@exec_prefix@
> +libdir=@libdir@
> +plugindir=${libdir}/@PACKAGE@/plugins

Please remove plugindir in a separate paragraph.

> +includedir=@includedir@
> +
> +Name: oFono

This needs to be all lowercase. So just ofono.

> +Description: oFono - Open Source Telephony
> +Version: @VERSION@

You are missing the Libs: and Requires: line here.

> +Cflags: -I${includedir}

Something like this should do it:

prefix=@prefix@
exec_prefix=@exec_prefix@
libdir=@libdir@
includedir=@includedir@
 
plugindir=${libdir}/@PACKAGE@/plugins

Name: @PACKAGE@
Description: oFono - Open Source Telephony
Requires: glib-2.0 dbus-1
Version: @VERSION@
Libs: -module -avoid-version -export-symbols-regex @PACKAGE(a)_plugin_desc
Cflags: -I${includedir}

Otherwise you are a bit out of luck when building external plugins.

Regards

Marcel

Re: [PATCH 2/2] Support for pkg-config

[=?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont]

[-- Attachment #1: Type: text/plain, Size: 632 bytes --]

On Wednesday 26 January 2011 13:27:23 ext Marcel Holtmann, you wrote:
> prefix=@prefix@
> exec_prefix=@exec_prefix@
> libdir=@libdir@
> includedir=@includedir@
> 
> plugindir=${libdir}/@PACKAGE@/plugins
> 
> Name: @PACKAGE@
> Description: oFono - Open Source Telephony
> Requires: glib-2.0 dbus-1
> Version: @VERSION@
> Libs: -module -avoid-version -export-symbols-regex @PACKAGE(a)_plugin_desc

In my experience, automake does not fancy LDFLAGS in LIBADD/LDADD. Call that a 
pkg-config limitation if you want.

> Cflags: -I${includedir}


-- 
Rémi Denis-Courmont
Nokia Devices R&D, Maemo Software, Helsinki

Re: [PATCH 2/2] Support for pkg-config

[Marcel Holtmann]

[-- Attachment #1: Type: text/plain, Size: 647 bytes --]

Hi Remi,

> > prefix=@prefix@
> > exec_prefix=@exec_prefix@
> > libdir=@libdir@
> > includedir=@includedir@
> > 
> > plugindir=${libdir}/@PACKAGE@/plugins
> > 
> > Name: @PACKAGE@
> > Description: oFono - Open Source Telephony
> > Requires: glib-2.0 dbus-1
> > Version: @VERSION@
> > Libs: -module -avoid-version -export-symbols-regex @PACKAGE(a)_plugin_desc
> 
> In my experience, automake does not fancy LDFLAGS in LIBADD/LDADD. Call that a 
> pkg-config limitation if you want.

so pkg-config --libs clearly is for "all linker flags". So that seems
pretty much fine. Alternatives are --libs-only-*.

Regards

Marcel

Re: [PATCH 2/2] Support for pkg-config

[=?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont]

[-- Attachment #1: Type: text/plain, Size: 382 bytes --]

On Wednesday 26 January 2011 13:27:23 ext Marcel Holtmann, you wrote:
> Name: @PACKAGE@
> Description: oFono - Open Source Telephony
> Requires: glib-2.0 dbus-1

Hmm, wasn't the goal to not leak glib through the public API? I could not find 
any glib type from a quick check in include/ofono/ ...



-- 
Rémi Denis-Courmont
Nokia Devices R&D, Maemo Software, Helsinki

Re: [PATCH 2/2] Support for pkg-config

[Marcel Holtmann]

[-- Attachment #1: Type: text/plain, Size: 439 bytes --]

Hi Remi,

> > Name: @PACKAGE@
> > Description: oFono - Open Source Telephony
> > Requires: glib-2.0 dbus-1
> 
> Hmm, wasn't the goal to not leak glib through the public API? I could not find 
> any glib type from a quick check in include/ofono/ ...

let's face it, you will integrate with glib one way or another in your
plugin. That we don't wanna our API to be using glib types is a
different story.

Regards

Marcel

Re: [PATCH 1/2] Use dist_ prefix as applicable

[Marcel Holtmann]

[-- Attachment #1: Type: text/plain, Size: 236 bytes --]

Hi Remi,

>  Makefile.am |   11 ++++-------
>  1 files changed, 4 insertions(+), 7 deletions(-)

patch has been applied, but in the future please prefix the subject with
something like build: or similar.

Regards

Marcel