Skip to contentSkip to conversation

package/perl-db_file: new package

PATCH New Bernd Kuhls 2 participants 12 messages Apr 26, 2015 – Dec 5, 2015

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/6 package/perl-db_file: new package

---
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
package/Config.in +1 -0
index 4561b58..fadb050 100644
@@ -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"
package/perl-db_file/Config.in new file +12 -0
new file mode 100644
index 0000000..a2e4516
@@ -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
package/perl-db_file/perl-db_file.hash new file +3 -0
new file mode 100644
index 0000000..df0113c
@@ -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
package/perl-db_file/perl-db_file.mk new file +13 -0
new file mode 100644
index 0000000..1501926
@@ -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))
Signature · 1 line
1.7.10.4
#1 lore raw

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

---
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
package/Config.in +1 -0
index fadb050..d2ee5a7 100644
@@ -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"
package/perl-encode-detect/Config.in new file +11 -0
new file mode 100644
index 0000000..049eae0
@@ -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
package/perl-encode-detect/perl-encode-detect.hash new file +3 -0
new file mode 100644
index 0000000..c727a44
@@ -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
package/perl-encode-detect/perl-encode-detect.mk new file +14 -0
new file mode 100644
index 0000000..53b882b
@@ -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))
Signature · 1 line
1.7.10.4
#2 lore raw

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

Dear Bernd Kuhls,

package/perl-encode-detect/Config.in line 6

Quoted 12 lines · Bernd Kuhls
package/perl-encode-detect/Config.in new file +6 -0
new file mode 100644
index 0000000..049eae0
@@ -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 '.'.

Quoted 6 lines

new file mode 100644
index 0000000..53b882b

package/perl-encode-detect/perl-encode-detect.mk new file +1 -0
@@ -0,0 +1,14 @@
+################################################################################

package/perl-encode-detect/perl-encode-detect.mk line 11

Quoted 10 lines
package/perl-encode-detect/perl-encode-detect.mk new file +10 -0
+#
+# 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

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#3 lore raw

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

In reply to Bernd Kuhls

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

---
v2: no changes

 package/perl-netaddr-ip/perl-netaddr-ip.mk |    1 +
 1 file changed, 1 insertion(+)
package/perl-netaddr-ip/perl-netaddr-ip.mk +1 -0
index e8ace9d..aaa3ef0 100644
@@ -14,3 +14,4 @@ PERL_NETADDR_IP_LICENSE_FILES = Artistic Copying
 PERL_NETADDR_IP_CONF_OPTS = -noxs
 
 $(eval $(perl-package))
+$(eval $(host-perl-package))
Signature · 1 line
1.7.10.4
#4 lore raw

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

In reply to Bernd Kuhls

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

---
v2: no changes

 package/perl-net-dns/perl-net-dns.mk |    1 +
 1 file changed, 1 insertion(+)
package/perl-net-dns/perl-net-dns.mk +1 -0
index 75c0a1b..e25d615 100644
@@ -12,3 +12,4 @@ PERL_NET_DNS_LICENSE = Artistic or GPLv1+
 PERL_NET_DNS_LICENSE_FILES = README
 
 $(eval $(perl-package))
+$(eval $(host-perl-package))
Signature · 1 line
1.7.10.4
#5 lore raw

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

In reply to Bernd Kuhls

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

---
v2: no changes

 package/perl-html-parser/perl-html-parser.mk |    1 +
 1 file changed, 1 insertion(+)
package/perl-html-parser/perl-html-parser.mk +1 -0
index ecdd7fd..60944fd 100644
@@ -12,3 +12,4 @@ PERL_HTML_PARSER_LICENSE = Artistic or GPLv1+
 PERL_HTML_PARSER_LICENSE_FILES = README
 
 $(eval $(perl-package))
+$(eval $(host-perl-package))
Signature · 1 line
1.7.10.4
#6 lore raw

PATCH v2 6/6 package/perl-mail-spamassassin: new package

In reply to 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.

---
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
package/Config.in +1 -0
index d2ee5a7..34435a0 100644
@@ -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"
package/perl-mail-spamassassin/Config.in new file +18 -0
new file mode 100644
index 0000000..799a6a7
@@ -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
package/perl-mail-spamassassin/perl-mail-spamassassin.hash new file +3 -0
new file mode 100644
index 0000000..7e2896f
@@ -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
package/perl-mail-spamassassin/perl-mail-spamassassin.mk new file +22 -0
new file mode 100644
index 0000000..227c76d
@@ -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))
Signature · 1 line
1.7.10.4
#7 lore raw

PATCH v2 6/6 package/perl-mail-spamassassin: new package

Dear Bernd Kuhls,

Quoted 3 lines · 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.

What about fixing Spamassassin instead?

Quoted 5 lines

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

And this seems broken. Read on below.

package/perl-mail-spamassassin/Config.in line 6

Quoted 12 lines
package/perl-mail-spamassassin/Config.in new file +6 -0
new file mode 100644
index 0000000..799a6a7
@@ -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.

package/perl-mail-spamassassin/Config.in line 11

Quoted 5 lines
package/perl-mail-spamassassin/Config.in new file +5 -0
+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

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#8 lore raw

PATCH v2 6/6 package/perl-mail-spamassassin: new package

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

Quoted 8 lines

Dear Bernd Kuhls,

Quoted 3 lines · 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.

What about fixing Spamassassin instead?

Hi Thomas,

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

Regards, Bernd

#9 lore raw

PATCH v2 6/6 package/perl-mail-spamassassin: new package

Bernd, Fran?ois,

Quoted 11 lines · Bernd Kuhls
Quoted 6 lines · Bernd Kuhls
Quoted 3 lines

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

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#10 lore raw

PATCH v2 6/6 package/perl-mail-spamassassin: new package

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

Quoted 9 lines

Bernd, Fran?ois,

Quoted 5 lines · Bernd Kuhls
Quoted 5 lines · Bernd Kuhls
Quoted 3 lines

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.

Quoted 8 lines
Quoted 6 lines
Quoted 1 line

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

#11 lore raw

PATCH v2 1/6 package/perl-db_file: new package

In reply to Bernd Kuhls

Dear Bernd Kuhls,

Quoted 4 lines · Bernd Kuhls

---
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

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