sconeserver: new package
PATCH 1/1 sconeserver: new package
From: Simon Dawson <[email protected]>
--- package/Config.in | 1 + package/sconeserver/Config.in | 100 ++++++++++++++++++++++++++++++++++++ package/sconeserver/sconeserver.mk | 49 ++++++++++++++++++ 3 files changed, 150 insertions(+) create mode 100644 package/sconeserver/Config.in create mode 100644 package/sconeserver/sconeserver.mk
package/Config.in +1 -0
@@ -591,6 +591,7 @@ source "package/rsh-redone/Config.in" source "package/rsync/Config.in" source "package/rtorrent/Config.in" source "package/samba/Config.in" +source "package/sconeserver/Config.in" source "package/ser2net/Config.in" source "package/socat/Config.in" source "package/socketcand/Config.in"
package/sconeserver/Config.in new file +100 -0
100 lines not on this page · raw message
package/sconeserver/sconeserver.mk new file +49 -0
49 lines not on this page · raw message
Signature · 1 line
1.7.9.5
PATCH 1/1 sconeserver: new package
On 07/23/12 15:49, spdawson at gmail.com wrote:
[snip]
Quoted 10 lines
package/sconeserver/Config.in new file +4
-0@@ -0,0 +1,100 @@ +config BR2_PACKAGE_SCONESERVER + bool "sconeserver" + depends on BR2_INSTALL_LIBSTDCPP + select BR2_PACKAGE_PCRE
package/sconeserver/Config.in line 14
Quoted 10 lines
package/sconeserver/Config.in new file +10
-0+ help + Sconeserver is a modular, object-orientated and extremely versatile + network server framework for GNU/Linux and UNIX-like platforms. + + http://www.sconemad.com/sconeserver/ + +comment "sconeserver requires a toolchain with C++ support enabled" + depends on !BR2_INSTALL_LIBSTDCPP + +menu "Modules"
I think a menuconfig is nicer in this case. Cfr. OpenCV.
package/sconeserver/Config.in line 19
Quoted 5 lines
package/sconeserver/Config.in new file +5
-0+ depends on BR2_PACKAGE_SCONESERVER + +config BR2_PACKAGE_SCONESERVER_EXAMPLES + bool "examples" + default n
n is the default, so this statement is redundant.
package/sconeserver/Config.in line 25
Quoted 6 lines
package/sconeserver/Config.in new file +6
-0+ help + Example modules for SconeServer + +config BR2_PACKAGE_SCONESERVER_SSL + bool "ssl" + default y
Why does this default to y?
Maybe you want
default y if BR2_PACKAGE_OPENSSL
however, I think that conflicts with the select statement below...
Quoted 6 lines
package/sconeserver/Config.in new file +6
-0+ select BR2_PACKAGE_OPENSSL + help + SSL module for SconeServer + +config BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE + bool "http::sconesite"
package/sconeserver/Config.in line 41
Quoted 10 lines
package/sconeserver/Config.in new file +10
-0+ default y + select BR2_PACKAGE_LIBXML2 + help + http::sconesite module for SconeServer + +config BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE_IMAGE + bool "http::sconesite::image" + default n + depends on BR2_PACKAGE_IMAGEMAGICK + select BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE
Shouldn't this be
depends on BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE
select BR2_PACKAGE_IMAGEMAGICK
package/sconeserver/Config.in line 48
Quoted 7 lines
package/sconeserver/Config.in new file +7
-0+ help + http::sconesite::image module for SconeServer + +config BR2_PACKAGE_SCONESERVER_MYSQL + bool "mysql" + default n + depends on BR2_PACKAGE_MYSQL_CLIENT
Why depends on rather than select?
Same for all the following...
Quoted 19 lines
package/sconeserver/Config.in new file +19
-0+ help + MySQL module for SconeServer + +config BR2_PACKAGE_SCONESERVER_BLUETOOTH + bool "bluetooth" + default n + depends on BR2_PACKAGE_BLUEZ_UTILS + help + Bluetooth module for SconeServer + +config BR2_PACKAGE_SCONESERVER_RSS + bool "rss" + default y + select BR2_PACKAGE_LIBXML2 + help + RSS module for SconeServer + +config BR2_PACKAGE_SCONESERVER_LOCATION + bool "location"
package/sconeserver/Config.in line 77
Quoted 10 lines
package/sconeserver/Config.in new file +10
-0+ default n + depends on BR2_PACKAGE_GPSD + help + Location module for SconeServer + +config BR2_PACKAGE_SCONESERVER_LETTUCE + bool "lettuce" + default n + help + Lettuce module for SconeServer
This one could use some explanation of what lettuce is...
package/sconeserver/Config.in line 85
Quoted 8 lines
package/sconeserver/Config.in new file +8
-0+ +config BR2_PACKAGE_SCONESERVER_MATHS + bool "maths" + default n + depends on BR2_PACKAGE_GMP + depends on BR2_PACKAGE_MPFR + help + Maths module for SconeServer
Maybe this one as well.
Quoted 21 lines
package/sconeserver/Config.in new file +15
-0+ +config BR2_PACKAGE_SCONESERVER_TESTBUILDER + bool "testbuilder" + default n + help + Testbuilder module for SconeServer + +config BR2_PACKAGE_SCONESERVER_UI + bool "ui" + default n + depends on BR2_PACKAGE_XLIB_LIBX11 + help + UI module for SconeServer + +endmenupackage/sconeserver/sconeserver.mk new file +0
-0
package/sconeserver/sconeserver.mk line 7
Quoted 7 lines
package/sconeserver/sconeserver.mk new file +7
-0@@ -0,0 +1,49 @@ +############################################################# +# +# sconeserver +# +############################################################# +# Release tarball doesn't build cleanly, so use a recent +# Subversion trunk snapshot.
Mention which is the release that doesn't build cleanly, this will
help version bumpers.
Quoted 18 lines
package/sconeserver/sconeserver.mk new file +18
-0+SCONESERVER_VERSION = 177 +SCONESERVER_SITE = \ + https://sconeserver.svn.sourceforge.net/svnroot/sconeserver/trunk +SCONESERVER_SITE_METHOD = svn + +SCONESERVER_LICENSE = GPLv2+ +SCONESERVER_LICENSE_FILES = COPYING + +SCONESERVER_CONF_OPT += \ + --with-ip \ + --with-local \ + $(if $(BR2_INET_IPV6),--with-ip6,--without-ip6) \ + $(if $(BR2_PACKAGE_SCONESERVER_EXAMPLES),--with-examples,--without-examples) \ + $(if $(BR2_PACKAGE_SCONESERVER_LETTUCE),--with-lettuce,--without-lettuce) \ + $(if $(BR2_PACKAGE_SCONESERVER_TESTBUILDER),--with-testbuilder,--without-testbuilder) \ + $(if $(BR2_PACKAGE_SCONESERVER_SSL),--with-ssl,--without-ssl) \ + $(if $(BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE),--with-sconesite,--without-sconesite) \ + $(if $(BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE_IMAGE),--with-sconesite-image,--without-sconesite-image) \
package/sconeserver/sconeserver.mk line 35
Quoted 10 lines
package/sconeserver/sconeserver.mk new file +10
-0+ $(if $(BR2_PACKAGE_SCONESERVER_MYSQL),--with-mysql,--without-mysql) \ + $(if $(BR2_PACKAGE_SCONESERVER_BLUETOOTH),--with-bluetooth,--without-bluetooth) \ + $(if $(BR2_PACKAGE_SCONESERVER_RSS),--with-rss,--without-rss) \ + $(if $(BR2_PACKAGE_SCONESERVER_LOCATION),--with-location,--without-location) \ + $(if $(BR2_PACKAGE_SCONESERVER_MATHS),--with-maths,--without-maths) \ + $(if $(BR2_PACKAGE_SCONESERVER_UI),--with-ui,--without-ui) + +# Help SconeServer to find the libxml2 headers. +ifeq ($(BR2_PACKAGE_LIBXML2),y) + SCONESERVER_CONF_OPT += CPPFLAGS="-I$(STAGING_DIR)/usr/include/libxml2"
Why doesn't pkg-config find this path? Are they missing a check for
libxml-2.0 maybe?
Quoted 4 lines
package/sconeserver/sconeserver.mk new file +4
-0+endif + +SCONESERVER_DEPENDENCIES = \ + pcre \
package/sconeserver/sconeserver.mk line 49
Quoted 10 lines
package/sconeserver/sconeserver.mk new file +10
-0+ $(if $(BR2_PACKAGE_SCONESERVER_SSL),openssl) \ + $(if $(or $(BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE),$(BR2_PACKAGE_SCONESERVER_RSS)),libxml2) \ + $(if $(BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE_IMAGE),imagemagick) \ + $(if $(BR2_PACKAGE_SCONESERVER_MYSQL),mysql_client) \ + $(if $(BR2_PACKAGE_SCONESERVER_BLUETOOTH),bluez_utils) \ + $(if $(BR2_PACKAGE_SCONESERVER_LOCATION),gpsd) \ + $(if $(BR2_PACKAGE_SCONESERVER_MATHS),gmp mpfr) \ + $(if $(BR2_PACKAGE_SCONESERVER_UI),xlib_libX11) + +$(eval $(autotools-package))
Regards,
Arnout
Signature · 6 lines
Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286540 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
PATCH 1/1 sconeserver: new package
Hi Simon, Arnout,
2012/7/26 Arnout Vandecappelle <[email protected]>:
Quoted 49 lines · spdawson at gmail.com
[snip]
Quoted 13 lines
package/sconeserver/Config.in new file +6
-0@@ -0,0 +1,100 @@ +config BR2_PACKAGE_SCONESERVER + bool "sconeserver" + depends on BR2_INSTALL_LIBSTDCPP + select BR2_PACKAGE_PCRE + help + Sconeserver is a modular, object-orientated and extremelyversatile
package/sconeserver/Config.in line 14
Quoted 8 lines
package/sconeserver/Config.in new file +8
-0+ network server framework for GNU/Linux and UNIX-like platforms. + + http://www.sconemad.com/sconeserver/ + +comment "sconeserver requires a toolchain with C++ support enabled" + depends on !BR2_INSTALL_LIBSTDCPP + +menu "Modules"I think a menuconfig is nicer in this case. Cfr. OpenCV.
package/sconeserver/Config.in line 19
Quoted 5 lines
package/sconeserver/Config.in new file +5
-0+ depends on BR2_PACKAGE_SCONESERVER + +config BR2_PACKAGE_SCONESERVER_EXAMPLES + bool "examples" + default nn is the default, so this statement is redundant.
package/sconeserver/Config.in line 25
Quoted 6 lines
package/sconeserver/Config.in new file +6
-0+ help + Example modules for SconeServer + +config BR2_PACKAGE_SCONESERVER_SSL + bool "ssl" + default yWhy does this default to y?
Quoted 7 lines
Maybe you want
default y ifBR2_PACKAGE_OPENSSLhowever, I think that conflicts with the select statement below...
Quoted 1 line
+ select
BR2_PACKAGE_OPENSSL
OTHO, you might wont to select which ssl implementation from openssl or polarssl you prefer.
Though sconeserver website only mentions openssl dependency, have you try using polarssl?
Quoted 7 lines
Quoted 7 lines
package/sconeserver/Config.in new file +7
-0+ help + SSL module for SconeServer + +config BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE + bool "http::sconesite" + default y + select BR2_PACKAGE_LIBXML2
Quoted 8 lines
package/sconeserver/Config.in line 41
Quoted 8 lines
package/sconeserver/Config.in new file +8
-0+ help + http::sconesite module for SconeServer + +config BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE_IMAGE + bool "http::sconesite::image" + default n + depends on BR2_PACKAGE_IMAGEMAGICK + select BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE
We usually prefer using "select" for package or library dependency, reserving "depends on" for more bigger dependency like X or something deep in the toolchain like RPC or WCHAR support.
See: http://buildroot.org/downloads/manual/manual.html#_literal_config_in_literal_file
Quoted 150 lines
Shouldn't this be
depends onBR2_PACKAGE_SCONESERVER_HTTP_SCONESITE
selectBR2_PACKAGE_IMAGEMAGICKpackage/sconeserver/Config.in line 48
Quoted 7 lines
package/sconeserver/Config.in new file +7
-0+ help + http::sconesite::image module for SconeServer + +config BR2_PACKAGE_SCONESERVER_MYSQL + bool "mysql" + default n + depends on BR2_PACKAGE_MYSQL_CLIENTWhy depends on rather than select?
Same for all the following...
Quoted 19 lines
package/sconeserver/Config.in new file +19
-0+ help + MySQL module for SconeServer + +config BR2_PACKAGE_SCONESERVER_BLUETOOTH + bool "bluetooth" + default n + depends on BR2_PACKAGE_BLUEZ_UTILS + help + Bluetooth module for SconeServer + +config BR2_PACKAGE_SCONESERVER_RSS + bool "rss" + default y + select BR2_PACKAGE_LIBXML2 + help + RSS module for SconeServer + +config BR2_PACKAGE_SCONESERVER_LOCATION + bool "location"package/sconeserver/Config.in line 77
Quoted 10 lines
package/sconeserver/Config.in new file +10
-0+ default n + depends on BR2_PACKAGE_GPSD + help + Location module for SconeServer + +config BR2_PACKAGE_SCONESERVER_LETTUCE + bool "lettuce" + default n + help + Lettuce module for SconeServerThis one could use some explanation of what lettuce is...
package/sconeserver/Config.in line 85
Quoted 8 lines
package/sconeserver/Config.in new file +8
-0+ +config BR2_PACKAGE_SCONESERVER_MATHS + bool "maths" + default n + depends on BR2_PACKAGE_GMP + depends on BR2_PACKAGE_MPFR + help + Maths module for SconeServerMaybe this one as well.
Quoted 22 lines
package/sconeserver/Config.in new file +15
-0+ +config BR2_PACKAGE_SCONESERVER_TESTBUILDER + bool "testbuilder" + default n + help + Testbuilder module for SconeServer + +config BR2_PACKAGE_SCONESERVER_UI + bool "ui" + default n + depends on BR2_PACKAGE_XLIB_LIBX11 + help + UI module for SconeServer + +endmenupackage/sconeserver/sconeserver.mk new file +0
-0package/sconeserver/sconeserver.mk line 7
Quoted 7 lines
package/sconeserver/sconeserver.mk new file +7
-0@@ -0,0 +1,49 @@ +############################################################# +# +# sconeserver +# +############################################################# +# Release tarball doesn't build cleanly, so use a recent +# Subversion trunk snapshot.Mention which is the release that doesn't build cleanly, this will
help version bumpers.Quoted 40 lines
package/sconeserver/sconeserver.mk new file +13
-0+SCONESERVER_VERSION = 177 +SCONESERVER_SITE = \ + https://sconeserver.svn.sourceforge.net/svnroot/sconeserver/trunk +SCONESERVER_SITE_METHOD = svn + +SCONESERVER_LICENSE = GPLv2+ +SCONESERVER_LICENSE_FILES = COPYING + +SCONESERVER_CONF_OPT += \ + --with-ip \ + --with-local \ + $(if $(BR2_INET_IPV6),--with-ip6,--without-ip6) \ + $(if$(BR2_PACKAGE_SCONESERVER_EXAMPLES),--with-examples,--without-examples) \
+ $(if
$(BR2_PACKAGE_SCONESERVER_LETTUCE),--with-lettuce,--without-lettuce) \
+ $(if
$(BR2_PACKAGE_SCONESERVER_TESTBUILDER),--with-testbuilder,--without-testbuilder) \package/sconeserver/sconeserver.mk new file +2
-0+ $(if $(BR2_PACKAGE_SCONESERVER_SSL),--with-ssl,--without-ssl) \ + $(if$(BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE),--with-sconesite,--without-sconesite)
\
+ $(if
$(BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE_IMAGE),--with-sconesite-image,--without-sconesite-image) \
+ $(if
$(BR2_PACKAGE_SCONESERVER_MYSQL),--with-mysql,--without-mysql) \
+ $(if
$(BR2_PACKAGE_SCONESERVER_BLUETOOTH),--with-bluetooth,--without-bluetooth) \package/sconeserver/sconeserver.mk new file +2
-0+ $(if $(BR2_PACKAGE_SCONESERVER_RSS),--with-rss,--without-rss) \ + $(if$(BR2_PACKAGE_SCONESERVER_LOCATION),--with-location,--without-location) \
+ $(if
$(BR2_PACKAGE_SCONESERVER_MATHS),--with-maths,--without-maths) \package/sconeserver/sconeserver.mk new file +5
-0+ $(if $(BR2_PACKAGE_SCONESERVER_UI),--with-ui,--without-ui) + +# Help SconeServer to find the libxml2 headers. +ifeq ($(BR2_PACKAGE_LIBXML2),y) + SCONESERVER_CONF_OPT +=Quoted 1 line
CPPFLAGS="-I$(STAGING_DIR)/usr/include/libxml2"
Why doesn't pkg-config find this path? Are they missing a check for
libxml-2.0 maybe?
Quoted 5 lines
package/sconeserver/sconeserver.mk line 40
Quoted 5 lines
package/sconeserver/sconeserver.mk new file +5
-0+endif + +SCONESERVER_DEPENDENCIES = \ + pcre \ + $(if $(BR2_PACKAGE_SCONESERVER_SSL),openssl) \
Since the ssl option (like most of of the following ones) has effect on both SCONESERVER_DEPENDENCIES and SCONESERVER_CONF_OPT, I would prefer to see them nested in the same if[n]eq statement:
ifneq ($(BR2_PACKAGE_SCONESERVER_SSL),)SCONESERVER_CONF_OPT += --with-sslSCONESERVER_DEPENDENCIES += openssl
elseSCONESERVER_CONF_OPT += --without-ssl
endif
IMO, this makes things easier to maintain.
Quoted 4 lines
Quoted 4 lines
+ $(if $(or
$(BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE),$(BR2_PACKAGE_SCONESERVER_RSS)),libxml2) \
+ $(if $(BR2_PACKAGE_SCONESERVER_HTTP_SCONESITE_IMAGE),imagemagick)
Quoted 9 lines
package/sconeserver/sconeserver.mk line 49
Quoted 8 lines
\
package/sconeserver/sconeserver.mk new file +7
-0+ $(if $(BR2_PACKAGE_SCONESERVER_MYSQL),mysql_client) \ + $(if $(BR2_PACKAGE_SCONESERVER_BLUETOOTH),bluez_utils) \ + $(if $(BR2_PACKAGE_SCONESERVER_LOCATION),gpsd) \ + $(if $(BR2_PACKAGE_SCONESERVER_MATHS),gmp mpfr) \ + $(if $(BR2_PACKAGE_SCONESERVER_UI),xlib_libX11) + +$(eval $(autotools-package))
Regards,
Signature · 1 line
Sam
PATCH 1/1 sconeserver: new package
Le Fri, 27 Jul 2012 01:14:43 +0200,
Samuel Martin <[email protected]> a ?crit :
Quoted 4 lines
Since the ssl option (like most of of the following ones) has effect on both
SCONESERVER_DEPENDENCIESandSCONESERVER_CONF_OPT, I would prefer to see them nested in the same if[n]eq statement:
Quoted 8 lines
ifneq ($(BR2_PACKAGE_SCONESERVER_SSL),)
SCONESERVER_CONF_OPT+= --with-sslSCONESERVER_DEPENDENCIES+= openssl
elseSCONESERVER_CONF_OPT+= --without-ssl
endifIMO, this makes things easier to maintain.
I agree. It's a bit longer from a number of lines perspective, but it's much more logical, IMO.
Thomas
Signature · 4 lines
Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com
PATCH 1/1 sconeserver: new package
In reply to Arnout VandecappelleLe Thu, 26 Jul 2012 18:57:45 +0200,
Arnout Vandecappelle <[email protected]> a ?crit :
Quoted 9 lines
package/sconeserver/Config.in line 25
Quoted 6 lines
package/sconeserver/Config.in new file +6
-0+ help + Example modules for SconeServer + +config BR2_PACKAGE_SCONESERVER_SSL + bool "ssl" + default yWhy does this default to y?
Quoted 4 lines
Maybe you want
default y ifBR2_PACKAGE_OPENSSLhowever, I think that conflicts with the select statement below...
Also in general we don't have sub-options for things that depend on
packages: the package automatically enables the feature if the required dependency is available.
I have the same problem with the VLC package currently being proposed by Ismael: it has a huge number of sub-options, for each potential dependency that the package might use.
On one hand having all these sub-options is a bit annoying, but:
(*) It allows to have sub-options for things that depend on other
libraries, but also for things that don't have dependencies. Otherwise, it's a bit strange: for --enable/--disable flags that don't require dependencies, we have a kconfig option, but for --enable/--disable flags that require dependencies, they get automatically enabled when the required dependency is available.
(*) You may want to have SSL support in one component of your system,
but not necessarily others.
Thoughts?
Thomas
Signature · 4 lines
Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com
PATCH 1/1 sconeserver: new package
Hi Arnout, Samuel, Thomas.
Thanks for the feedback. I'll submit a reworked patch.
Simon.
PATCH 1/1 sconeserver: new package
In reply to Thomas PetazzoniQuoted 13 lines · Thomas Petazzoni
I have the same problem with the VLC package currently being proposed by Ismael: it has a huge number of sub-options, for each potential dependency that the package might use.
On one hand having all these sub-options is a bit annoying, but:
(*) It allows to have sub-options for things that depend on other
libraries, but also for things that don't have dependencies. Otherwise, it's a bit strange: for --enable/--disable flags that don't require dependencies, we have a kconfig option, but for --enable/--disable flags that require dependencies, they get automatically enabled when the required dependency is available.
Quoted 2 lines
(*) You may want to have SSL support in one component of your system,
but not necessarily others.
I think it depends a bit on the situation. For openssl support, it
makes more sense to just add it when available, because it normally doesn't increase the size of the binary (much) to have openssl support enabled.
However, sconeserver (and vlc as well) are modular packages, which you
can basically view as a collection of sub-packages. So in that case, it does make more sense to select which modules you want to have enabled.
So for sconeserver, I prefer to keep all the sub-options. Except
maybe the SSL module, that's not really a separate server but a library wrapper, so it should probably be selected automatically if openssl is.
Regards,
Arnout
Signature · 6 lines
Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286540 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
PATCH 1/1 sconeserver: new package
Quoted 10 lines · Arnout Vandecappelle
I think it depends a bit on the situation. For openssl support, it
makes more sense to just add it when available, because it normally doesn't increase the size of the binary (much) to have openssl support enabled.However, sconeserver (and vlc as well) are modular packages, which you
can basically view as a collection of sub-packages. So in that case, it does make more sense to select which modules you want to have enabled.
Quoted 4 lines
So for sconeserver, I prefer to keep all the sub-options. Except
maybe the SSL module, that's not really a separate server but a library wrapper, so it should probably be selected automatically if openssl is.
Yes, I tend to agree.
Simon.
PATCH 1/1 sconeserver: new package
In reply to Arnout VandecappelleLe Fri, 27 Jul 2012 10:27:45 +0200,
Arnout Vandecappelle <[email protected]> a ?crit :
Quoted 10 lines
I think it depends a bit on the situation. For openssl support, it
makes more sense to just add it when available, because it normally doesn't increase the size of the binary (much) to have openssl support enabled.However, sconeserver (and vlc as well) are modular packages, which
you can basically view as a collection of sub-packages. So in that case, it does make more sense to select which modules you want to have enabled.
Quoted 4 lines
So for sconeserver, I prefer to keep all the sub-options. Except
maybe the SSL module, that's not really a separate server but a library wrapper, so it should probably be selected automatically if openssl is.
So the patch should be adjusted for the SSL case at least.
Thomas
Signature · 4 lines
Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com
PATCH 1/1 sconeserver: new package
Quoted 1 line · Thomas Petazzoni
So the patch should be adjusted for the SSL case at least.
Yes, agreed.
Simon.
PATCH 1/1 sconeserver: new package
In reply to Arnout VandecappelleQuoted 2 lines · Arnout Vandecappelle
Why doesn't pkg-config find this path? Are they missing a check for
libxml-2.0 maybe?
Sorry to backtrack the thread a little, but I just want to reply to this question before it gets forgotten. Yes, the package configure script does check for libxml2; but the check doesn't appear to be doing quite the right thing. I haven't yet got to the bottom of what's wrong here.
Simon.