* [PATCH][resend] perl: fix build failure if building dir has the "blib" string
@ 2015-07-10 8:54 rongqing.li
2015-07-10 9:09 ` Jens Rehsack
2015-07-10 9:39 ` Jussi Kukkonen
0 siblings, 2 replies; 10+ messages in thread
From: rongqing.li @ 2015-07-10 8:54 UTC (permalink / raw)
To: openembedded-core
From: Wenzong Fan <wenzong.fan@windriver.com>
If build dir includes the string "blib", filter it out from @INC may empty
the @INC and cause build errors like:
Can't locate ExtUtils/MakeMaker.pm in @INC \
(you may need to install the ExtUtils::MakeMaker module) \
(@INC contains: .) at Makefile.PL
Signed-off-by: Roy Li <rongqing.li@windriver.com>
Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
---
meta/recipes-devtools/perl/perl-native_5.22.0.bb | 1 +
...-PathTools-don-t-filter-out-blib-from-INC.patch | 33 ++++++++++++++++++++++
meta/recipes-devtools/perl/perl_5.22.0.bb | 1 +
3 files changed, 35 insertions(+)
create mode 100644 meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
diff --git a/meta/recipes-devtools/perl/perl-native_5.22.0.bb b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
index dbcf2a6..a9a1cab 100644
--- a/meta/recipes-devtools/perl/perl-native_5.22.0.bb
+++ b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
@@ -8,6 +8,7 @@ SRC_URI += "\
file://MM_Unix.pm.patch \
file://debian/errno_ver.diff \
file://dynaloaderhack.patch \
+ file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
"
SRC_URI[md5sum] = "ff0f09b17de426eff323426cb140ee79"
diff --git a/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
new file mode 100644
index 0000000..7dd9041
--- /dev/null
+++ b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
@@ -0,0 +1,33 @@
+From 90c252cecc38aed5d5faedb30485dd6eee2e54eb Mon Sep 17 00:00:00 2001
+From: Wenzong Fan <wenzong.fan@windriver.com>
+Date: Wed, 11 Feb 2015 15:14:40 +0800
+Subject: [PATCH] perl / PathTools: don't filter out blib from @INC
+
+If $TOPDIR includes the string "blib", filter it out from @INC may empty
+the @INC and cause build errors like:
+
+ Can't locate ExtUtils/MakeMaker.pm in @INC \
+ (you may need to install the ExtUtils::MakeMaker module) \
+ (@INC contains: .) at Makefile.PL
+
+Upstream-Status: Pending
+
+Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
+---
+ dist/PathTools/Makefile.PL | 3 ---
+ 1 file changed, 3 deletions(-)
+
+diff --git a/dist/PathTools/Makefile.PL b/dist/PathTools/Makefile.PL
+index 1b21de4..f562cb2 100644
+--- a/dist/PathTools/Makefile.PL
++++ b/dist/PathTools/Makefile.PL
+@@ -1,6 +1,3 @@
+-
+-BEGIN { @INC = grep {!/blib/} @INC }
+-
+ require 5.005;
+ use ExtUtils::MakeMaker;
+ WriteMakefile
+--
+1.9.1
+
diff --git a/meta/recipes-devtools/perl/perl_5.22.0.bb b/meta/recipes-devtools/perl/perl_5.22.0.bb
index 050a98c..7f78998 100644
--- a/meta/recipes-devtools/perl/perl_5.22.0.bb
+++ b/meta/recipes-devtools/perl/perl_5.22.0.bb
@@ -61,6 +61,7 @@ SRC_URI += " \
file://t-run-switches.t-perl5-perl.patch \
file://ext-ODBM_File-hints-linux.pl-link-libgdbm_compat.patch \
file://ext-ODBM_File-t-odbm.t-fix-the-path-of-dbmt_common.p.patch \
+ file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
"
# Fix test case issues
--
1.9.1
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH][resend] perl: fix build failure if building dir has the "blib" string
2015-07-10 8:54 [PATCH][resend] perl: fix build failure if building dir has the "blib" string rongqing.li
@ 2015-07-10 9:09 ` Jens Rehsack
2015-07-10 9:28 ` Rongqing Li
2015-07-10 9:39 ` Jussi Kukkonen
1 sibling, 1 reply; 10+ messages in thread
From: Jens Rehsack @ 2015-07-10 9:09 UTC (permalink / raw)
To: rongqing.li; +Cc: openembedded-core
Hi,
When build dir includes contain blib, there is an error which should be solved first.
Can you please give an example where this condition is true?
And maybe refer where you reported it upstream?
Best regards,
Jens
> Am 10.07.2015 um 10:54 schrieb rongqing.li@windriver.com:
>
> From: Wenzong Fan <wenzong.fan@windriver.com>
>
> If build dir includes the string "blib", filter it out from @INC may empty
> the @INC and cause build errors like:
>
> Can't locate ExtUtils/MakeMaker.pm in @INC \
> (you may need to install the ExtUtils::MakeMaker module) \
> (@INC contains: .) at Makefile.PL
>
> Signed-off-by: Roy Li <rongqing.li@windriver.com>
> Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
> ---
> meta/recipes-devtools/perl/perl-native_5.22.0.bb | 1 +
> ...-PathTools-don-t-filter-out-blib-from-INC.patch | 33 ++++++++++++++++++++++
> meta/recipes-devtools/perl/perl_5.22.0.bb | 1 +
> 3 files changed, 35 insertions(+)
> create mode 100644 meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>
> diff --git a/meta/recipes-devtools/perl/perl-native_5.22.0.bb b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
> index dbcf2a6..a9a1cab 100644
> --- a/meta/recipes-devtools/perl/perl-native_5.22.0.bb
> +++ b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
> @@ -8,6 +8,7 @@ SRC_URI += "\
> file://MM_Unix.pm.patch \
> file://debian/errno_ver.diff \
> file://dynaloaderhack.patch \
> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
> "
>
> SRC_URI[md5sum] = "ff0f09b17de426eff323426cb140ee79"
> diff --git a/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
> new file mode 100644
> index 0000000..7dd9041
> --- /dev/null
> +++ b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
> @@ -0,0 +1,33 @@
> +From 90c252cecc38aed5d5faedb30485dd6eee2e54eb Mon Sep 17 00:00:00 2001
> +From: Wenzong Fan <wenzong.fan@windriver.com>
> +Date: Wed, 11 Feb 2015 15:14:40 +0800
> +Subject: [PATCH] perl / PathTools: don't filter out blib from @INC
> +
> +If $TOPDIR includes the string "blib", filter it out from @INC may empty
> +the @INC and cause build errors like:
> +
> + Can't locate ExtUtils/MakeMaker.pm in @INC \
> + (you may need to install the ExtUtils::MakeMaker module) \
> + (@INC contains: .) at Makefile.PL
> +
> +Upstream-Status: Pending
> +
> +Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
> +---
> + dist/PathTools/Makefile.PL | 3 ---
> + 1 file changed, 3 deletions(-)
> +
> +diff --git a/dist/PathTools/Makefile.PL b/dist/PathTools/Makefile.PL
> +index 1b21de4..f562cb2 100644
> +--- a/dist/PathTools/Makefile.PL
> ++++ b/dist/PathTools/Makefile.PL
> +@@ -1,6 +1,3 @@
> +-
> +-BEGIN { @INC = grep {!/blib/} @INC }
> +-
> + require 5.005;
> + use ExtUtils::MakeMaker;
> + WriteMakefile
> +--
> +1.9.1
> +
> diff --git a/meta/recipes-devtools/perl/perl_5.22.0.bb b/meta/recipes-devtools/perl/perl_5.22.0.bb
> index 050a98c..7f78998 100644
> --- a/meta/recipes-devtools/perl/perl_5.22.0.bb
> +++ b/meta/recipes-devtools/perl/perl_5.22.0.bb
> @@ -61,6 +61,7 @@ SRC_URI += " \
> file://t-run-switches.t-perl5-perl.patch \
> file://ext-ODBM_File-hints-linux.pl-link-libgdbm_compat.patch \
> file://ext-ODBM_File-t-odbm.t-fix-the-path-of-dbmt_common.p.patch \
> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
> "
>
> # Fix test case issues
> --
> 1.9.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
--
Jens Rehsack - rehsack@gmail.com
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH][resend] perl: fix build failure if building dir has the "blib" string
2015-07-10 9:09 ` Jens Rehsack
@ 2015-07-10 9:28 ` Rongqing Li
2015-07-10 10:08 ` Jens Rehsack
0 siblings, 1 reply; 10+ messages in thread
From: Rongqing Li @ 2015-07-10 9:28 UTC (permalink / raw)
To: Jens Rehsack; +Cc: openembedded-core
On 2015年07月10日 17:09, Jens Rehsack wrote:
> Hi,
>
> When build dir includes contain blib, there is an error which should be solved first.
>
> Can you please give an example where this condition is true?
> And maybe refer where you reported it upstream?
>
the below commands can trigger this bug; under Poky repo:
#source oe-init-build-env blib;
#bitbake perl
or work on source code:
#mkdir blib
#cd blib
#unxz perl-5.22.0.tar.xz
#tar -xvf perl-5.22.0.tar
#cd perl-5.22.0
#./Configure -des -Dprefix=./
#make
-Roy
> Best regards,
> Jens
>
>> Am 10.07.2015 um 10:54 schrieb rongqing.li@windriver.com:
>>
>> From: Wenzong Fan <wenzong.fan@windriver.com>
>>
>> If build dir includes the string "blib", filter it out from @INC may empty
>> the @INC and cause build errors like:
>>
>> Can't locate ExtUtils/MakeMaker.pm in @INC \
>> (you may need to install the ExtUtils::MakeMaker module) \
>> (@INC contains: .) at Makefile.PL
>>
>> Signed-off-by: Roy Li <rongqing.li@windriver.com>
>> Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
>> ---
>> meta/recipes-devtools/perl/perl-native_5.22.0.bb | 1 +
>> ...-PathTools-don-t-filter-out-blib-from-INC.patch | 33 ++++++++++++++++++++++
>> meta/recipes-devtools/perl/perl_5.22.0.bb | 1 +
>> 3 files changed, 35 insertions(+)
>> create mode 100644 meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>>
>> diff --git a/meta/recipes-devtools/perl/perl-native_5.22.0.bb b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>> index dbcf2a6..a9a1cab 100644
>> --- a/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>> +++ b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>> @@ -8,6 +8,7 @@ SRC_URI += "\
>> file://MM_Unix.pm.patch \
>> file://debian/errno_ver.diff \
>> file://dynaloaderhack.patch \
>> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
>> "
>>
>> SRC_URI[md5sum] = "ff0f09b17de426eff323426cb140ee79"
>> diff --git a/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>> new file mode 100644
>> index 0000000..7dd9041
>> --- /dev/null
>> +++ b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>> @@ -0,0 +1,33 @@
>> +From 90c252cecc38aed5d5faedb30485dd6eee2e54eb Mon Sep 17 00:00:00 2001
>> +From: Wenzong Fan <wenzong.fan@windriver.com>
>> +Date: Wed, 11 Feb 2015 15:14:40 +0800
>> +Subject: [PATCH] perl / PathTools: don't filter out blib from @INC
>> +
>> +If $TOPDIR includes the string "blib", filter it out from @INC may empty
>> +the @INC and cause build errors like:
>> +
>> + Can't locate ExtUtils/MakeMaker.pm in @INC \
>> + (you may need to install the ExtUtils::MakeMaker module) \
>> + (@INC contains: .) at Makefile.PL
>> +
>> +Upstream-Status: Pending
>> +
>> +Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
>> +---
>> + dist/PathTools/Makefile.PL | 3 ---
>> + 1 file changed, 3 deletions(-)
>> +
>> +diff --git a/dist/PathTools/Makefile.PL b/dist/PathTools/Makefile.PL
>> +index 1b21de4..f562cb2 100644
>> +--- a/dist/PathTools/Makefile.PL
>> ++++ b/dist/PathTools/Makefile.PL
>> +@@ -1,6 +1,3 @@
>> +-
>> +-BEGIN { @INC = grep {!/blib/} @INC }
>> +-
>> + require 5.005;
>> + use ExtUtils::MakeMaker;
>> + WriteMakefile
>> +--
>> +1.9.1
>> +
>> diff --git a/meta/recipes-devtools/perl/perl_5.22.0.bb b/meta/recipes-devtools/perl/perl_5.22.0.bb
>> index 050a98c..7f78998 100644
>> --- a/meta/recipes-devtools/perl/perl_5.22.0.bb
>> +++ b/meta/recipes-devtools/perl/perl_5.22.0.bb
>> @@ -61,6 +61,7 @@ SRC_URI += " \
>> file://t-run-switches.t-perl5-perl.patch \
>> file://ext-ODBM_File-hints-linux.pl-link-libgdbm_compat.patch \
>> file://ext-ODBM_File-t-odbm.t-fix-the-path-of-dbmt_common.p.patch \
>> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
>> "
>>
>> # Fix test case issues
>> --
>> 1.9.1
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
--
Best Reagrds,
Roy | RongQing Li
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH][resend] perl: fix build failure if building dir has the "blib" string
2015-07-10 9:28 ` Rongqing Li
@ 2015-07-10 10:08 ` Jens Rehsack
2015-07-13 2:11 ` Rongqing Li
0 siblings, 1 reply; 10+ messages in thread
From: Jens Rehsack @ 2015-07-10 10:08 UTC (permalink / raw)
To: Rongqing Li; +Cc: openembedded-core
> Am 10.07.2015 um 11:28 schrieb Rongqing Li <rongqing.li@windriver.com>:
>
>
>
> On 2015年07月10日 17:09, Jens Rehsack wrote:
>> Hi,
>>
>> When build dir includes contain blib, there is an error which should be solved first.
>>
>> Can you please give an example where this condition is true?
>> And maybe refer where you reported it upstream?
>>
>
> the below commands can trigger this bug; under Poky repo:
>
> #source oe-init-build-env blib;
> #bitbake perl
Simply add a comment that blib as build-env name is not supported as long as no
comment from PathTools maintainers agree on the patch. I rate your patch as dangerous
for common perl module building...
I agree that PathTools should handle @INC parts containing blib somewhere, but
your patch results in any build environment of any MakeMaker base module might be
broken.
If you tell me the RT ticket number where you reported the issue upstream and
send your patch for review, I will join and argue for fixing the issue (even when
I think the patch needs review).
> or work on source code:
>
> #mkdir blib
> #cd blib
> #unxz perl-5.22.0.tar.xz
> #tar -xvf perl-5.22.0.tar
> #cd perl-5.22.0
> #./Configure -des -Dprefix=./
> #make
Don't do that - neither for PathTools nor for any other perl module.
Cheers,
Jens
> -Roy
>
>
>> Best regards,
>> Jens
>>
>>> Am 10.07.2015 um 10:54 schrieb rongqing.li@windriver.com:
>>>
>>> From: Wenzong Fan <wenzong.fan@windriver.com>
>>>
>>> If build dir includes the string "blib", filter it out from @INC may empty
>>> the @INC and cause build errors like:
>>>
>>> Can't locate ExtUtils/MakeMaker.pm in @INC \
>>> (you may need to install the ExtUtils::MakeMaker module) \
>>> (@INC contains: .) at Makefile.PL
>>>
>>> Signed-off-by: Roy Li <rongqing.li@windriver.com>
>>> Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
>>> ---
>>> meta/recipes-devtools/perl/perl-native_5.22.0.bb | 1 +
>>> ...-PathTools-don-t-filter-out-blib-from-INC.patch | 33 ++++++++++++++++++++++
>>> meta/recipes-devtools/perl/perl_5.22.0.bb | 1 +
>>> 3 files changed, 35 insertions(+)
>>> create mode 100644 meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>>>
>>> diff --git a/meta/recipes-devtools/perl/perl-native_5.22.0.bb b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>>> index dbcf2a6..a9a1cab 100644
>>> --- a/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>>> +++ b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>>> @@ -8,6 +8,7 @@ SRC_URI += "\
>>> file://MM_Unix.pm.patch \
>>> file://debian/errno_ver.diff \
>>> file://dynaloaderhack.patch \
>>> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
>>> "
>>>
>>> SRC_URI[md5sum] = "ff0f09b17de426eff323426cb140ee79"
>>> diff --git a/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>>> new file mode 100644
>>> index 0000000..7dd9041
>>> --- /dev/null
>>> +++ b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>>> @@ -0,0 +1,33 @@
>>> +From 90c252cecc38aed5d5faedb30485dd6eee2e54eb Mon Sep 17 00:00:00 2001
>>> +From: Wenzong Fan <wenzong.fan@windriver.com>
>>> +Date: Wed, 11 Feb 2015 15:14:40 +0800
>>> +Subject: [PATCH] perl / PathTools: don't filter out blib from @INC
>>> +
>>> +If $TOPDIR includes the string "blib", filter it out from @INC may empty
>>> +the @INC and cause build errors like:
>>> +
>>> + Can't locate ExtUtils/MakeMaker.pm in @INC \
>>> + (you may need to install the ExtUtils::MakeMaker module) \
>>> + (@INC contains: .) at Makefile.PL
>>> +
>>> +Upstream-Status: Pending
>>> +
>>> +Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
>>> +---
>>> + dist/PathTools/Makefile.PL | 3 ---
>>> + 1 file changed, 3 deletions(-)
>>> +
>>> +diff --git a/dist/PathTools/Makefile.PL b/dist/PathTools/Makefile.PL
>>> +index 1b21de4..f562cb2 100644
>>> +--- a/dist/PathTools/Makefile.PL
>>> ++++ b/dist/PathTools/Makefile.PL
>>> +@@ -1,6 +1,3 @@
>>> +-
>>> +-BEGIN { @INC = grep {!/blib/} @INC }
>>> +-
>>> + require 5.005;
>>> + use ExtUtils::MakeMaker;
>>> + WriteMakefile
>>> +--
>>> +1.9.1
>>> +
>>> diff --git a/meta/recipes-devtools/perl/perl_5.22.0.bb b/meta/recipes-devtools/perl/perl_5.22.0.bb
>>> index 050a98c..7f78998 100644
>>> --- a/meta/recipes-devtools/perl/perl_5.22.0.bb
>>> +++ b/meta/recipes-devtools/perl/perl_5.22.0.bb
>>> @@ -61,6 +61,7 @@ SRC_URI += " \
>>> file://t-run-switches.t-perl5-perl.patch \
>>> file://ext-ODBM_File-hints-linux.pl-link-libgdbm_compat.patch \
>>> file://ext-ODBM_File-t-odbm.t-fix-the-path-of-dbmt_common.p.patch \
>>> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
>>> "
>>>
>>> # Fix test case issues
>>> --
>>> 1.9.1
>>>
>>> --
>>> _______________________________________________
>>> Openembedded-core mailing list
>>> Openembedded-core@lists.openembedded.org
>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>
> --
> Best Reagrds,
> Roy | RongQing Li
--
Jens Rehsack - rehsack@gmail.com
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH][resend] perl: fix build failure if building dir has the "blib" string
2015-07-10 10:08 ` Jens Rehsack
@ 2015-07-13 2:11 ` Rongqing Li
0 siblings, 0 replies; 10+ messages in thread
From: Rongqing Li @ 2015-07-13 2:11 UTC (permalink / raw)
To: Jens Rehsack; +Cc: openembedded-core
On 2015年07月10日 18:08, Jens Rehsack wrote:
>
>> Am 10.07.2015 um 11:28 schrieb Rongqing Li <rongqing.li@windriver.com>:
>>
>>
>>
>> On 2015年07月10日 17:09, Jens Rehsack wrote:
>>> Hi,
>>>
>>> When build dir includes contain blib, there is an error which should be solved first.
>>>
>>> Can you please give an example where this condition is true?
>>> And maybe refer where you reported it upstream?
>>>
>>
>> the below commands can trigger this bug; under Poky repo:
>>
>> #source oe-init-build-env blib;
>> #bitbake perl
>
> Simply add a comment that blib as build-env name is not supported as long as no
> comment from PathTools maintainers agree on the patch. I rate your patch as dangerous
> for common perl module building...
>
> I agree that PathTools should handle @INC parts containing blib somewhere, but
> your patch results in any build environment of any MakeMaker base module might be
> broken.
>
> If you tell me the RT ticket number where you reported the issue upstream and
> send your patch for review, I will join and argue for fixing the issue (even when
> I think the patch needs review).
open a defect for perl
https://rt.perl.org/Ticket/Display.html?id=125603
thanks
-Roy
>
>> or work on source code:
>>
>> #mkdir blib
>> #cd blib
>> #unxz perl-5.22.0.tar.xz
>> #tar -xvf perl-5.22.0.tar
>> #cd perl-5.22.0
>> #./Configure -des -Dprefix=./
>> #make
>
> Don't do that - neither for PathTools nor for any other perl module.
>
> Cheers,
> Jens
>
>> -Roy
>>
>>
>>> Best regards,
>>> Jens
>>>
>>>> Am 10.07.2015 um 10:54 schrieb rongqing.li@windriver.com:
>>>>
>>>> From: Wenzong Fan <wenzong.fan@windriver.com>
>>>>
>>>> If build dir includes the string "blib", filter it out from @INC may empty
>>>> the @INC and cause build errors like:
>>>>
>>>> Can't locate ExtUtils/MakeMaker.pm in @INC \
>>>> (you may need to install the ExtUtils::MakeMaker module) \
>>>> (@INC contains: .) at Makefile.PL
>>>>
>>>> Signed-off-by: Roy Li <rongqing.li@windriver.com>
>>>> Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
>>>> ---
>>>> meta/recipes-devtools/perl/perl-native_5.22.0.bb | 1 +
>>>> ...-PathTools-don-t-filter-out-blib-from-INC.patch | 33 ++++++++++++++++++++++
>>>> meta/recipes-devtools/perl/perl_5.22.0.bb | 1 +
>>>> 3 files changed, 35 insertions(+)
>>>> create mode 100644 meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>>>>
>>>> diff --git a/meta/recipes-devtools/perl/perl-native_5.22.0.bb b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>>>> index dbcf2a6..a9a1cab 100644
>>>> --- a/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>>>> +++ b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>>>> @@ -8,6 +8,7 @@ SRC_URI += "\
>>>> file://MM_Unix.pm.patch \
>>>> file://debian/errno_ver.diff \
>>>> file://dynaloaderhack.patch \
>>>> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
>>>> "
>>>>
>>>> SRC_URI[md5sum] = "ff0f09b17de426eff323426cb140ee79"
>>>> diff --git a/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>>>> new file mode 100644
>>>> index 0000000..7dd9041
>>>> --- /dev/null
>>>> +++ b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>>>> @@ -0,0 +1,33 @@
>>>> +From 90c252cecc38aed5d5faedb30485dd6eee2e54eb Mon Sep 17 00:00:00 2001
>>>> +From: Wenzong Fan <wenzong.fan@windriver.com>
>>>> +Date: Wed, 11 Feb 2015 15:14:40 +0800
>>>> +Subject: [PATCH] perl / PathTools: don't filter out blib from @INC
>>>> +
>>>> +If $TOPDIR includes the string "blib", filter it out from @INC may empty
>>>> +the @INC and cause build errors like:
>>>> +
>>>> + Can't locate ExtUtils/MakeMaker.pm in @INC \
>>>> + (you may need to install the ExtUtils::MakeMaker module) \
>>>> + (@INC contains: .) at Makefile.PL
>>>> +
>>>> +Upstream-Status: Pending
>>>> +
>>>> +Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
>>>> +---
>>>> + dist/PathTools/Makefile.PL | 3 ---
>>>> + 1 file changed, 3 deletions(-)
>>>> +
>>>> +diff --git a/dist/PathTools/Makefile.PL b/dist/PathTools/Makefile.PL
>>>> +index 1b21de4..f562cb2 100644
>>>> +--- a/dist/PathTools/Makefile.PL
>>>> ++++ b/dist/PathTools/Makefile.PL
>>>> +@@ -1,6 +1,3 @@
>>>> +-
>>>> +-BEGIN { @INC = grep {!/blib/} @INC }
>>>> +-
>>>> + require 5.005;
>>>> + use ExtUtils::MakeMaker;
>>>> + WriteMakefile
>>>> +--
>>>> +1.9.1
>>>> +
>>>> diff --git a/meta/recipes-devtools/perl/perl_5.22.0.bb b/meta/recipes-devtools/perl/perl_5.22.0.bb
>>>> index 050a98c..7f78998 100644
>>>> --- a/meta/recipes-devtools/perl/perl_5.22.0.bb
>>>> +++ b/meta/recipes-devtools/perl/perl_5.22.0.bb
>>>> @@ -61,6 +61,7 @@ SRC_URI += " \
>>>> file://t-run-switches.t-perl5-perl.patch \
>>>> file://ext-ODBM_File-hints-linux.pl-link-libgdbm_compat.patch \
>>>> file://ext-ODBM_File-t-odbm.t-fix-the-path-of-dbmt_common.p.patch \
>>>> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
>>>> "
>>>>
>>>> # Fix test case issues
>>>> --
>>>> 1.9.1
>>>>
>>>> --
>>>> _______________________________________________
>>>> Openembedded-core mailing list
>>>> Openembedded-core@lists.openembedded.org
>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>>
>>
>> --
>> Best Reagrds,
>> Roy | RongQing Li
>
--
Best Reagrds,
Roy | RongQing Li
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH][resend] perl: fix build failure if building dir has the "blib" string
2015-07-10 8:54 [PATCH][resend] perl: fix build failure if building dir has the "blib" string rongqing.li
2015-07-10 9:09 ` Jens Rehsack
@ 2015-07-10 9:39 ` Jussi Kukkonen
2015-07-27 1:01 ` Rongqing Li
1 sibling, 1 reply; 10+ messages in thread
From: Jussi Kukkonen @ 2015-07-10 9:39 UTC (permalink / raw)
To: rongqing.li; +Cc: Patches and discussions about the oe-core layer
On 10 July 2015 at 11:54, <rongqing.li@windriver.com> wrote:
> From: Wenzong Fan <wenzong.fan@windriver.com>
>
> If build dir includes the string "blib", filter it out from @INC may empty
> the @INC and cause build errors like:
>
> Can't locate ExtUtils/MakeMaker.pm in @INC \
> (you may need to install the ExtUtils::MakeMaker module) \
> (@INC contains: .) at Makefile.PL
The grep certainly looks overly zealous, but surely upstream has a
reason for having this code... If you're going to remove the code
altogether, you should mention why those original reasons do not apply
here.
Jussi
> Signed-off-by: Roy Li <rongqing.li@windriver.com>
> Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
> ---
> meta/recipes-devtools/perl/perl-native_5.22.0.bb | 1 +
> ...-PathTools-don-t-filter-out-blib-from-INC.patch | 33 ++++++++++++++++++++++
> meta/recipes-devtools/perl/perl_5.22.0.bb | 1 +
> 3 files changed, 35 insertions(+)
> create mode 100644 meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>
> diff --git a/meta/recipes-devtools/perl/perl-native_5.22.0.bb b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
> index dbcf2a6..a9a1cab 100644
> --- a/meta/recipes-devtools/perl/perl-native_5.22.0.bb
> +++ b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
> @@ -8,6 +8,7 @@ SRC_URI += "\
> file://MM_Unix.pm.patch \
> file://debian/errno_ver.diff \
> file://dynaloaderhack.patch \
> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
> "
>
> SRC_URI[md5sum] = "ff0f09b17de426eff323426cb140ee79"
> diff --git a/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
> new file mode 100644
> index 0000000..7dd9041
> --- /dev/null
> +++ b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
> @@ -0,0 +1,33 @@
> +From 90c252cecc38aed5d5faedb30485dd6eee2e54eb Mon Sep 17 00:00:00 2001
> +From: Wenzong Fan <wenzong.fan@windriver.com>
> +Date: Wed, 11 Feb 2015 15:14:40 +0800
> +Subject: [PATCH] perl / PathTools: don't filter out blib from @INC
> +
> +If $TOPDIR includes the string "blib", filter it out from @INC may empty
> +the @INC and cause build errors like:
> +
> + Can't locate ExtUtils/MakeMaker.pm in @INC \
> + (you may need to install the ExtUtils::MakeMaker module) \
> + (@INC contains: .) at Makefile.PL
> +
> +Upstream-Status: Pending
> +
> +Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
> +---
> + dist/PathTools/Makefile.PL | 3 ---
> + 1 file changed, 3 deletions(-)
> +
> +diff --git a/dist/PathTools/Makefile.PL b/dist/PathTools/Makefile.PL
> +index 1b21de4..f562cb2 100644
> +--- a/dist/PathTools/Makefile.PL
> ++++ b/dist/PathTools/Makefile.PL
> +@@ -1,6 +1,3 @@
> +-
> +-BEGIN { @INC = grep {!/blib/} @INC }
> +-
> + require 5.005;
> + use ExtUtils::MakeMaker;
> + WriteMakefile
> +--
> +1.9.1
> +
> diff --git a/meta/recipes-devtools/perl/perl_5.22.0.bb b/meta/recipes-devtools/perl/perl_5.22.0.bb
> index 050a98c..7f78998 100644
> --- a/meta/recipes-devtools/perl/perl_5.22.0.bb
> +++ b/meta/recipes-devtools/perl/perl_5.22.0.bb
> @@ -61,6 +61,7 @@ SRC_URI += " \
> file://t-run-switches.t-perl5-perl.patch \
> file://ext-ODBM_File-hints-linux.pl-link-libgdbm_compat.patch \
> file://ext-ODBM_File-t-odbm.t-fix-the-path-of-dbmt_common.p.patch \
> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
> "
>
> # Fix test case issues
> --
> 1.9.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH][resend] perl: fix build failure if building dir has the "blib" string
2015-07-10 9:39 ` Jussi Kukkonen
@ 2015-07-27 1:01 ` Rongqing Li
2015-07-28 12:53 ` Burton, Ross
0 siblings, 1 reply; 10+ messages in thread
From: Rongqing Li @ 2015-07-27 1:01 UTC (permalink / raw)
To: Jussi Kukkonen; +Cc: Patches and discussions about the oe-core layer
On 2015年07月10日 17:39, Jussi Kukkonen wrote:
> On 10 July 2015 at 11:54, <rongqing.li@windriver.com> wrote:
>> From: Wenzong Fan <wenzong.fan@windriver.com>
>>
>> If build dir includes the string "blib", filter it out from @INC may empty
>> the @INC and cause build errors like:
>>
>> Can't locate ExtUtils/MakeMaker.pm in @INC \
>> (you may need to install the ExtUtils::MakeMaker module) \
>> (@INC contains: .) at Makefile.PL
>
> The grep certainly looks overly zealous, but surely upstream has a
> reason for having this code... If you're going to remove the code
> altogether, you should mention why those original reasons do not apply
> here.
>
> Jussi
I think we can merge this patch.
I open a defect and submit this patch, as a result, noone knows
what this line can fix.
https://rt.perl.org/Public/Bug/Display.html?id=125603
-Roy
>
>> Signed-off-by: Roy Li <rongqing.li@windriver.com>
>> Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
>> ---
>> meta/recipes-devtools/perl/perl-native_5.22.0.bb | 1 +
>> ...-PathTools-don-t-filter-out-blib-from-INC.patch | 33 ++++++++++++++++++++++
>> meta/recipes-devtools/perl/perl_5.22.0.bb | 1 +
>> 3 files changed, 35 insertions(+)
>> create mode 100644 meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>>
>> diff --git a/meta/recipes-devtools/perl/perl-native_5.22.0.bb b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>> index dbcf2a6..a9a1cab 100644
>> --- a/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>> +++ b/meta/recipes-devtools/perl/perl-native_5.22.0.bb
>> @@ -8,6 +8,7 @@ SRC_URI += "\
>> file://MM_Unix.pm.patch \
>> file://debian/errno_ver.diff \
>> file://dynaloaderhack.patch \
>> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
>> "
>>
>> SRC_URI[md5sum] = "ff0f09b17de426eff323426cb140ee79"
>> diff --git a/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>> new file mode 100644
>> index 0000000..7dd9041
>> --- /dev/null
>> +++ b/meta/recipes-devtools/perl/perl/perl-PathTools-don-t-filter-out-blib-from-INC.patch
>> @@ -0,0 +1,33 @@
>> +From 90c252cecc38aed5d5faedb30485dd6eee2e54eb Mon Sep 17 00:00:00 2001
>> +From: Wenzong Fan <wenzong.fan@windriver.com>
>> +Date: Wed, 11 Feb 2015 15:14:40 +0800
>> +Subject: [PATCH] perl / PathTools: don't filter out blib from @INC
>> +
>> +If $TOPDIR includes the string "blib", filter it out from @INC may empty
>> +the @INC and cause build errors like:
>> +
>> + Can't locate ExtUtils/MakeMaker.pm in @INC \
>> + (you may need to install the ExtUtils::MakeMaker module) \
>> + (@INC contains: .) at Makefile.PL
>> +
>> +Upstream-Status: Pending
>> +
>> +Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
>> +---
>> + dist/PathTools/Makefile.PL | 3 ---
>> + 1 file changed, 3 deletions(-)
>> +
>> +diff --git a/dist/PathTools/Makefile.PL b/dist/PathTools/Makefile.PL
>> +index 1b21de4..f562cb2 100644
>> +--- a/dist/PathTools/Makefile.PL
>> ++++ b/dist/PathTools/Makefile.PL
>> +@@ -1,6 +1,3 @@
>> +-
>> +-BEGIN { @INC = grep {!/blib/} @INC }
>> +-
>> + require 5.005;
>> + use ExtUtils::MakeMaker;
>> + WriteMakefile
>> +--
>> +1.9.1
>> +
>> diff --git a/meta/recipes-devtools/perl/perl_5.22.0.bb b/meta/recipes-devtools/perl/perl_5.22.0.bb
>> index 050a98c..7f78998 100644
>> --- a/meta/recipes-devtools/perl/perl_5.22.0.bb
>> +++ b/meta/recipes-devtools/perl/perl_5.22.0.bb
>> @@ -61,6 +61,7 @@ SRC_URI += " \
>> file://t-run-switches.t-perl5-perl.patch \
>> file://ext-ODBM_File-hints-linux.pl-link-libgdbm_compat.patch \
>> file://ext-ODBM_File-t-odbm.t-fix-the-path-of-dbmt_common.p.patch \
>> + file://perl-PathTools-don-t-filter-out-blib-from-INC.patch \
>> "
>>
>> # Fix test case issues
>> --
>> 1.9.1
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
--
Best Reagrds,
Roy | RongQing Li
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH][resend] perl: fix build failure if building dir has the "blib" string
2015-07-27 1:01 ` Rongqing Li
@ 2015-07-28 12:53 ` Burton, Ross
2015-07-29 3:08 ` Rongqing Li
0 siblings, 1 reply; 10+ messages in thread
From: Burton, Ross @ 2015-07-28 12:53 UTC (permalink / raw)
To: Rongqing Li; +Cc: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 571 bytes --]
On 27 July 2015 at 02:01, Rongqing Li <rongqing.li@windriver.com> wrote:
> I think we can merge this patch.
>
> I open a defect and submit this patch, as a result, noone knows
> what this line can fix.
>
>
> https://rt.perl.org/Public/Bug/Display.html?id=125603
>
Reading the bug it seems a more upstream-friendly fix (i.e. one that they
would also accept and have in fact asked for) would be to change the regex
to /\bblib\b/. This solves the problem of building in paths that contain
the substring "blib" *and* retains the intent of the logic.
Ross
[-- Attachment #2: Type: text/html, Size: 1102 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH][resend] perl: fix build failure if building dir has the "blib" string
2015-07-28 12:53 ` Burton, Ross
@ 2015-07-29 3:08 ` Rongqing Li
2015-07-29 13:25 ` Burton, Ross
0 siblings, 1 reply; 10+ messages in thread
From: Rongqing Li @ 2015-07-29 3:08 UTC (permalink / raw)
To: Burton, Ross; +Cc: Patches and discussions about the oe-core layer
On 2015年07月28日 20:53, Burton, Ross wrote:
>
> On 27 July 2015 at 02:01, Rongqing Li <rongqing.li@windriver.com
> <mailto:rongqing.li@windriver.com>> wrote:
>
> I think we can merge this patch.
>
> I open a defect and submit this patch, as a result, noone knows
> what this line can fix.
>
>
> https://rt.perl.org/Public/Bug/Display.html?id=125603
>
>
> Reading the bug it seems a more upstream-friendly fix (i.e. one that
> they would also accept and have in fact asked for) would be to change
> the regex to /\bblib\b/. This solves the problem of building in paths
> that contain the substring "blib" *and* retains the intent of the logic.
>
it can not fix all issues to replace /blib/ with /\bblib\b/.
/\bblib\b/ will make the building path "/tmp/xxxxblibyy/" work.
but if the building path has sub-dir blib, it does not work,
like /tmp/blib/
since \b is anchor in perl
I have update the perl bug report
https://rt.perl.org/Public/Bug/Display.html?id=125603
-R
> Ross
--
Best Reagrds,
Roy | RongQing Li
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2015-07-29 13:26 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-10 8:54 [PATCH][resend] perl: fix build failure if building dir has the "blib" string rongqing.li
2015-07-10 9:09 ` Jens Rehsack
2015-07-10 9:28 ` Rongqing Li
2015-07-10 10:08 ` Jens Rehsack
2015-07-13 2:11 ` Rongqing Li
2015-07-10 9:39 ` Jussi Kukkonen
2015-07-27 1:01 ` Rongqing Li
2015-07-28 12:53 ` Burton, Ross
2015-07-29 3:08 ` Rongqing Li
2015-07-29 13:25 ` Burton, Ross
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.