[meta-perl][PATCH 1/2] libsub-exporter-progressive-perl: Replace DEPENDS_PN with DEPENDS

[meta-perl][PATCH 1/2] libsub-exporter-progressive-perl: Replace DEPENDS_PN with DEPENDS

[Khem Raj]

Fixes
libsub-exporter-progressive-perl_0.001013.bb: QA Issue: recipe uses DEPENDS_${PN}, should use DEPENDS [pkgvarcheck]

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../libsub/libsub-exporter-progressive-perl_0.001013.bb         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-perl/recipes-perl/libsub/libsub-exporter-progressive-perl_0.001013.bb b/meta-perl/recipes-perl/libsub/libsub-exporter-progressive-perl_0.001013.bb
index 64d19c0f1f..e484f4a7a0 100644
--- a/meta-perl/recipes-perl/libsub/libsub-exporter-progressive-perl_0.001013.bb
+++ b/meta-perl/recipes-perl/libsub/libsub-exporter-progressive-perl_0.001013.bb
@@ -13,7 +13,7 @@ HOMEPAGE = "https://metacpan.org/pod/Sub-Exporter-Progressive/"
 LICENSE = "Artistic-1.0 | GPL-1.0+"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=003fa970662359a43ac2c2961108b0f1"
 
-DEPENDS_${PN} = " perl-module-test-more"
+DEPENDS += " perl-module-test-more"
 
 SRC_URI = "${CPAN_MIRROR}/authors/id/F/FR/FREW/Sub-Exporter-Progressive-${PV}.tar.gz"
 SRC_URI[md5sum] = "72cf6acdd2a0a8b105821a4db98e4ebe"
-- 
2.22.0

[meta-initramfs][PATCH 2/2] klcc-cross: Recognise --unwindlib clang option

[Khem Raj]

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../klibc/files/klcc-cross-accept-clang-options.patch | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/meta-initramfs/recipes-devtools/klibc/files/klcc-cross-accept-clang-options.patch b/meta-initramfs/recipes-devtools/klibc/files/klcc-cross-accept-clang-options.patch
index 07703aa536..3b7031e981 100644
--- a/meta-initramfs/recipes-devtools/klibc/files/klcc-cross-accept-clang-options.patch
+++ b/meta-initramfs/recipes-devtools/klibc/files/klcc-cross-accept-clang-options.patch
@@ -3,16 +3,17 @@ meta-clang passes this option to compiler defaults
 Upstream-Status: Pending
 Signed-off-by: Khem Raj <raj.khem@gmail.com>
 
-Index: git/klcc/klcc.in
-===================================================================
---- git.orig/klcc/klcc.in
-+++ git/klcc/klcc.in
-@@ -207,6 +207,9 @@ while ( defined($a = shift(@ARGV)) ) {
+--- a/klcc/klcc.in
++++ b/klcc/klcc.in
+@@ -207,6 +207,12 @@ while ( defined($a = shift(@ARGV)) ) {
      } elsif ( $a =~ /^--([sysroot=])(.*)$/ ) {
  	# Override gcc encoded sysroot
  	push(@ccopt, $a);
 +    } elsif ( $a eq '-no-integrated-as' ) {
 +	# Allow clang options
++	push(@ccopt, $a);
++    } elsif ( $a =~ '--unwindlib=.*' ) {
++	# Allow clang options
 +	push(@ccopt, $a);
      } else {
  	die "$0: unknown option: $a\n";
-- 
2.22.0

Re: [meta-perl][PATCH 1/2] libsub-exporter-progressive-perl: Replace DEPENDS_PN with DEPENDS

[Martin Jansa]

Shouldn't this one be rather RDEPENDS_${PN}?

It was building without this build time dependency before and it's not
-native, so I guess it was rather typo in runtime dependency?

On Sat, Aug 3, 2019 at 3:11 AM Khem Raj <raj.khem@gmail.com> wrote:

> Fixes
> libsub-exporter-progressive-perl_0.001013.bb: QA Issue: recipe uses
> DEPENDS_${PN}, should use DEPENDS [pkgvarcheck]
>
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  .../libsub/libsub-exporter-progressive-perl_0.001013.bb         | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-perl/recipes-perl/libsub/
> libsub-exporter-progressive-perl_0.001013.bb
> b/meta-perl/recipes-perl/libsub/
> libsub-exporter-progressive-perl_0.001013.bb
> index 64d19c0f1f..e484f4a7a0 100644
> --- a/meta-perl/recipes-perl/libsub/
> libsub-exporter-progressive-perl_0.001013.bb
> +++ b/meta-perl/recipes-perl/libsub/
> libsub-exporter-progressive-perl_0.001013.bb
> @@ -13,7 +13,7 @@ HOMEPAGE = "
> https://metacpan.org/pod/Sub-Exporter-Progressive/"
>  LICENSE = "Artistic-1.0 | GPL-1.0+"
>  LIC_FILES_CHKSUM = "file://LICENSE;md5=003fa970662359a43ac2c2961108b0f1"
>
> -DEPENDS_${PN} = " perl-module-test-more"
> +DEPENDS += " perl-module-test-more"
>
>  SRC_URI =
> "${CPAN_MIRROR}/authors/id/F/FR/FREW/Sub-Exporter-Progressive-${PV}.tar.gz"
>  SRC_URI[md5sum] = "72cf6acdd2a0a8b105821a4db98e4ebe"
> --
> 2.22.0
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>

Re: [meta-perl][PATCH 1/2] libsub-exporter-progressive-perl: Replace DEPENDS_PN with DEPENDS

[Khem Raj]

Yes, you are right. I already fixed it locally will send v2 soon

On Sat, Aug 3, 2019 at 3:11 AM Martin Jansa <martin.jansa@gmail.com> wrote:

> Shouldn't this one be rather RDEPENDS_${PN}?
>
> It was building without this build time dependency before and it's not
> -native, so I guess it was rather typo in runtime dependency?
>
> On Sat, Aug 3, 2019 at 3:11 AM Khem Raj <raj.khem@gmail.com> wrote:
>
>> Fixes
>> libsub-exporter-progressive-perl_0.001013.bb: QA Issue: recipe uses
>> DEPENDS_${PN}, should use DEPENDS [pkgvarcheck]
>>
>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> ---
>>  .../libsub/libsub-exporter-progressive-perl_0.001013.bb         | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta-perl/recipes-perl/libsub/
>> libsub-exporter-progressive-perl_0.001013.bb
>> b/meta-perl/recipes-perl/libsub/
>> libsub-exporter-progressive-perl_0.001013.bb
>> index 64d19c0f1f..e484f4a7a0 100644
>> --- a/meta-perl/recipes-perl/libsub/
>> libsub-exporter-progressive-perl_0.001013.bb
>> +++ b/meta-perl/recipes-perl/libsub/
>> libsub-exporter-progressive-perl_0.001013.bb
>> @@ -13,7 +13,7 @@ HOMEPAGE = "
>> https://metacpan.org/pod/Sub-Exporter-Progressive/"
>>  LICENSE = "Artistic-1.0 | GPL-1.0+"
>>  LIC_FILES_CHKSUM = "file://LICENSE;md5=003fa970662359a43ac2c2961108b0f1"
>>
>> -DEPENDS_${PN} = " perl-module-test-more"
>> +DEPENDS += " perl-module-test-more"
>>
>>  SRC_URI =
>> "${CPAN_MIRROR}/authors/id/F/FR/FREW/Sub-Exporter-Progressive-${PV}.tar.gz"
>>  SRC_URI[md5sum] = "72cf6acdd2a0a8b105821a4db98e4ebe"
>> --
>> 2.22.0
>>
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>>
>

Re: [meta-perl][PATCH 1/2] libsub-exporter-progressive-perl: Replace DEPENDS_PN with DEPENDS

[Adrian Bunk]

On Sat, Aug 03, 2019 at 12:11:32PM +0200, Martin Jansa wrote:
> Shouldn't this one be rather RDEPENDS_${PN}?
> 
> It was building without this build time dependency before and it's not
> -native, so I guess it was rather typo in runtime dependency?

This also sounds wrong to me.

As expected it uses Test::More only when running tests,
so forcing a runtime dependency is unnecessary.

RDEPENDS_${PN}-ptest would be correct if there was a ptest package.

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed