* [Buildroot] [PATCH v2 1/2] pcre: add UTF support option
@ 2013-10-25 10:28 yegorslists at googlemail.com
2013-10-25 10:28 ` [Buildroot] [PATCH v2 2/2] ModemManager: add new package yegorslists at googlemail.com
2013-10-27 17:24 ` [Buildroot] [PATCH v2 1/2] pcre: add UTF support option Thomas Petazzoni
0 siblings, 2 replies; 6+ messages in thread
From: yegorslists at googlemail.com @ 2013-10-25 10:28 UTC (permalink / raw)
To: buildroot
From: Yegor Yefremov <yegorslists@googlemail.com>
Add BR2_PACKAGE_PCRE_UTF option, that activates
--enable-unicode-properties for UTF-8/16/32.
Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
package/pcre/Config.in | 5 +++++
package/pcre/pcre.mk | 1 +
2 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/package/pcre/Config.in b/package/pcre/Config.in
index dc92274..3ca27c0 100644
--- a/package/pcre/Config.in
+++ b/package/pcre/Config.in
@@ -19,4 +19,9 @@ config BR2_PACKAGE_PCRE_32
help
This option builds the 32-bits pcre library, i.e 'libpcre32'
+config BR2_PACKAGE_PCRE_UTF
+ bool "UTF support"
+ help
+ This option enables UTF support
+
endif
diff --git a/package/pcre/pcre.mk b/package/pcre/pcre.mk
index bf86a3d..0d1d66b 100644
--- a/package/pcre/pcre.mk
+++ b/package/pcre/pcre.mk
@@ -19,5 +19,6 @@ endif
PCRE_CONF_OPT += --enable-pcre8
PCRE_CONF_OPT += $(if $(BR2_PACKAGE_PCRE_16),--enable-pcre16,--disable-pcre16)
PCRE_CONF_OPT += $(if $(BR2_PACKAGE_PCRE_32),--enable-pcre32,--disable-pcre32)
+PCRE_CONF_OPT += $(if $(BR2_PACKAGE_PCRE_UTF),--enable-unicode-properties,--disable-unicode-properties)
$(eval $(autotools-package))
--
1.7.7
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 2/2] ModemManager: add new package
2013-10-25 10:28 [Buildroot] [PATCH v2 1/2] pcre: add UTF support option yegorslists at googlemail.com
@ 2013-10-25 10:28 ` yegorslists at googlemail.com
2013-10-25 10:55 ` Yegor Yefremov
2013-10-27 17:42 ` Thomas Petazzoni
2013-10-27 17:24 ` [Buildroot] [PATCH v2 1/2] pcre: add UTF support option Thomas Petazzoni
1 sibling, 2 replies; 6+ messages in thread
From: yegorslists at googlemail.com @ 2013-10-25 10:28 UTC (permalink / raw)
To: buildroot
From: Yegor Yefremov <yegorslists@googlemail.com>
Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
Changes:
v2: typos, add select QMI to the config option, don't install to staging
package/Config.in | 1 +
package/modem-manager/Config.in | 33 ++++++++++++++++++++++++++++++++
package/modem-manager/modem-manager.mk | 21 ++++++++++++++++++++
3 files changed, 55 insertions(+), 0 deletions(-)
create mode 100644 package/modem-manager/Config.in
create mode 100644 package/modem-manager/modem-manager.mk
diff --git a/package/Config.in b/package/Config.in
index 0e6f86c..34c0e7b 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -820,6 +820,7 @@ source "package/lrzsz/Config.in"
source "package/macchanger/Config.in"
source "package/mii-diag/Config.in"
source "package/minidlna/Config.in"
+source "package/modem-manager/Config.in"
source "package/mongoose/Config.in"
source "package/mongrel2/Config.in"
source "package/mrouted/Config.in"
diff --git a/package/modem-manager/Config.in b/package/modem-manager/Config.in
new file mode 100644
index 0000000..202d846
--- /dev/null
+++ b/package/modem-manager/Config.in
@@ -0,0 +1,33 @@
+config BR2_PACKAGE_MODEM_MANAGER
+ bool "ModemManager"
+ depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
+ select BR2_PACKAGE_DBUS
+ depends on BR2_INET_IPV6
+ depends on BR2_LARGEFILE # acl
+ depends on BR2_USE_WCHAR # libglib2 and gnutls
+ depends on BR2_TOOLCHAIN_HAS_THREADS # dbus, libglib2
+ depends on BR2_USE_MMU # dbus
+ select BR2_PACKAGE_DBUS_GLIB
+ select BR2_PACKAGE_UDEV
+ select BR2_PACKAGE_UDEV_ALL_EXTRAS
+ select BR2_PACKAGE_PCRE
+ select BR2_PACKAGE_PCRE_UTF
+ help
+ ModemManager is a DBus-activated daemon which controls mobile
+ broadband (2G/3G/4G) devices and connections.
+
+ http://www.freedesktop.org/wiki/Software/ModemManager/
+
+if BR2_PACKAGE_MODEM_MANAGER
+
+config BR2_PACKAGE_MODEM_MANAGER_LIBQMI
+ bool "QMI support"
+ select BR2_PACKAGE_LIBQMI
+ help
+ This option enables support for QMI protocol
+endif
+
+comment "ModemkManager requires a toolchain with IPV6, LARGEFILE, WCHAR and thread support"
+ depends on !BR2_INET_IPV6 || !BR2_LARGEFILE || !BR2_USE_WCHAR || \
+ !BR2_TOOLCHAIN_HAS_THREADS
+
diff --git a/package/modem-manager/modem-manager.mk b/package/modem-manager/modem-manager.mk
new file mode 100644
index 0000000..ebf2617
--- /dev/null
+++ b/package/modem-manager/modem-manager.mk
@@ -0,0 +1,21 @@
+################################################################################
+#
+# modem-manager
+#
+################################################################################
+
+MODEM_MANAGER_VERSION = 1.0.0
+MODEM_MANAGER_SOURCE = ModemManager-$(MODEM_MANAGER_VERSION).tar.xz
+MODEM_MANAGER_SITE = http://www.freedesktop.org/software/ModemManager/
+MODEM_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib host-intltool
+
+MODEM_MANAGER_CONF_OPT = \
+ --without-mbim
+
+ifeq ($(BR2_PACKAGE_MODEM_MANAGER_LIBQMI),y)
+ MODEM_MANAGER_DEPENDENCIES += libqmi
+else
+ MODEM_MANAGER_CONF_OPT += --without-qmi
+endif
+
+$(eval $(autotools-package))
--
1.7.7
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 2/2] ModemManager: add new package
2013-10-25 10:28 ` [Buildroot] [PATCH v2 2/2] ModemManager: add new package yegorslists at googlemail.com
@ 2013-10-25 10:55 ` Yegor Yefremov
2013-10-27 17:42 ` Thomas Petazzoni
1 sibling, 0 replies; 6+ messages in thread
From: Yegor Yefremov @ 2013-10-25 10:55 UTC (permalink / raw)
To: buildroot
On Fri, Oct 25, 2013 at 12:28 PM, <yegorslists@googlemail.com> wrote:
> From: Yegor Yefremov <yegorslists@googlemail.com>
>
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
> ---
> Changes:
> v2: typos, add select QMI to the config option, don't install to staging
>
> package/Config.in | 1 +
> package/modem-manager/Config.in | 33 ++++++++++++++++++++++++++++++++
> package/modem-manager/modem-manager.mk | 21 ++++++++++++++++++++
> 3 files changed, 55 insertions(+), 0 deletions(-)
> create mode 100644 package/modem-manager/Config.in
> create mode 100644 package/modem-manager/modem-manager.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 0e6f86c..34c0e7b 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -820,6 +820,7 @@ source "package/lrzsz/Config.in"
> source "package/macchanger/Config.in"
> source "package/mii-diag/Config.in"
> source "package/minidlna/Config.in"
> +source "package/modem-manager/Config.in"
> source "package/mongoose/Config.in"
> source "package/mongrel2/Config.in"
> source "package/mrouted/Config.in"
> diff --git a/package/modem-manager/Config.in b/package/modem-manager/Config.in
> new file mode 100644
> index 0000000..202d846
> --- /dev/null
> +++ b/package/modem-manager/Config.in
> @@ -0,0 +1,33 @@
> +config BR2_PACKAGE_MODEM_MANAGER
> + bool "ModemManager"
> + depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
> + select BR2_PACKAGE_DBUS
> + depends on BR2_INET_IPV6
> + depends on BR2_LARGEFILE # acl
> + depends on BR2_USE_WCHAR # libglib2 and gnutls
> + depends on BR2_TOOLCHAIN_HAS_THREADS # dbus, libglib2
> + depends on BR2_USE_MMU # dbus
> + select BR2_PACKAGE_DBUS_GLIB
> + select BR2_PACKAGE_UDEV
> + select BR2_PACKAGE_UDEV_ALL_EXTRAS
> + select BR2_PACKAGE_PCRE
> + select BR2_PACKAGE_PCRE_UTF
> + help
> + ModemManager is a DBus-activated daemon which controls mobile
> + broadband (2G/3G/4G) devices and connections.
> +
> + http://www.freedesktop.org/wiki/Software/ModemManager/
> +
> +if BR2_PACKAGE_MODEM_MANAGER
> +
> +config BR2_PACKAGE_MODEM_MANAGER_LIBQMI
> + bool "QMI support"
> + select BR2_PACKAGE_LIBQMI
> + help
> + This option enables support for QMI protocol
> +endif
> +
> +comment "ModemkManager requires a toolchain with IPV6, LARGEFILE, WCHAR and thread support"
> + depends on !BR2_INET_IPV6 || !BR2_LARGEFILE || !BR2_USE_WCHAR || \
> + !BR2_TOOLCHAIN_HAS_THREADS
> +
> diff --git a/package/modem-manager/modem-manager.mk b/package/modem-manager/modem-manager.mk
> new file mode 100644
> index 0000000..ebf2617
> --- /dev/null
> +++ b/package/modem-manager/modem-manager.mk
> @@ -0,0 +1,21 @@
> +################################################################################
> +#
> +# modem-manager
> +#
> +################################################################################
> +
> +MODEM_MANAGER_VERSION = 1.0.0
> +MODEM_MANAGER_SOURCE = ModemManager-$(MODEM_MANAGER_VERSION).tar.xz
> +MODEM_MANAGER_SITE = http://www.freedesktop.org/software/ModemManager/
Forgot about licencing information :-(
MODEM_MANAGER_LICENSE = GPLv2
MODEM_MANAGER_LICENSE_FILES = COPYING
> +MODEM_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib host-intltool
> +
> +MODEM_MANAGER_CONF_OPT = \
> + --without-mbim
> +
> +ifeq ($(BR2_PACKAGE_MODEM_MANAGER_LIBQMI),y)
> + MODEM_MANAGER_DEPENDENCIES += libqmi
> +else
> + MODEM_MANAGER_CONF_OPT += --without-qmi
> +endif
> +
> +$(eval $(autotools-package))
> --
> 1.7.7
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 2/2] ModemManager: add new package
2013-10-25 10:28 ` [Buildroot] [PATCH v2 2/2] ModemManager: add new package yegorslists at googlemail.com
2013-10-25 10:55 ` Yegor Yefremov
@ 2013-10-27 17:42 ` Thomas Petazzoni
2013-10-27 19:31 ` Yegor Yefremov
1 sibling, 1 reply; 6+ messages in thread
From: Thomas Petazzoni @ 2013-10-27 17:42 UTC (permalink / raw)
To: buildroot
Hello,
On Fri, 25 Oct 2013 12:28:18 +0200, yegorslists at googlemail.com wrote:
> diff --git a/package/modem-manager/Config.in b/package/modem-manager/Config.in
> new file mode 100644
> index 0000000..202d846
> --- /dev/null
> +++ b/package/modem-manager/Config.in
> @@ -0,0 +1,33 @@
> +config BR2_PACKAGE_MODEM_MANAGER
> + bool "ModemManager"
I think we want all the package names to be lower case.
> + depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
> + select BR2_PACKAGE_DBUS
> + depends on BR2_INET_IPV6
> + depends on BR2_LARGEFILE # acl
> + depends on BR2_USE_WCHAR # libglib2 and gnutls
> + depends on BR2_TOOLCHAIN_HAS_THREADS # dbus, libglib2
> + depends on BR2_USE_MMU # dbus
> + select BR2_PACKAGE_DBUS_GLIB
> + select BR2_PACKAGE_UDEV
> + select BR2_PACKAGE_UDEV_ALL_EXTRAS
> + select BR2_PACKAGE_PCRE
> + select BR2_PACKAGE_PCRE_UTF
> + help
> + ModemManager is a DBus-activated daemon which controls mobile
> + broadband (2G/3G/4G) devices and connections.
> +
> + http://www.freedesktop.org/wiki/Software/ModemManager/
> +
> +if BR2_PACKAGE_MODEM_MANAGER
> +
> +config BR2_PACKAGE_MODEM_MANAGER_LIBQMI
> + bool "QMI support"
> + select BR2_PACKAGE_LIBQMI
> + help
> + This option enables support for QMI protocol
> +endif
> +
> +comment "ModemkManager requires a toolchain with IPV6, LARGEFILE, WCHAR and thread support"
typo on ModemManager (which should be lower case as well here).
> + depends on !BR2_INET_IPV6 || !BR2_LARGEFILE || !BR2_USE_WCHAR || \
> + !BR2_TOOLCHAIN_HAS_THREADS
Shouldn't we have a comment about the udev dependency?
> +MODEM_MANAGER_VERSION = 1.0.0
> +MODEM_MANAGER_SOURCE = ModemManager-$(MODEM_MANAGER_VERSION).tar.xz
> +MODEM_MANAGER_SITE = http://www.freedesktop.org/software/ModemManager/
> +MODEM_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib host-intltool
You don't depend on pcre?
Also, as you mentioned in a reply, please add the licensing
information. Note that it seems to be GPLv2+ and not GPLv2.
> +MODEM_MANAGER_CONF_OPT = \
> + --without-mbim
One too many tabs in this indentation.
> +ifeq ($(BR2_PACKAGE_MODEM_MANAGER_LIBQMI),y)
> + MODEM_MANAGER_DEPENDENCIES += libqmi
MODEM_MANAGER_CONF_OPT += --with-qmi
> +else
> + MODEM_MANAGER_CONF_OPT += --without-qmi
> +endif
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 2/2] ModemManager: add new package
2013-10-27 17:42 ` Thomas Petazzoni
@ 2013-10-27 19:31 ` Yegor Yefremov
0 siblings, 0 replies; 6+ messages in thread
From: Yegor Yefremov @ 2013-10-27 19:31 UTC (permalink / raw)
To: buildroot
On Sun, Oct 27, 2013 at 6:42 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Fri, 25 Oct 2013 12:28:18 +0200, yegorslists at googlemail.com wrote:
>> diff --git a/package/modem-manager/Config.in b/package/modem-manager/Config.in
>> new file mode 100644
>> index 0000000..202d846
>> --- /dev/null
>> +++ b/package/modem-manager/Config.in
>> @@ -0,0 +1,33 @@
>> +config BR2_PACKAGE_MODEM_MANAGER
>> + bool "ModemManager"
>
> I think we want all the package names to be lower case.
Should NetworkManager be also renamed in networkmanager?
>> + depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
>> + select BR2_PACKAGE_DBUS
>> + depends on BR2_INET_IPV6
>> + depends on BR2_LARGEFILE # acl
>> + depends on BR2_USE_WCHAR # libglib2 and gnutls
>> + depends on BR2_TOOLCHAIN_HAS_THREADS # dbus, libglib2
>> + depends on BR2_USE_MMU # dbus
>> + select BR2_PACKAGE_DBUS_GLIB
>> + select BR2_PACKAGE_UDEV
>> + select BR2_PACKAGE_UDEV_ALL_EXTRAS
>> + select BR2_PACKAGE_PCRE
>> + select BR2_PACKAGE_PCRE_UTF
>> + help
>> + ModemManager is a DBus-activated daemon which controls mobile
>> + broadband (2G/3G/4G) devices and connections.
>> +
>> + http://www.freedesktop.org/wiki/Software/ModemManager/
>> +
>> +if BR2_PACKAGE_MODEM_MANAGER
>> +
>> +config BR2_PACKAGE_MODEM_MANAGER_LIBQMI
>> + bool "QMI support"
>> + select BR2_PACKAGE_LIBQMI
>> + help
>> + This option enables support for QMI protocol
>> +endif
>> +
>> +comment "ModemkManager requires a toolchain with IPV6, LARGEFILE, WCHAR and thread support"
>
> typo on ModemManager (which should be lower case as well here).
ACK
>> + depends on !BR2_INET_IPV6 || !BR2_LARGEFILE || !BR2_USE_WCHAR || \
>> + !BR2_TOOLCHAIN_HAS_THREADS
>
> Shouldn't we have a comment about the udev dependency?
ACK
>> +MODEM_MANAGER_VERSION = 1.0.0
>> +MODEM_MANAGER_SOURCE = ModemManager-$(MODEM_MANAGER_VERSION).tar.xz
>> +MODEM_MANAGER_SITE = http://www.freedesktop.org/software/ModemManager/
>> +MODEM_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib host-intltool
>
> You don't depend on pcre?
ACK
> Also, as you mentioned in a reply, please add the licensing
> information. Note that it seems to be GPLv2+ and not GPLv2.
>
>> +MODEM_MANAGER_CONF_OPT = \
>> + --without-mbim
>
> One too many tabs in this indentation.
ACK
>> +ifeq ($(BR2_PACKAGE_MODEM_MANAGER_LIBQMI),y)
>> + MODEM_MANAGER_DEPENDENCIES += libqmi
>
> MODEM_MANAGER_CONF_OPT += --with-qmi
ACK
>> +else
>> + MODEM_MANAGER_CONF_OPT += --without-qmi
>> +endif
Thanks for review.
Yegor
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 1/2] pcre: add UTF support option
2013-10-25 10:28 [Buildroot] [PATCH v2 1/2] pcre: add UTF support option yegorslists at googlemail.com
2013-10-25 10:28 ` [Buildroot] [PATCH v2 2/2] ModemManager: add new package yegorslists at googlemail.com
@ 2013-10-27 17:24 ` Thomas Petazzoni
1 sibling, 0 replies; 6+ messages in thread
From: Thomas Petazzoni @ 2013-10-27 17:24 UTC (permalink / raw)
To: buildroot
Hello,
On Fri, 25 Oct 2013 12:28:17 +0200, yegorslists at googlemail.com wrote:
> +config BR2_PACKAGE_PCRE_UTF
> + bool "UTF support"
> + help
> + This option enables UTF support
I would maybe change this to UTF-8/16/32 support, which is what
the ./configure help text says.
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-10-27 19:31 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-25 10:28 [Buildroot] [PATCH v2 1/2] pcre: add UTF support option yegorslists at googlemail.com
2013-10-25 10:28 ` [Buildroot] [PATCH v2 2/2] ModemManager: add new package yegorslists at googlemail.com
2013-10-25 10:55 ` Yegor Yefremov
2013-10-27 17:42 ` Thomas Petazzoni
2013-10-27 19:31 ` Yegor Yefremov
2013-10-27 17:24 ` [Buildroot] [PATCH v2 1/2] pcre: add UTF support option Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox