diff mbox series

[meta-perl,v2] libmodule-build-tiny-perl: upgrade 0.045 -> 0.046

Message ID 20230807164452.381251-1-ticotimo@gmail.com
State New
Headers show
Series [meta-perl,v2] libmodule-build-tiny-perl: upgrade 0.045 -> 0.046 | expand

Commit Message

Tim Orling Aug. 7, 2023, 4:44 p.m. UTC 
* Drop redundant do_install
* Use four spaces for indent in RDEPENDS
* RDEPENDS:
  - += not =
  - sort alphabetically
  - Add perl-module-perl-ostype
  - Add perl-module-extutils-cbuilder-base
  - Replace perl-module-ipc-open3 with -ipc-cmd
  - Add perl-module-metadata
  - Add perl-module-parse-cpan-meta
  - Add perl-module-extutils-install
  - Add perl-module-encode-encoding
* Enable ptest
* RDEPENDS for ptest:
  - Add perl-module-ipc-open2
  - Add perl-module-blib
  - Add packagegroup-core-standalone-sdk-target
  - Add gcc
  - Add binutils
  - Add perl-dev (for "EXTERN.h")
* INSANE_SKIP dev-deps for -ptest

0.046 2023-06-01 19:31:21+02:00 Europe/Amsterdam
      - Add src/ to include paths

License-Update: Updated Artistic-1.0 (Perl) text

Signed-off-by: Tim Orling <ticotimo@gmail.com>
---
Changes in v2:
  * use packagegroup-core-standalone-sdk-target intead of
    individual libgcc-dev etc. components to be musl friendly
  * ptest results are the same for qemux86-musl and qemux86-64

 .../include/ptest-packagelists-meta-perl.inc  |  1 +
 .../libmodule-build-tiny-perl_0.045.bb        | 53 ---------------
 .../libmodule-build-tiny-perl_0.046.bb        | 67 +++++++++++++++++++
 3 files changed, 68 insertions(+), 53 deletions(-)
 delete mode 100644 meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb
 create mode 100644 meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb

Comments

Khem Raj Aug. 7, 2023, 5:26 p.m. UTC | #1 
On Mon, Aug 7, 2023 at 9:47 AM Tim Orling <ticotimo@gmail.com> wrote:
>
> * Drop redundant do_install
> * Use four spaces for indent in RDEPENDS
> * RDEPENDS:
>   - += not =
>   - sort alphabetically
>   - Add perl-module-perl-ostype
>   - Add perl-module-extutils-cbuilder-base
>   - Replace perl-module-ipc-open3 with -ipc-cmd
>   - Add perl-module-metadata
>   - Add perl-module-parse-cpan-meta
>   - Add perl-module-extutils-install
>   - Add perl-module-encode-encoding
> * Enable ptest
> * RDEPENDS for ptest:
>   - Add perl-module-ipc-open2
>   - Add perl-module-blib
>   - Add packagegroup-core-standalone-sdk-target
>   - Add gcc
>   - Add binutils
>   - Add perl-dev (for "EXTERN.h")
> * INSANE_SKIP dev-deps for -ptest
>
> 0.046 2023-06-01 19:31:21+02:00 Europe/Amsterdam
>       - Add src/ to include paths
>
> License-Update: Updated Artistic-1.0 (Perl) text
>
> Signed-off-by: Tim Orling <ticotimo@gmail.com>
> ---
> Changes in v2:
>   * use packagegroup-core-standalone-sdk-target intead of
>     individual libgcc-dev etc. components to be musl friendly
>   * ptest results are the same for qemux86-musl and qemux86-64
>
>  .../include/ptest-packagelists-meta-perl.inc  |  1 +
>  .../libmodule-build-tiny-perl_0.045.bb        | 53 ---------------
>  .../libmodule-build-tiny-perl_0.046.bb        | 67 +++++++++++++++++++
>  3 files changed, 68 insertions(+), 53 deletions(-)
>  delete mode 100644 meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb
>  create mode 100644 meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb
>
> diff --git a/meta-perl/conf/include/ptest-packagelists-meta-perl.inc b/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
> index 1a8c759b1..e27981de7 100644
> --- a/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
> +++ b/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
> @@ -13,6 +13,7 @@ PTESTS_FAST_META_PERL = "\
>      libextutils-config-perl \
>      libhtml-tagset-perl \
>      libmozilla-ca-perl \
> +    libmodule-build-tiny-perl \
>      libstrictures-perl \
>      libterm-readkey-perl \
>      libtest-nowarnings-perl \
> diff --git a/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb b/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb
> deleted file mode 100644
> index 17ee50d7c..000000000
> --- a/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb
> +++ /dev/null
> @@ -1,53 +0,0 @@
> -SUMMARY = "Module::Build::Tiny - A tiny replacement for Module::Build"
> -DESCRIPTION = "Many Perl distributions use a Build.PL file instead of a \
> -Makefile.PL file to drive distribution configuration, build, test and \
> -installation. Traditionally, Build.PL uses Module::Build as the underlying \
> -build system. This module provides a simple, lightweight, drop-in replacement. \
> -Whereas Module::Build has over 6,700 lines of code; this module has less than \
> -120, yet supports the features needed by most distributions."
> -SECTION = "libs"
> -
> -HOMEPAGE = "http://search.cpan.org/~leont/Module-Build-Tiny/"
> -
> -LICENSE = "Artistic-1.0 | GPL-1.0-or-later"
> -LIC_FILES_CHKSUM = "file://LICENSE;md5=6374136f6774db2f51cc116f5efbf705"
> -
> -DEPENDS = "libextutils-config-perl-native libextutils-helpers-perl-native libextutils-installpaths-perl-native"
> -
> -SRC_URI = "${CPAN_MIRROR}/authors/id/L/LE/LEONT/Module-Build-Tiny-${PV}.tar.gz"
> -SRC_URI[sha256sum] = "d20692eee246d0b329b7f7659f36286b14ae0696fe91078a64b7078f8876d300"
> -
> -S = "${WORKDIR}/Module-Build-Tiny-${PV}"
> -
> -inherit cpan_build
> -
> -do_install () {
> -        cpan_build_do_install
> -}
> -
> -RDEPENDS:${PN} = " libextutils-config-perl \
> -                   libextutils-helpers-perl \
> -                   libextutils-installpaths-perl \
> -                   perl-module-xsloader \
> -                   perl-module-file-spec \
> -                   perl-module-io-handle \
> -                   perl-module-tap-harness-env \
> -                   perl-module-ipc-open3 \
> -                   perl-module-file-path \
> -                   perl-module-cpan \
> -                   perl-module-extutils-cbuilder \
> -                   perl-module-getopt-long \
> -                   perl-module-extutils-makemaker \
> -                   perl-module-exporter \
> -                   perl-module-carp \
> -                   perl-module-test-more \
> -                   perl-module-text-parsewords \
> -                   perl-module-load \
> -                   perl-module-file-temp \
> -                   perl-module-data-dumper \
> -                   perl-module-extutils-parsexs \
> -                   perl-module-pod-man \
> -                   perl-module-json-pp \
> -"
> -
> -BBCLASSEXTEND = "native"
> diff --git a/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb b/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb
> new file mode 100644
> index 000000000..efbccdda8
> --- /dev/null
> +++ b/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb
> @@ -0,0 +1,67 @@
> +SUMMARY = "Module::Build::Tiny - A tiny replacement for Module::Build"
> +DESCRIPTION = "Many Perl distributions use a Build.PL file instead of a \
> +Makefile.PL file to drive distribution configuration, build, test and \
> +installation. Traditionally, Build.PL uses Module::Build as the underlying \
> +build system. This module provides a simple, lightweight, drop-in replacement. \
> +Whereas Module::Build has over 6,700 lines of code; this module has less than \
> +120, yet supports the features needed by most distributions."
> +SECTION = "libs"
> +
> +HOMEPAGE = "http://search.cpan.org/~leont/Module-Build-Tiny/"
> +
> +LICENSE = "Artistic-1.0 | GPL-1.0-or-later"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=147a7b891d1584d3d33e478b1f2327c5"
> +
> +DEPENDS = "libextutils-config-perl-native libextutils-helpers-perl-native libextutils-installpaths-perl-native"
> +
> +SRC_URI = "${CPAN_MIRROR}/authors/id/L/LE/LEONT/Module-Build-Tiny-${PV}.tar.gz"
> +SRC_URI[sha256sum] = "be193027e2debb4c9926434aaf1aa46c9fc7bf4db83dcc347eb6e359ee878289"
> +
> +S = "${WORKDIR}/Module-Build-Tiny-${PV}"
> +
> +inherit cpan_build ptest-perl
> +
> +RDEPENDS:${PN} += " \
> +    libextutils-config-perl \
> +    libextutils-helpers-perl \
> +    libextutils-installpaths-perl \
> +    perl-module-carp \
> +    perl-module-cpan \
> +    perl-module-data-dumper \
> +    perl-module-encode-encoding \
> +    perl-module-exporter \
> +    perl-module-extutils-cbuilder \
> +    perl-module-extutils-cbuilder-base \
> +    perl-module-extutils-install \
> +    perl-module-extutils-makemaker \
> +    perl-module-extutils-parsexs \
> +    perl-module-file-path \
> +    perl-module-file-spec \
> +    perl-module-file-temp \
> +    perl-module-getopt-long \
> +    perl-module-io-handle \
> +    perl-module-ipc-cmd \
> +    perl-module-json-pp \
> +    perl-module-load \
> +    perl-module-metadata \
> +    perl-module-parse-cpan-meta \
> +    perl-module-perl-ostype \
> +    perl-module-pod-man \
> +    perl-module-tap-harness-env \
> +    perl-module-test-more \
> +    perl-module-text-parsewords \
> +    perl-module-xsloader \
> +"
> +
> +RDEPENDS:${PN}-ptest += " \
> +    binutils \
> +    gcc \

binutils and gcc should be part of packagegroup below
so you can remove it as well.

> +    packagegroup-core-standalone-sdk-target \
> +    perl-dev \
> +    perl-module-blib \
> +    perl-module-ipc-open2 \
> +"
> +
> +INSANE_SKIP:${PN}-ptest += "dev-deps"
> +
> +BBCLASSEXTEND = "native"
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#104302): https://lists.openembedded.org/g/openembedded-devel/message/104302
> Mute This Topic: https://lists.openembedded.org/mt/100603983/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Tim Orling Aug. 7, 2023, 5:46 p.m. UTC | #2 
On Mon, Aug 7, 2023 at 10:26 AM Khem Raj <raj.khem@gmail.com> wrote:

> On Mon, Aug 7, 2023 at 9:47 AM Tim Orling <ticotimo@gmail.com> wrote:
> >
> > * Drop redundant do_install
> > * Use four spaces for indent in RDEPENDS
> > * RDEPENDS:
> >   - += not =
> >   - sort alphabetically
> >   - Add perl-module-perl-ostype
> >   - Add perl-module-extutils-cbuilder-base
> >   - Replace perl-module-ipc-open3 with -ipc-cmd
> >   - Add perl-module-metadata
> >   - Add perl-module-parse-cpan-meta
> >   - Add perl-module-extutils-install
> >   - Add perl-module-encode-encoding
> > * Enable ptest
> > * RDEPENDS for ptest:
> >   - Add perl-module-ipc-open2
> >   - Add perl-module-blib
> >   - Add packagegroup-core-standalone-sdk-target
> >   - Add gcc
> >   - Add binutils
> >   - Add perl-dev (for "EXTERN.h")
> > * INSANE_SKIP dev-deps for -ptest
> >
> > 0.046 2023-06-01 19:31:21+02:00 Europe/Amsterdam
> >       - Add src/ to include paths
> >
> > License-Update: Updated Artistic-1.0 (Perl) text
> >
> > Signed-off-by: Tim Orling <ticotimo@gmail.com>
> > ---
> > Changes in v2:
> >   * use packagegroup-core-standalone-sdk-target intead of
> >     individual libgcc-dev etc. components to be musl friendly
> >   * ptest results are the same for qemux86-musl and qemux86-64
> >
> >  .../include/ptest-packagelists-meta-perl.inc  |  1 +
> >  .../libmodule-build-tiny-perl_0.045.bb        | 53 ---------------
> >  .../libmodule-build-tiny-perl_0.046.bb        | 67 +++++++++++++++++++
> >  3 files changed, 68 insertions(+), 53 deletions(-)
> >  delete mode 100644 meta-perl/recipes-perl/libmodule/
> libmodule-build-tiny-perl_0.045.bb
> >  create mode 100644 meta-perl/recipes-perl/libmodule/
> libmodule-build-tiny-perl_0.046.bb
> >
> > diff --git a/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
> b/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
> > index 1a8c759b1..e27981de7 100644
> > --- a/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
> > +++ b/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
> > @@ -13,6 +13,7 @@ PTESTS_FAST_META_PERL = "\
> >      libextutils-config-perl \
> >      libhtml-tagset-perl \
> >      libmozilla-ca-perl \
> > +    libmodule-build-tiny-perl \
> >      libstrictures-perl \
> >      libterm-readkey-perl \
> >      libtest-nowarnings-perl \
> > diff --git a/meta-perl/recipes-perl/libmodule/
> libmodule-build-tiny-perl_0.045.bb b/meta-perl/recipes-perl/libmodule/
> libmodule-build-tiny-perl_0.045.bb
> > deleted file mode 100644
> > index 17ee50d7c..000000000
> > --- a/meta-perl/recipes-perl/libmodule/
> libmodule-build-tiny-perl_0.045.bb
> > +++ /dev/null
> > @@ -1,53 +0,0 @@
> > -SUMMARY = "Module::Build::Tiny - A tiny replacement for Module::Build"
> > -DESCRIPTION = "Many Perl distributions use a Build.PL file instead of a
> \
> > -Makefile.PL file to drive distribution configuration, build, test and \
> > -installation. Traditionally, Build.PL uses Module::Build as the
> underlying \
> > -build system. This module provides a simple, lightweight, drop-in
> replacement. \
> > -Whereas Module::Build has over 6,700 lines of code; this module has
> less than \
> > -120, yet supports the features needed by most distributions."
> > -SECTION = "libs"
> > -
> > -HOMEPAGE = "http://search.cpan.org/~leont/Module-Build-Tiny/"
> > -
> > -LICENSE = "Artistic-1.0 | GPL-1.0-or-later"
> > -LIC_FILES_CHKSUM = "file://LICENSE;md5=6374136f6774db2f51cc116f5efbf705"
> > -
> > -DEPENDS = "libextutils-config-perl-native
> libextutils-helpers-perl-native libextutils-installpaths-perl-native"
> > -
> > -SRC_URI =
> "${CPAN_MIRROR}/authors/id/L/LE/LEONT/Module-Build-Tiny-${PV}.tar.gz"
> > -SRC_URI[sha256sum] =
> "d20692eee246d0b329b7f7659f36286b14ae0696fe91078a64b7078f8876d300"
> > -
> > -S = "${WORKDIR}/Module-Build-Tiny-${PV}"
> > -
> > -inherit cpan_build
> > -
> > -do_install () {
> > -        cpan_build_do_install
> > -}
> > -
> > -RDEPENDS:${PN} = " libextutils-config-perl \
> > -                   libextutils-helpers-perl \
> > -                   libextutils-installpaths-perl \
> > -                   perl-module-xsloader \
> > -                   perl-module-file-spec \
> > -                   perl-module-io-handle \
> > -                   perl-module-tap-harness-env \
> > -                   perl-module-ipc-open3 \
> > -                   perl-module-file-path \
> > -                   perl-module-cpan \
> > -                   perl-module-extutils-cbuilder \
> > -                   perl-module-getopt-long \
> > -                   perl-module-extutils-makemaker \
> > -                   perl-module-exporter \
> > -                   perl-module-carp \
> > -                   perl-module-test-more \
> > -                   perl-module-text-parsewords \
> > -                   perl-module-load \
> > -                   perl-module-file-temp \
> > -                   perl-module-data-dumper \
> > -                   perl-module-extutils-parsexs \
> > -                   perl-module-pod-man \
> > -                   perl-module-json-pp \
> > -"
> > -
> > -BBCLASSEXTEND = "native"
> > diff --git a/meta-perl/recipes-perl/libmodule/
> libmodule-build-tiny-perl_0.046.bb b/meta-perl/recipes-perl/libmodule/
> libmodule-build-tiny-perl_0.046.bb
> > new file mode 100644
> > index 000000000..efbccdda8
> > --- /dev/null
> > +++ b/meta-perl/recipes-perl/libmodule/
> libmodule-build-tiny-perl_0.046.bb
> > @@ -0,0 +1,67 @@
> > +SUMMARY = "Module::Build::Tiny - A tiny replacement for Module::Build"
> > +DESCRIPTION = "Many Perl distributions use a Build.PL file instead of a
> \
> > +Makefile.PL file to drive distribution configuration, build, test and \
> > +installation. Traditionally, Build.PL uses Module::Build as the
> underlying \
> > +build system. This module provides a simple, lightweight, drop-in
> replacement. \
> > +Whereas Module::Build has over 6,700 lines of code; this module has
> less than \
> > +120, yet supports the features needed by most distributions."
> > +SECTION = "libs"
> > +
> > +HOMEPAGE = "http://search.cpan.org/~leont/Module-Build-Tiny/"
> > +
> > +LICENSE = "Artistic-1.0 | GPL-1.0-or-later"
> > +LIC_FILES_CHKSUM = "file://LICENSE;md5=147a7b891d1584d3d33e478b1f2327c5"
> > +
> > +DEPENDS = "libextutils-config-perl-native
> libextutils-helpers-perl-native libextutils-installpaths-perl-native"
> > +
> > +SRC_URI =
> "${CPAN_MIRROR}/authors/id/L/LE/LEONT/Module-Build-Tiny-${PV}.tar.gz"
> > +SRC_URI[sha256sum] =
> "be193027e2debb4c9926434aaf1aa46c9fc7bf4db83dcc347eb6e359ee878289"
> > +
> > +S = "${WORKDIR}/Module-Build-Tiny-${PV}"
> > +
> > +inherit cpan_build ptest-perl
> > +
> > +RDEPENDS:${PN} += " \
> > +    libextutils-config-perl \
> > +    libextutils-helpers-perl \
> > +    libextutils-installpaths-perl \
> > +    perl-module-carp \
> > +    perl-module-cpan \
> > +    perl-module-data-dumper \
> > +    perl-module-encode-encoding \
> > +    perl-module-exporter \
> > +    perl-module-extutils-cbuilder \
> > +    perl-module-extutils-cbuilder-base \
> > +    perl-module-extutils-install \
> > +    perl-module-extutils-makemaker \
> > +    perl-module-extutils-parsexs \
> > +    perl-module-file-path \
> > +    perl-module-file-spec \
> > +    perl-module-file-temp \
> > +    perl-module-getopt-long \
> > +    perl-module-io-handle \
> > +    perl-module-ipc-cmd \
> > +    perl-module-json-pp \
> > +    perl-module-load \
> > +    perl-module-metadata \
> > +    perl-module-parse-cpan-meta \
> > +    perl-module-perl-ostype \
> > +    perl-module-pod-man \
> > +    perl-module-tap-harness-env \
> > +    perl-module-test-more \
> > +    perl-module-text-parsewords \
> > +    perl-module-xsloader \
> > +"
> > +
> > +RDEPENDS:${PN}-ptest += " \
> > +    binutils \
> > +    gcc \
>
> binutils and gcc should be part of packagegroup below
> so you can remove it as well.
>
>
They are not. I built without them added and neither "gcc" (-musl-gcc) nor
"as" (-musl-as) were present.
The packagegroup only includes libgcc and friends, but not the tools in
/usr/bin/


> > +    packagegroup-core-standalone-sdk-target \
> > +    perl-dev \
> > +    perl-module-blib \
> > +    perl-module-ipc-open2 \
> > +"
> > +
> > +INSANE_SKIP:${PN}-ptest += "dev-deps"
> > +
> > +BBCLASSEXTEND = "native"
> > --
> > 2.34.1
> >
> >
> > -=-=-=-=-=-=-=-=-=-=-=-
> > Links: You receive all messages sent to this group.
> > View/Reply Online (#104302):
> https://lists.openembedded.org/g/openembedded-devel/message/104302
> > Mute This Topic: https://lists.openembedded.org/mt/100603983/1997914
> > Group Owner: openembedded-devel+owner@lists.openembedded.org
> > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [
> raj.khem@gmail.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >
>
Khem Raj Aug. 7, 2023, 5:53 p.m. UTC | #3 
On Mon, Aug 7, 2023 at 10:46 AM Tim Orling <ticotimo@gmail.com> wrote:
>
>
>
> On Mon, Aug 7, 2023 at 10:26 AM Khem Raj <raj.khem@gmail.com> wrote:
>>
>> On Mon, Aug 7, 2023 at 9:47 AM Tim Orling <ticotimo@gmail.com> wrote:
>> >
>> > * Drop redundant do_install
>> > * Use four spaces for indent in RDEPENDS
>> > * RDEPENDS:
>> >   - += not =
>> >   - sort alphabetically
>> >   - Add perl-module-perl-ostype
>> >   - Add perl-module-extutils-cbuilder-base
>> >   - Replace perl-module-ipc-open3 with -ipc-cmd
>> >   - Add perl-module-metadata
>> >   - Add perl-module-parse-cpan-meta
>> >   - Add perl-module-extutils-install
>> >   - Add perl-module-encode-encoding
>> > * Enable ptest
>> > * RDEPENDS for ptest:
>> >   - Add perl-module-ipc-open2
>> >   - Add perl-module-blib
>> >   - Add packagegroup-core-standalone-sdk-target
>> >   - Add gcc
>> >   - Add binutils
>> >   - Add perl-dev (for "EXTERN.h")
>> > * INSANE_SKIP dev-deps for -ptest
>> >
>> > 0.046 2023-06-01 19:31:21+02:00 Europe/Amsterdam
>> >       - Add src/ to include paths
>> >
>> > License-Update: Updated Artistic-1.0 (Perl) text
>> >
>> > Signed-off-by: Tim Orling <ticotimo@gmail.com>
>> > ---
>> > Changes in v2:
>> >   * use packagegroup-core-standalone-sdk-target intead of
>> >     individual libgcc-dev etc. components to be musl friendly
>> >   * ptest results are the same for qemux86-musl and qemux86-64
>> >
>> >  .../include/ptest-packagelists-meta-perl.inc  |  1 +
>> >  .../libmodule-build-tiny-perl_0.045.bb        | 53 ---------------
>> >  .../libmodule-build-tiny-perl_0.046.bb        | 67 +++++++++++++++++++
>> >  3 files changed, 68 insertions(+), 53 deletions(-)
>> >  delete mode 100644 meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb
>> >  create mode 100644 meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb
>> >
>> > diff --git a/meta-perl/conf/include/ptest-packagelists-meta-perl.inc b/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
>> > index 1a8c759b1..e27981de7 100644
>> > --- a/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
>> > +++ b/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
>> > @@ -13,6 +13,7 @@ PTESTS_FAST_META_PERL = "\
>> >      libextutils-config-perl \
>> >      libhtml-tagset-perl \
>> >      libmozilla-ca-perl \
>> > +    libmodule-build-tiny-perl \
>> >      libstrictures-perl \
>> >      libterm-readkey-perl \
>> >      libtest-nowarnings-perl \
>> > diff --git a/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb b/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb
>> > deleted file mode 100644
>> > index 17ee50d7c..000000000
>> > --- a/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb
>> > +++ /dev/null
>> > @@ -1,53 +0,0 @@
>> > -SUMMARY = "Module::Build::Tiny - A tiny replacement for Module::Build"
>> > -DESCRIPTION = "Many Perl distributions use a Build.PL file instead of a \
>> > -Makefile.PL file to drive distribution configuration, build, test and \
>> > -installation. Traditionally, Build.PL uses Module::Build as the underlying \
>> > -build system. This module provides a simple, lightweight, drop-in replacement. \
>> > -Whereas Module::Build has over 6,700 lines of code; this module has less than \
>> > -120, yet supports the features needed by most distributions."
>> > -SECTION = "libs"
>> > -
>> > -HOMEPAGE = "http://search.cpan.org/~leont/Module-Build-Tiny/"
>> > -
>> > -LICENSE = "Artistic-1.0 | GPL-1.0-or-later"
>> > -LIC_FILES_CHKSUM = "file://LICENSE;md5=6374136f6774db2f51cc116f5efbf705"
>> > -
>> > -DEPENDS = "libextutils-config-perl-native libextutils-helpers-perl-native libextutils-installpaths-perl-native"
>> > -
>> > -SRC_URI = "${CPAN_MIRROR}/authors/id/L/LE/LEONT/Module-Build-Tiny-${PV}.tar.gz"
>> > -SRC_URI[sha256sum] = "d20692eee246d0b329b7f7659f36286b14ae0696fe91078a64b7078f8876d300"
>> > -
>> > -S = "${WORKDIR}/Module-Build-Tiny-${PV}"
>> > -
>> > -inherit cpan_build
>> > -
>> > -do_install () {
>> > -        cpan_build_do_install
>> > -}
>> > -
>> > -RDEPENDS:${PN} = " libextutils-config-perl \
>> > -                   libextutils-helpers-perl \
>> > -                   libextutils-installpaths-perl \
>> > -                   perl-module-xsloader \
>> > -                   perl-module-file-spec \
>> > -                   perl-module-io-handle \
>> > -                   perl-module-tap-harness-env \
>> > -                   perl-module-ipc-open3 \
>> > -                   perl-module-file-path \
>> > -                   perl-module-cpan \
>> > -                   perl-module-extutils-cbuilder \
>> > -                   perl-module-getopt-long \
>> > -                   perl-module-extutils-makemaker \
>> > -                   perl-module-exporter \
>> > -                   perl-module-carp \
>> > -                   perl-module-test-more \
>> > -                   perl-module-text-parsewords \
>> > -                   perl-module-load \
>> > -                   perl-module-file-temp \
>> > -                   perl-module-data-dumper \
>> > -                   perl-module-extutils-parsexs \
>> > -                   perl-module-pod-man \
>> > -                   perl-module-json-pp \
>> > -"
>> > -
>> > -BBCLASSEXTEND = "native"
>> > diff --git a/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb b/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb
>> > new file mode 100644
>> > index 000000000..efbccdda8
>> > --- /dev/null
>> > +++ b/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb
>> > @@ -0,0 +1,67 @@
>> > +SUMMARY = "Module::Build::Tiny - A tiny replacement for Module::Build"
>> > +DESCRIPTION = "Many Perl distributions use a Build.PL file instead of a \
>> > +Makefile.PL file to drive distribution configuration, build, test and \
>> > +installation. Traditionally, Build.PL uses Module::Build as the underlying \
>> > +build system. This module provides a simple, lightweight, drop-in replacement. \
>> > +Whereas Module::Build has over 6,700 lines of code; this module has less than \
>> > +120, yet supports the features needed by most distributions."
>> > +SECTION = "libs"
>> > +
>> > +HOMEPAGE = "http://search.cpan.org/~leont/Module-Build-Tiny/"
>> > +
>> > +LICENSE = "Artistic-1.0 | GPL-1.0-or-later"
>> > +LIC_FILES_CHKSUM = "file://LICENSE;md5=147a7b891d1584d3d33e478b1f2327c5"
>> > +
>> > +DEPENDS = "libextutils-config-perl-native libextutils-helpers-perl-native libextutils-installpaths-perl-native"
>> > +
>> > +SRC_URI = "${CPAN_MIRROR}/authors/id/L/LE/LEONT/Module-Build-Tiny-${PV}.tar.gz"
>> > +SRC_URI[sha256sum] = "be193027e2debb4c9926434aaf1aa46c9fc7bf4db83dcc347eb6e359ee878289"
>> > +
>> > +S = "${WORKDIR}/Module-Build-Tiny-${PV}"
>> > +
>> > +inherit cpan_build ptest-perl
>> > +
>> > +RDEPENDS:${PN} += " \
>> > +    libextutils-config-perl \
>> > +    libextutils-helpers-perl \
>> > +    libextutils-installpaths-perl \
>> > +    perl-module-carp \
>> > +    perl-module-cpan \
>> > +    perl-module-data-dumper \
>> > +    perl-module-encode-encoding \
>> > +    perl-module-exporter \
>> > +    perl-module-extutils-cbuilder \
>> > +    perl-module-extutils-cbuilder-base \
>> > +    perl-module-extutils-install \
>> > +    perl-module-extutils-makemaker \
>> > +    perl-module-extutils-parsexs \
>> > +    perl-module-file-path \
>> > +    perl-module-file-spec \
>> > +    perl-module-file-temp \
>> > +    perl-module-getopt-long \
>> > +    perl-module-io-handle \
>> > +    perl-module-ipc-cmd \
>> > +    perl-module-json-pp \
>> > +    perl-module-load \
>> > +    perl-module-metadata \
>> > +    perl-module-parse-cpan-meta \
>> > +    perl-module-perl-ostype \
>> > +    perl-module-pod-man \
>> > +    perl-module-tap-harness-env \
>> > +    perl-module-test-more \
>> > +    perl-module-text-parsewords \
>> > +    perl-module-xsloader \
>> > +"
>> > +
>> > +RDEPENDS:${PN}-ptest += " \
>> > +    binutils \
>> > +    gcc \
>>
>> binutils and gcc should be part of packagegroup below
>> so you can remove it as well.
>>
>
> They are not. I built without them added and neither "gcc" (-musl-gcc) nor "as" (-musl-as) were present.
> The packagegroup only includes libgcc and friends, but not the tools in /usr/bin/
>

Looking at the core you are right. Contrary to name this packagegroup
does not provide tools but just runtime.
perhaps packagegroup-core-buildessential is more appropriate instead.

>>
>> > +    packagegroup-core-standalone-sdk-target \
>> > +    perl-dev \
>> > +    perl-module-blib \
>> > +    perl-module-ipc-open2 \
>> > +"
>> > +
>> > +INSANE_SKIP:${PN}-ptest += "dev-deps"
>> > +
>> > +BBCLASSEXTEND = "native"
>> > --
>> > 2.34.1
>> >
>> >
>> > -=-=-=-=-=-=-=-=-=-=-=-
>> > Links: You receive all messages sent to this group.
>> > View/Reply Online (#104302): https://lists.openembedded.org/g/openembedded-devel/message/104302
>> > Mute This Topic: https://lists.openembedded.org/mt/100603983/1997914
>> > Group Owner: openembedded-devel+owner@lists.openembedded.org
>> > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
>> > -=-=-=-=-=-=-=-=-=-=-=-
>> >
Tim Orling Aug. 8, 2023, 10:48 p.m. UTC | #4 
On Mon, Aug 7, 2023 at 10:53 AM Khem Raj <raj.khem@gmail.com> wrote:

> On Mon, Aug 7, 2023 at 10:46 AM Tim Orling <ticotimo@gmail.com> wrote:
> >
> <snip>
> >> > +RDEPENDS:${PN}-ptest += " \
> >> > +    binutils \
> >> > +    gcc \
> >>
> >> binutils and gcc should be part of packagegroup below
> >> so you can remove it as well.
> >>
> >
> > They are not. I built without them added and neither "gcc" (-musl-gcc)
> nor "as" (-musl-as) were present.
> > The packagegroup only includes libgcc and friends, but not the tools in
> /usr/bin/
> >
>
> Looking at the core you are right. Contrary to name this packagegroup
> does not provide tools but just runtime.
> perhaps packagegroup-core-buildessential is more appropriate instead.
>
>
Agreed. Even though packagegroup-core-buildessential has more tools than
are strictly required NOW it is an example for the future and should help
us be "future proof".
I'll update in v3.

Thank you for helping make meta-openembedded a better place.


> >>
> >> > +    packagegroup-core-standalone-sdk-target \
> >> > +    perl-dev \
> >> > +    perl-module-blib \
> >> > +    perl-module-ipc-open2 \
> >> > +"
> >> > +
> >> > +INSANE_SKIP:${PN}-ptest += "dev-deps"
> >> > +
> >> > +BBCLASSEXTEND = "native"
> >> > --
> >> > 2.34.1
> >> >
> >> >
> >> > -=-=-=-=-=-=-=-=-=-=-=-
> >> > Links: You receive all messages sent to this group.
> >> > View/Reply Online (#104302):
> https://lists.openembedded.org/g/openembedded-devel/message/104302
> >> > Mute This Topic: https://lists.openembedded.org/mt/100603983/1997914
> >> > Group Owner: openembedded-devel+owner@lists.openembedded.org
> >> > Unsubscribe:
> https://lists.openembedded.org/g/openembedded-devel/unsub [
> raj.khem@gmail.com]
> >> > -=-=-=-=-=-=-=-=-=-=-=-
> >> >
>
diff mbox series

Patch

diff --git a/meta-perl/conf/include/ptest-packagelists-meta-perl.inc b/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
index 1a8c759b1..e27981de7 100644
--- a/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
+++ b/meta-perl/conf/include/ptest-packagelists-meta-perl.inc
@@ -13,6 +13,7 @@  PTESTS_FAST_META_PERL = "\
     libextutils-config-perl \
     libhtml-tagset-perl \
     libmozilla-ca-perl \
+    libmodule-build-tiny-perl \
     libstrictures-perl \
     libterm-readkey-perl \
     libtest-nowarnings-perl \
diff --git a/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb b/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb
deleted file mode 100644
index 17ee50d7c..000000000
--- a/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.045.bb
+++ /dev/null
@@ -1,53 +0,0 @@ 
-SUMMARY = "Module::Build::Tiny - A tiny replacement for Module::Build"
-DESCRIPTION = "Many Perl distributions use a Build.PL file instead of a \
-Makefile.PL file to drive distribution configuration, build, test and \
-installation. Traditionally, Build.PL uses Module::Build as the underlying \
-build system. This module provides a simple, lightweight, drop-in replacement. \
-Whereas Module::Build has over 6,700 lines of code; this module has less than \
-120, yet supports the features needed by most distributions."
-SECTION = "libs"
-
-HOMEPAGE = "http://search.cpan.org/~leont/Module-Build-Tiny/"
-
-LICENSE = "Artistic-1.0 | GPL-1.0-or-later"
-LIC_FILES_CHKSUM = "file://LICENSE;md5=6374136f6774db2f51cc116f5efbf705"
-
-DEPENDS = "libextutils-config-perl-native libextutils-helpers-perl-native libextutils-installpaths-perl-native"
-
-SRC_URI = "${CPAN_MIRROR}/authors/id/L/LE/LEONT/Module-Build-Tiny-${PV}.tar.gz"
-SRC_URI[sha256sum] = "d20692eee246d0b329b7f7659f36286b14ae0696fe91078a64b7078f8876d300"
-
-S = "${WORKDIR}/Module-Build-Tiny-${PV}"
-
-inherit cpan_build
-
-do_install () {
-        cpan_build_do_install
-}
-
-RDEPENDS:${PN} = " libextutils-config-perl \
-                   libextutils-helpers-perl \
-                   libextutils-installpaths-perl \
-                   perl-module-xsloader \
-                   perl-module-file-spec \
-                   perl-module-io-handle \
-                   perl-module-tap-harness-env \
-                   perl-module-ipc-open3 \
-                   perl-module-file-path \
-                   perl-module-cpan \
-                   perl-module-extutils-cbuilder \
-                   perl-module-getopt-long \
-                   perl-module-extutils-makemaker \
-                   perl-module-exporter \
-                   perl-module-carp \
-                   perl-module-test-more \
-                   perl-module-text-parsewords \
-                   perl-module-load \
-                   perl-module-file-temp \
-                   perl-module-data-dumper \
-                   perl-module-extutils-parsexs \
-                   perl-module-pod-man \
-                   perl-module-json-pp \
-"
-
-BBCLASSEXTEND = "native"
diff --git a/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb b/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb
new file mode 100644
index 000000000..efbccdda8
--- /dev/null
+++ b/meta-perl/recipes-perl/libmodule/libmodule-build-tiny-perl_0.046.bb
@@ -0,0 +1,67 @@ 
+SUMMARY = "Module::Build::Tiny - A tiny replacement for Module::Build"
+DESCRIPTION = "Many Perl distributions use a Build.PL file instead of a \
+Makefile.PL file to drive distribution configuration, build, test and \
+installation. Traditionally, Build.PL uses Module::Build as the underlying \
+build system. This module provides a simple, lightweight, drop-in replacement. \
+Whereas Module::Build has over 6,700 lines of code; this module has less than \
+120, yet supports the features needed by most distributions."
+SECTION = "libs"
+
+HOMEPAGE = "http://search.cpan.org/~leont/Module-Build-Tiny/"
+
+LICENSE = "Artistic-1.0 | GPL-1.0-or-later"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=147a7b891d1584d3d33e478b1f2327c5"
+
+DEPENDS = "libextutils-config-perl-native libextutils-helpers-perl-native libextutils-installpaths-perl-native"
+
+SRC_URI = "${CPAN_MIRROR}/authors/id/L/LE/LEONT/Module-Build-Tiny-${PV}.tar.gz"
+SRC_URI[sha256sum] = "be193027e2debb4c9926434aaf1aa46c9fc7bf4db83dcc347eb6e359ee878289"
+
+S = "${WORKDIR}/Module-Build-Tiny-${PV}"
+
+inherit cpan_build ptest-perl
+
+RDEPENDS:${PN} += " \
+    libextutils-config-perl \
+    libextutils-helpers-perl \
+    libextutils-installpaths-perl \
+    perl-module-carp \
+    perl-module-cpan \
+    perl-module-data-dumper \
+    perl-module-encode-encoding \
+    perl-module-exporter \
+    perl-module-extutils-cbuilder \
+    perl-module-extutils-cbuilder-base \
+    perl-module-extutils-install \
+    perl-module-extutils-makemaker \
+    perl-module-extutils-parsexs \
+    perl-module-file-path \
+    perl-module-file-spec \
+    perl-module-file-temp \
+    perl-module-getopt-long \
+    perl-module-io-handle \
+    perl-module-ipc-cmd \
+    perl-module-json-pp \
+    perl-module-load \
+    perl-module-metadata \
+    perl-module-parse-cpan-meta \
+    perl-module-perl-ostype \
+    perl-module-pod-man \
+    perl-module-tap-harness-env \
+    perl-module-test-more \
+    perl-module-text-parsewords \
+    perl-module-xsloader \
+"
+
+RDEPENDS:${PN}-ptest += " \
+    binutils \
+    gcc \
+    packagegroup-core-standalone-sdk-target \
+    perl-dev \
+    perl-module-blib \
+    perl-module-ipc-open2 \
+"
+
+INSANE_SKIP:${PN}-ptest += "dev-deps"
+
+BBCLASSEXTEND = "native"