Kang Kai : perl: update dependency creating script

oe-commits Martin Jansa 2 participants 4 messages Jan 7, 2013 – Jan 14, 2013 1 message not in the archive

lore

Share & tools

lore

Martin Jansa

author

2013-01-07 12:45 UTC #1 lore raw

In reply to a message not in the archive

Quoted 21 lines · On Mon, Jan 07, 2013 at 11:36:03AM +0000, [email protected] wrote:

Module: openembedded-core.git
Branch: master
Commit: 35291c97edc05756788d206457c56dcc41edb637
URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=35291c97edc05756788d206457c56dcc41edb637
Author: Kang Kai <[email protected]>
Date:   Mon Nov  5 10:42:00 2012 +0800
perl: update dependency creating script
Base on dependency create script provided by Saul, I update it with:
1 remove document strings in perl module files that may import fake
dependencies.
2 not all the 'use' or 'require' clauses start at the begin of line, and
some of them don't end with ';', such as:
use vars qw(
            $AUTOLOAD
	    ...

Quoted 6 lines

3 filter some strings that they are not real dependencies: some of them
are wrong result of script, some are conditional use/require, and some are cpan extra modules that we don't need.
Signed-off-by Kang Kai <[email protected]>
Signed-off-by Saul Wold <[email protected]>

This breaks parsing when
http://patchwork.openembedded.org/patch/40997/
is used.

Parsing recipes...ERROR: There is a confusing multiline, partially commented expression on line 10 of file /OE/shr-core/openembedded-core/meta/recipes-devtools/perl/perl-rdepends_5.14.2.inc (| sed "s/\/.*\.pm: */ += /g;s/[\"\']//g;s/;.*/\"/g;s/+= .*$require\|use$\> */+= \"perl-module-/g;s/CPANPLUS::.*/cpanplus/g;s/CPAN::.*/cpan/g;s/::/-/g;s/ [^+\"].*//g;s/_/-/g;s/\.pl\"$/\"/;s/\"\?\$/\"/;s/(//;" | tr [:upper:] [:lower:] \). Please clarify whether this is all a comment or should be parsed.
ERROR: Unable to parse /OE/shr-core/openembedded-core/meta/recipes-devtools/perl/perl_5.14.2.bb: Exited with "1"
ERROR: Command execution failed: Exited with 1

Quoted 12 lines

---
 .../recipes-devtools/perl/perl-rdepends_5.14.2.inc | 2336 +++++++++++++++++++-
 1 files changed, 2287 insertions(+), 49 deletions(-)
Diff: http://git.openembedded.org/?p=openembedded-core.git/?a=commitdiff;h=35291c97edc05756788d206457c56dcc41edb637
_______________________________________________
Openembedded-commits mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-commits

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Kang Kai

2013-01-14 03:17 UTC #2 lore raw

Quoted 28 lines · Martin Jansa

Quoted 21 lines · On Mon, Jan 07, 2013 at 11:36:03AM +0000, [email protected] wrote:
Module: openembedded-core.git
Branch: master
Commit: 35291c97edc05756788d206457c56dcc41edb637
URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=35291c97edc05756788d206457c56dcc41edb637
Author: Kang Kai<[email protected]>
Date:   Mon Nov  5 10:42:00 2012 +0800
perl: update dependency creating script
Base on dependency create script provided by Saul, I update it with:
1 remove document strings in perl module files that may import fake
dependencies.
2 not all the 'use' or 'require' clauses start at the begin of line, and
some of them don't end with ';', such as:
use vars qw(
             $AUTOLOAD
	    ...
Quoted 6 lines
3 filter some strings that they are not real dependencies: some of them
are wrong result of script, some are conditional use/require, and some are cpan extra modules that we don't need.
Signed-off-by Kang Kai<[email protected]>
Signed-off-by Saul Wold<[email protected]>

Quoted 8 lines

This breaks parsing when
http://patchwork.openembedded.org/patch/40997/
is used.
Parsing recipes...ERROR: There is a confusing multiline, partially commented expression on line 10 of file /OE/shr-core/openembedded-core/meta/recipes-devtools/perl/perl-rdepends_5.14.2.inc (| sed "s/\/.*\.pm: */ += /g;s/[\"\']//g;s/;.*/\"/g;s/+= .*$require\|use$\> */+= \"perl-module-/g;s/CPANPLUS::.*/cpanplus/g;s/CPAN::.*/cpan/g;s/::/-/g;s/ [^+\"].*//g;s/_/-/g;s/\.pl\"$/\"/;s/\"\?\$/\"/;s/(//;" | tr [:upper:] [:lower:] \). Please clarify whether this is all a comment or should be parsed.
ERROR: Unable to parse /OE/shr-core/openembedded-core/meta/recipes-devtools/perl/perl_5.14.2.bb: Exited with "1"
ERROR: Command execution failed: Exited with 1

Hi Jansa,

Because my boy was born, I have been leaving office and missed this message. Thanks for you fix.

Regards,
Kai

Quoted 12 lines

Quoted 12 lines
---
  .../recipes-devtools/perl/perl-rdepends_5.14.2.inc | 2336 +++++++++++++++++++-
  1 files changed, 2287 insertions(+), 49 deletions(-)
Diff: http://git.openembedded.org/?p=openembedded-core.git/?a=commitdiff;h=35291c97edc05756788d206457c56dcc41edb637
_______________________________________________
Openembedded-commits mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-commits

Martin Jansa

author

2013-01-14 08:30 UTC #3 lore raw

Quoted 33 lines · Kang Kai

Quoted 32 lines · Martin Jansa
Quoted 21 lines · On Mon, Jan 07, 2013 at 11:36:03AM +0000, [email protected] wrote:
Module: openembedded-core.git
Branch: master
Commit: 35291c97edc05756788d206457c56dcc41edb637
URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=35291c97edc05756788d206457c56dcc41edb637
Author: Kang Kai<[email protected]>
Date:   Mon Nov  5 10:42:00 2012 +0800
perl: update dependency creating script
Base on dependency create script provided by Saul, I update it with:
1 remove document strings in perl module files that may import fake
dependencies.
2 not all the 'use' or 'require' clauses start at the begin of line, and
some of them don't end with ';', such as:
use vars qw(
             $AUTOLOAD
	    ...
Quoted 6 lines
3 filter some strings that they are not real dependencies: some of them
are wrong result of script, some are conditional use/require, and some are cpan extra modules that we don't need.
Signed-off-by Kang Kai<[email protected]>
Signed-off-by Saul Wold<[email protected]>
This breaks parsing when
http://patchwork.openembedded.org/patch/40997/
is used.

Quoted 8 lines

Quoted 4 lines
Parsing recipes...ERROR: There is a confusing multiline, partially commented expression on line 10 of file /OE/shr-core/openembedded-core/meta/recipes-devtools/perl/perl-rdepends_5.14.2.inc (| sed "s/\/.*\.pm: */ += /g;s/[\"\']//g;s/;.*/\"/g;s/+= .*$require\|use$\> */+= \"perl-module-/g;s/CPANPLUS::.*/cpanplus/g;s/CPAN::.*/cpan/g;s/::/-/g;s/ [^+\"].*//g;s/_/-/g;s/\.pl\"$/\"/;s/\"\?\$/\"/;s/(//;" | tr [:upper:] [:lower:] \). Please clarify whether this is all a comment or should be parsed.
ERROR: Unable to parse /OE/shr-core/openembedded-core/meta/recipes-devtools/perl/perl_5.14.2.bb: Exited with "1"
ERROR: Command execution failed: Exited with 1
Hi Jansa,
Because my boy was born, I have been leaving office and missed this message.

np, congratz!

Cheers,

Quoted 13 lines

Quoted 12 lines
Quoted 12 lines
---
  .../recipes-devtools/perl/perl-rdepends_5.14.2.inc | 2336 +++++++++++++++++++-
  1 files changed, 2287 insertions(+), 49 deletions(-)
Diff: http://git.openembedded.org/?p=openembedded-core.git/?a=commitdiff;h=35291c97edc05756788d206457c56dcc41edb637
_______________________________________________
Openembedded-commits mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-commits

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Kang Kai

2013-01-14 09:53 UTC #4 lore raw

Quoted 34 lines · Martin Jansa

Quoted 33 lines · Kang Kai
Quoted 32 lines · Martin Jansa
Quoted 21 lines · On Mon, Jan 07, 2013 at 11:36:03AM +0000, [email protected] wrote:
Module: openembedded-core.git
Branch: master
Commit: 35291c97edc05756788d206457c56dcc41edb637
URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=35291c97edc05756788d206457c56dcc41edb637
Author: Kang Kai<[email protected]>
Date:   Mon Nov  5 10:42:00 2012 +0800
perl: update dependency creating script
Base on dependency create script provided by Saul, I update it with:
1 remove document strings in perl module files that may import fake
dependencies.
2 not all the 'use' or 'require' clauses start at the begin of line, and
some of them don't end with ';', such as:
use vars qw(
              $AUTOLOAD
	    ...
Quoted 6 lines
3 filter some strings that they are not real dependencies: some of them
are wrong result of script, some are conditional use/require, and some are cpan extra modules that we don't need.
Signed-off-by Kang Kai<[email protected]>
Signed-off-by Saul Wold<[email protected]>
This breaks parsing when
http://patchwork.openembedded.org/patch/40997/
is used.

Quoted 8 lines

Quoted 7 lines
Quoted 4 lines
Parsing recipes...ERROR: There is a confusing multiline, partially commented expression on line 10 of file /OE/shr-core/openembedded-core/meta/recipes-devtools/perl/perl-rdepends_5.14.2.inc (| sed "s/\/.*\.pm: */ += /g;s/[\"\']//g;s/;.*/\"/g;s/+= .*$require\|use$\> */+= \"perl-module-/g;s/CPANPLUS::.*/cpanplus/g;s/CPAN::.*/cpan/g;s/::/-/g;s/ [^+\"].*//g;s/_/-/g;s/\.pl\"$/\"/;s/\"\?\$/\"/;s/(//;" | tr [:upper:] [:lower:] \).
Please clarify whether this is all a comment or should be parsed.
ERROR: Unable to parse /OE/shr-core/openembedded-core/meta/recipes-devtools/perl/perl_5.14.2.bb: Exited with "1"
ERROR: Command execution failed: Exited with 1
Hi Jansa,
Because my boy was born, I have been leaving office and missed this message.
np, congratz!

Thank you!

Regards,
Kai

Quoted 14 lines

Cheers,
Quoted 11 lines
Quoted 11 lines
Quoted 11 lines
---
   .../recipes-devtools/perl/perl-rdepends_5.14.2.inc | 2336 +++++++++++++++++++-
   1 files changed, 2287 insertions(+), 49 deletions(-)
Diff: http://git.openembedded.org/?p=openembedded-core.git/?a=commitdiff;h=35291c97edc05756788d206457c56dcc41edb637
_______________________________________________
Openembedded-commits mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-commits