[Buildroot] [PATCH v2 1/6] package/perl-db_file: new package

[Buildroot] [PATCH v2 1/6] package/perl-db_file: new package

[Bernd Kuhls]

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: no changes

 package/Config.in                      |    1 +
 package/perl-db_file/Config.in         |   12 ++++++++++++
 package/perl-db_file/perl-db_file.hash |    3 +++
 package/perl-db_file/perl-db_file.mk   |   13 +++++++++++++
 4 files changed, 29 insertions(+)
 create mode 100644 package/perl-db_file/Config.in
 create mode 100644 package/perl-db_file/perl-db_file.hash
 create mode 100644 package/perl-db_file/perl-db_file.mk

diff --git a/package/Config.in b/package/Config.in
index 4561b58..fadb050 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -506,6 +506,7 @@ menu "Perl libraries/modules"
 	source "package/perl-crypt-openssl-random/Config.in"
 	source "package/perl-crypt-openssl-rsa/Config.in"
 	source "package/perl-datetime-tiny/Config.in"
+	source "package/perl-db_file/Config.in"
 	source "package/perl-digest-hmac/Config.in"
 	source "package/perl-digest-sha1/Config.in"
 	source "package/perl-encode-locale/Config.in"
diff --git a/package/perl-db_file/Config.in b/package/perl-db_file/Config.in
new file mode 100644
index 0000000..a2e4516
--- /dev/null
+++ b/package/perl-db_file/Config.in
@@ -0,0 +1,12 @@
+config BR2_PACKAGE_PERL_DB_FILE
+	bool "perl-db_file"
+	depends on !BR2_STATIC_LIBS
+	select BR2_PACKAGE_BERKELEYDB
+	select BR2_PACKAGE_BERKELEYDB_COMPAT185
+	help
+	  Perl5 access to Berkeley DB version 1.x
+
+	  https://metacpan.org/release/DB_File
+
+comment "perl-db_file needs a toolchain w/ dynamic library"
+	depends on BR2_STATIC_LIBS
diff --git a/package/perl-db_file/perl-db_file.hash b/package/perl-db_file/perl-db_file.hash
new file mode 100644
index 0000000..df0113c
--- /dev/null
+++ b/package/perl-db_file/perl-db_file.hash
@@ -0,0 +1,3 @@
+# retrieved by scancpan from http://cpan.metacpan.org/
+md5    0ae7910cabc31a44e50b713a8a475514 DB_File-1.835.tar.gz
+sha256 41206f39a1bac49db8c1595e300b04c70e1393b2d78ccb9ef15c5c0b81037cfc DB_File-1.835.tar.gz
diff --git a/package/perl-db_file/perl-db_file.mk b/package/perl-db_file/perl-db_file.mk
new file mode 100644
index 0000000..1501926
--- /dev/null
+++ b/package/perl-db_file/perl-db_file.mk
@@ -0,0 +1,13 @@
+################################################################################
+#
+# perl-db_file
+#
+################################################################################
+
+PERL_DB_FILE_VERSION = 1.835
+PERL_DB_FILE_SOURCE = DB_File-$(PERL_DB_FILE_VERSION).tar.gz
+PERL_DB_FILE_SITE = $(BR2_CPAN_MIRROR)/authors/id/P/PM/PMQS
+PERL_DB_FILE_DEPENDENCIES = berkeleydb perl
+PERL_DB_FILE_LICENSE = Artistic or GPLv1+
+
+$(eval $(perl-package))
-- 
1.7.10.4

[Buildroot] [PATCH v2 2/6] package/perl-encode-detect: new package

[Bernd Kuhls]

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

---
v2: depends on c++

 package/Config.in                                  |    1 +
 package/perl-encode-detect/Config.in               |   11 +++++++++++
 package/perl-encode-detect/perl-encode-detect.hash |    3 +++
 package/perl-encode-detect/perl-encode-detect.mk   |   14 ++++++++++++++
 4 files changed, 29 insertions(+)
 create mode 100644 package/perl-encode-detect/Config.in
 create mode 100644 package/perl-encode-detect/perl-encode-detect.hash
 create mode 100644 package/perl-encode-detect/perl-encode-detect.mk

diff --git a/package/Config.in b/package/Config.in
index fadb050..d2ee5a7 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -509,6 +509,7 @@ menu "Perl libraries/modules"
 	source "package/perl-db_file/Config.in"
 	source "package/perl-digest-hmac/Config.in"
 	source "package/perl-digest-sha1/Config.in"
+	source "package/perl-encode-detect/Config.in"
 	source "package/perl-encode-locale/Config.in"
 	source "package/perl-file-listing/Config.in"
 	source "package/perl-file-util/Config.in"
diff --git a/package/perl-encode-detect/Config.in b/package/perl-encode-detect/Config.in
new file mode 100644
index 0000000..049eae0
--- /dev/null
+++ b/package/perl-encode-detect/Config.in
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_PERL_ENCODE_DETECT
+	bool "perl-encode-detect"
+	depends on BR2_INSTALL_LIBSTDCPP
+	depends on !BR2_STATIC_LIBS
+	help
+	  An Encode::Encoding subclass that detects the encoding of data
+
+	  https://metacpan.org/release/Encode-Detect
+
+comment "perl-encode-detect needs a toolchain w/ C++, dynamic library"
+	depends on BR2_STATIC_LIBS || !BR2_INSTALL_LIBSTDCPP
diff --git a/package/perl-encode-detect/perl-encode-detect.hash b/package/perl-encode-detect/perl-encode-detect.hash
new file mode 100644
index 0000000..c727a44
--- /dev/null
+++ b/package/perl-encode-detect/perl-encode-detect.hash
@@ -0,0 +1,3 @@
+# retrieved by scancpan from http://cpan.metacpan.org/
+md5    ee9faf55d7105c97b02b8ebe590819c7 Encode-Detect-1.01.tar.gz
+sha256 834d893aa7db6ce3f158afbd0e432d6ed15a276e0940db0a74be13fd9c4bbbf1 Encode-Detect-1.01.tar.gz
diff --git a/package/perl-encode-detect/perl-encode-detect.mk b/package/perl-encode-detect/perl-encode-detect.mk
new file mode 100644
index 0000000..53b882b
--- /dev/null
+++ b/package/perl-encode-detect/perl-encode-detect.mk
@@ -0,0 +1,14 @@
+################################################################################
+#
+# perl-encode-detect
+#
+################################################################################
+
+PERL_ENCODE_DETECT_VERSION = 1.01
+PERL_ENCODE_DETECT_SOURCE = Encode-Detect-$(PERL_ENCODE_DETECT_VERSION).tar.gz
+PERL_ENCODE_DETECT_SITE = $(BR2_CPAN_MIRROR)/authors/id/J/JG/JGMYERS
+PERL_ENCODE_DETECT_DEPENDENCIES = perl host-perl-module-build
+PERL_ENCODE_DETECT_LICENSE = open_source
+PERL_ENCODE_DETECT_LICENSE_FILES = LICENSE
+
+$(eval $(perl-package))
-- 
1.7.10.4

[Buildroot] [PATCH v2 3/6] package/perl-netaddr-ip: Add host variant

[Bernd Kuhls]

Needed to make Mail-SpamAssassin?s dependency check happy.

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: no changes

 package/perl-netaddr-ip/perl-netaddr-ip.mk |    1 +
 1 file changed, 1 insertion(+)

diff --git a/package/perl-netaddr-ip/perl-netaddr-ip.mk b/package/perl-netaddr-ip/perl-netaddr-ip.mk
index e8ace9d..aaa3ef0 100644
--- a/package/perl-netaddr-ip/perl-netaddr-ip.mk
+++ b/package/perl-netaddr-ip/perl-netaddr-ip.mk
@@ -14,3 +14,4 @@ PERL_NETADDR_IP_LICENSE_FILES = Artistic Copying
 PERL_NETADDR_IP_CONF_OPTS = -noxs
 
 $(eval $(perl-package))
+$(eval $(host-perl-package))
-- 
1.7.10.4

[Buildroot] [PATCH v2 4/6] package/perl-net-dns: Add host variant

[Bernd Kuhls]

Needed to make Mail-SpamAssassin?s dependency check happy.

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: no changes

 package/perl-net-dns/perl-net-dns.mk |    1 +
 1 file changed, 1 insertion(+)

diff --git a/package/perl-net-dns/perl-net-dns.mk b/package/perl-net-dns/perl-net-dns.mk
index 75c0a1b..e25d615 100644
--- a/package/perl-net-dns/perl-net-dns.mk
+++ b/package/perl-net-dns/perl-net-dns.mk
@@ -12,3 +12,4 @@ PERL_NET_DNS_LICENSE = Artistic or GPLv1+
 PERL_NET_DNS_LICENSE_FILES = README
 
 $(eval $(perl-package))
+$(eval $(host-perl-package))
-- 
1.7.10.4

[Buildroot] [PATCH v2 5/6] package/perl-html-parser: Add host variant

[Bernd Kuhls]

Needed to make Mail-SpamAssassin?s dependency check happy.

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: no changes

 package/perl-html-parser/perl-html-parser.mk |    1 +
 1 file changed, 1 insertion(+)

diff --git a/package/perl-html-parser/perl-html-parser.mk b/package/perl-html-parser/perl-html-parser.mk
index ecdd7fd..60944fd 100644
--- a/package/perl-html-parser/perl-html-parser.mk
+++ b/package/perl-html-parser/perl-html-parser.mk
@@ -12,3 +12,4 @@ PERL_HTML_PARSER_LICENSE = Artistic or GPLv1+
 PERL_HTML_PARSER_LICENSE_FILES = README
 
 $(eval $(perl-package))
+$(eval $(host-perl-package))
-- 
1.7.10.4

[Buildroot] [PATCH v2 6/6] package/perl-mail-spamassassin: new package

[Bernd Kuhls]

This package contains a dependency check which only checks for installed
modules in host-perl, not the target variant. Therefore some host package
of perl modules are needed as dependency.

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

---
v2: - select perl-encode-detect only when c++ is available

 package/Config.in                                  |    1 +
 package/perl-mail-spamassassin/Config.in           |   18 ++++++++++++++++
 .../perl-mail-spamassassin.hash                    |    3 +++
 .../perl-mail-spamassassin.mk                      |   22 ++++++++++++++++++++
 4 files changed, 44 insertions(+)
 create mode 100644 package/perl-mail-spamassassin/Config.in
 create mode 100644 package/perl-mail-spamassassin/perl-mail-spamassassin.hash
 create mode 100644 package/perl-mail-spamassassin/perl-mail-spamassassin.mk

diff --git a/package/Config.in b/package/Config.in
index d2ee5a7..34435a0 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -529,6 +529,7 @@ menu "Perl libraries/modules"
 	source "package/perl-libwww-perl/Config.in"
 	source "package/perl-lwp-mediatypes/Config.in"
 	source "package/perl-mail-dkim/Config.in"
+	source "package/perl-mail-spamassassin/Config.in"
 	source "package/perl-mailtools/Config.in"
 	source "package/perl-mime-base64/Config.in"
 	source "package/perl-mojolicious/Config.in"
diff --git a/package/perl-mail-spamassassin/Config.in b/package/perl-mail-spamassassin/Config.in
new file mode 100644
index 0000000..799a6a7
--- /dev/null
+++ b/package/perl-mail-spamassassin/Config.in
@@ -0,0 +1,18 @@
+config BR2_PACKAGE_PERL_MAIL_SPAMASSASSIN
+	bool "perl-mail-spamassassin"
+	depends on !BR2_STATIC_LIBS
+	select BR2_PACKAGE_PERL_DB_FILE
+	select BR2_PACKAGE_PERL_DIGEST_SHA1
+	select BR2_PACKAGE_PERL_ENCODE_DETECT if BR2_INSTALL_LIBSTDCPP
+	select BR2_PACKAGE_PERL_HTML_PARSER
+	select BR2_PACKAGE_PERL_MAIL_DKIM
+	select BR2_PACKAGE_PERL_NET_DNS
+	select BR2_PACKAGE_PERL_NETADDR_IP
+	help
+	  SpamAssassin is an extensible email filter which is used to
+	  identify spam
+
+	  http://spamassassin.apache.com/
+
+comment "perl-mail-spamassassin needs a toolchain w/ dynamic library"
+	depends on BR2_STATIC_LIBS
diff --git a/package/perl-mail-spamassassin/perl-mail-spamassassin.hash b/package/perl-mail-spamassassin/perl-mail-spamassassin.hash
new file mode 100644
index 0000000..7e2896f
--- /dev/null
+++ b/package/perl-mail-spamassassin/perl-mail-spamassassin.hash
@@ -0,0 +1,3 @@
+# retrieved by scancpan from http://cpan.metacpan.org/
+md5    5d0b50cee3bfa905cca35c33296c8c2a Mail-SpamAssassin-3.4.0.tar.gz
+sha256 244914c30976844878a7f129fd503eb40986c68a3800f416c3a68b14507c0a64 Mail-SpamAssassin-3.4.0.tar.gz
diff --git a/package/perl-mail-spamassassin/perl-mail-spamassassin.mk b/package/perl-mail-spamassassin/perl-mail-spamassassin.mk
new file mode 100644
index 0000000..227c76d
--- /dev/null
+++ b/package/perl-mail-spamassassin/perl-mail-spamassassin.mk
@@ -0,0 +1,22 @@
+################################################################################
+#
+# perl-mail-spamassassin
+#
+################################################################################
+
+PERL_MAIL_SPAMASSASSIN_VERSION = 3.4.0
+PERL_MAIL_SPAMASSASSIN_SOURCE = Mail-SpamAssassin-$(PERL_MAIL_SPAMASSASSIN_VERSION).tar.gz
+PERL_MAIL_SPAMASSASSIN_SITE = $(BR2_CPAN_MIRROR)/authors/id/K/KM/KMCGRAIL/SpamAssassin
+PERL_MAIL_SPAMASSASSIN_LICENSE = Artistic or GPLv1+
+PERL_MAIL_SPAMASSASSIN_LICENSE_FILES = LICENSE
+PERL_MAIL_SPAMASSASSIN_DEPENDENCIES = \
+	perl \
+	perl-db_file \
+	perl-digest-sha1 \
+	perl-encode-detect \
+	perl-html-parser host-perl-html-parser \
+	perl-mail-dkim \
+	perl-net-dns host-perl-net-dns \
+	perl-netaddr-ip host-perl-netaddr-ip
+
+$(eval $(perl-package))
-- 
1.7.10.4

[Buildroot] [PATCH v2 6/6] package/perl-mail-spamassassin: new package

[Thomas Petazzoni]

Dear Bernd Kuhls,

On Sun, 26 Apr 2015 11:37:19 +0200, Bernd Kuhls wrote:
> This package contains a dependency check which only checks for installed
> modules in host-perl, not the target variant. Therefore some host package
> of perl modules are needed as dependency.

What about fixing Spamassassin instead?

> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> 
> ---
> v2: - select perl-encode-detect only when c++ is available

And this seems broken. Read on below.

> diff --git a/package/perl-mail-spamassassin/Config.in b/package/perl-mail-spamassassin/Config.in
> new file mode 100644
> index 0000000..799a6a7
> --- /dev/null
> +++ b/package/perl-mail-spamassassin/Config.in
> @@ -0,0 +1,18 @@
> +config BR2_PACKAGE_PERL_MAIL_SPAMASSASSIN
> +	bool "perl-mail-spamassassin"
> +	depends on !BR2_STATIC_LIBS
> +	select BR2_PACKAGE_PERL_DB_FILE
> +	select BR2_PACKAGE_PERL_DIGEST_SHA1
> +	select BR2_PACKAGE_PERL_ENCODE_DETECT if BR2_INSTALL_LIBSTDCPP

So if C++ is not available, we are not selecting
BR2_PACKAGE_PERL_ENCODE_DETECT.

> +PERL_MAIL_SPAMASSASSIN_DEPENDENCIES = \
> +	perl \
> +	perl-db_file \
> +	perl-digest-sha1 \
> +	perl-encode-detect \

But we depend on it anyway.

So really, there are only a few solutions:

 *) perl-encode-detect is a mandatory dependency of spamassassin. In
    this case, spamassassin must have a depends on C++.

 *) perl-encode-detect is an optional dependency of spamassassin. In
    this case, do nothing in Config.in, and use ifeq
    ($(BR2_PACKAGE_PERL_ENCODE_DETECT),y) in the .mk file to add it in
    PERL_MAIL_SPAMASSASSIN_DEPENDENCIES.

 *) or if you really want to enable it automatically when C++ is
    available, keep your select in the Config.in, but use ifeq
    ($(BR2_PACKAGE_PERL_ENCODE_DETECT),y) in the .mk file to add it in
    PERL_MAIL_SPAMASSASSIN_DEPENDENCIES.

Best regards,

Thomas Petazzoni
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v2 1/6] package/perl-db_file: new package

[Thomas Petazzoni]

Dear Bernd Kuhls,

On Sun, 26 Apr 2015 11:37:14 +0200, Bernd Kuhls wrote:
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v2: no changes

I've applied, after renaming to perl-db-file instead of perl-db_file.
Probably the scancpan script should be adjusted: we prefer '-' as a
separator in package names, rather than '_'. Yes I know there are some
counter examples, especially X.org packages.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v2 2/6] package/perl-encode-detect: new package

[Thomas Petazzoni]

Dear Bernd Kuhls,

On Sun, 26 Apr 2015 11:37:15 +0200, Bernd Kuhls wrote:

> diff --git a/package/perl-encode-detect/Config.in b/package/perl-encode-detect/Config.in
> new file mode 100644
> index 0000000..049eae0
> --- /dev/null
> +++ b/package/perl-encode-detect/Config.in
> @@ -0,0 +1,11 @@
> +config BR2_PACKAGE_PERL_ENCODE_DETECT
> +	bool "perl-encode-detect"
> +	depends on BR2_INSTALL_LIBSTDCPP
> +	depends on !BR2_STATIC_LIBS
> +	help
> +	  An Encode::Encoding subclass that detects the encoding of data

Line slightly too long, and should end with a '.'.

> new file mode 100644
> index 0000000..53b882b
> --- /dev/null
> +++ b/package/perl-encode-detect/perl-encode-detect.mk
> @@ -0,0 +1,14 @@
> +################################################################################
> +#
> +# perl-encode-detect
> +#
> +################################################################################
> +
> +PERL_ENCODE_DETECT_VERSION = 1.01
> +PERL_ENCODE_DETECT_SOURCE = Encode-Detect-$(PERL_ENCODE_DETECT_VERSION).tar.gz
> +PERL_ENCODE_DETECT_SITE = $(BR2_CPAN_MIRROR)/authors/id/J/JG/JGMYERS
> +PERL_ENCODE_DETECT_DEPENDENCIES = perl host-perl-module-build
> +PERL_ENCODE_DETECT_LICENSE = open_source

open_source is not really a good license description. The license is
really MPL-1.1.

Applied with those two issues fixed.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v2 6/6] package/perl-mail-spamassassin: new package

[Bernd Kuhls]

Thomas Petazzoni <thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8
@public.gmane.org> wrote in news:20150426120256.63387117 at free-electrons.com:

> Dear Bernd Kuhls,
> 
> On Sun, 26 Apr 2015 11:37:19 +0200, Bernd Kuhls wrote:
>> This package contains a dependency check which only checks for installed
>> modules in host-perl, not the target variant. Therefore some host package
>> of perl modules are needed as dependency.
> 
> What about fixing Spamassassin instead?

Hi Thomas,

I tried and failed:
http://article.gmane.org/gmane.comp.lib.uclibc.buildroot/114369

Regards, Bernd

[Buildroot] [PATCH v2 6/6] package/perl-mail-spamassassin: new package

[Thomas Petazzoni]

Bernd, Fran?ois,

On Sun, 26 Apr 2015 12:55:29 +0200, Bernd Kuhls wrote:

> > On Sun, 26 Apr 2015 11:37:19 +0200, Bernd Kuhls wrote:
> >> This package contains a dependency check which only checks for installed
> >> modules in host-perl, not the target variant. Therefore some host package
> >> of perl modules are needed as dependency.
> > 
> > What about fixing Spamassassin instead?
> 
> Hi Thomas,
> 
> I tried and failed:
> http://article.gmane.org/gmane.comp.lib.uclibc.buildroot/114369

Argh. Fran?ois, maybe you can help on this by having a look?

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v2 6/6] package/perl-mail-spamassassin: new package

[Bernd Kuhls]

Am Sun, 26 Apr 2015 22:25:23 +0200 schrieb Thomas Petazzoni:

> Bernd, Fran?ois,
> 
> On Sun, 26 Apr 2015 12:55:29 +0200, Bernd Kuhls wrote:
> 
>> > On Sun, 26 Apr 2015 11:37:19 +0200, Bernd Kuhls wrote:
>> >> This package contains a dependency check which only checks for
>> >> installed modules in host-perl, not the target variant. Therefore
>> >> some host package of perl modules are needed as dependency.
>> > 
>> > What about fixing Spamassassin instead?
>> 
>> Hi Thomas,
>> 
>> I tried and failed:
>> http://article.gmane.org/gmane.comp.lib.uclibc.buildroot/114369
> 
> Argh. Fran?ois, maybe you can help on this by having a look?

Hi Thomas,

I am closing my Spamassassin patch series

http://patchwork.ozlabs.org/patch/464634/
http://patchwork.ozlabs.org/patch/464631/
http://patchwork.ozlabs.org/patch/464633/
http://patchwork.ozlabs.org/patch/464635/

as superseded because Fran?ois fixed the Spamassassin configure problem:
http://patchwork.ozlabs.org/patch/552870/

Regards, Bernd