Re: [PATCH 0-10 of 16 v2 1/8] meta: classes: use base.bbclass from bitbake

[Henning Schild <henning.schild@siemens.com>]

Am Tue, 8 Aug 2017 17:02:19 +0300
schrieb Alexander Smirnov <alex.bluesman.smirnov@gmail.com>:

> 2017-08-08 11:43 GMT+03:00 Henning Schild
> <henning.schild@siemens.com>:
> 
> > Am Mon, 7 Aug 2017 22:18:36 +0300
> > schrieb Alexander Smirnov <alex.bluesman.smirnov@gmail.com>:
> >  
> > > Hi again,
> > >
> > > it looks like this patch doesn't work for Isar. 'base.bbclass' is
> > > the one which is included by default to any bitbake operation. In
> > > the default 'base.bbclass' there is one very bad line:
> > >
> > > bitbake/classes/base.bbclass:
> > > 51: do_build[nostamp] = "1"
> > >
> > > This line disables stamps for build tasks, it completely destroys
> > > the pipeline. In your patch you move the line: do_build[nostamp]
> > > = "0" to 'isar-base.bbclass'. But to have this construction
> > > working you have to explicitely include this class to *every*
> > > recipe in Isar, what looks for me as huge overhead.  
> >
> > Ok i did not know what that line does but since i moved it to
> > isar-base and every class and recipe includes that, the semantics
> > is kept.
> >
> > Making sure local changes are always included is less of a problem
> > than copying files from bitbake and ending up with an outdated copy
> > again. 
> 
> We are speaking about sample base class in bitbake tree which by
> default provides *nothing*. As I can understand, this is just an
> example which doesn't affect the bitbake functionality. Nor OE, nor
> Yocto doesn't use it and implement their own base classes. Also in
> bitbake manual:
> https://www.yoctoproject.org/docs/1.6/bitbake-user-manual/bitbake-user-manual.html
> chapter A4, document proposes you to create your base class.
> 
> I was OK with this idea because I assumed that the deafult base class
> provides correct settings, but now I realized that it doesn't. So
> keeping broken base class for some reasons and overwrite it in
> run-time looks too compicated for such simple things.
> 
> 
> > Every recipe in isar will probably include dpdk-bin or dpdk-src,
> > which both inherit isar-base. The images also include isar-base.
> >  
> 
> It's not a question about workaround. It's about knowingly keeping of
> broken base class and fix it from another class. Meanwhile nothing is
> used from original base class. I don't really understand why we need
> base class then...

It provides 'listtasks' and 'showdata' they never used to work but are
handy now that they do. And the logging functions ...

Henning

> Alex
> 
> 
> >
> > Henning
> >  
> > > 2017-08-03 14:55 GMT+03:00 Henning Schild
> > > <henning.schild@siemens.com>:
> > >  
> > > > Do not use our own copy of that class and move local deviations
> > > > into isar-base. That also fixes the default tasks "showdata" and
> > > > "listtasks", which probably never worked.
> > > >
> > > > Signed-off-by: Henning Schild <henning.schild@siemens.com>
> > > > ---
> > > >  meta/classes/base.bbclass      | 81
> > > > +----------------------------- ------------
> > > >  meta/classes/dpkg.bbclass      |  2 ++
> > > >  meta/classes/ext4-img.bbclass  |  2 ++
> > > >  meta/classes/isar-base.bbclass | 36 +++++++++++++++++++
> > > >  4 files changed, 41 insertions(+), 80 deletions(-)
> > > >  mode change 100644 => 120000 meta/classes/base.bbclass
> > > >  create mode 100644 meta/classes/isar-base.bbclass
> > > >
> > > > diff --git a/meta/classes/base.bbclass
> > > > b/meta/classes/base.bbclass deleted file mode 100644
> > > > index 48b6bac..0000000
> > > > --- a/meta/classes/base.bbclass
> > > > +++ /dev/null
> > > > @@ -1,80 +0,0 @@
> > > > -# Copyright (C) 2003  Chris Larson
> > > > -#
> > > > -# Permission is hereby granted, free of charge, to any person
> > > > obtaining a -# copy of this software and associated
> > > > documentation files (the "Software"),
> > > > -# to deal in the Software without restriction, including
> > > > without limitation
> > > > -# the rights to use, copy, modify, merge, publish, distribute,
> > > > sublicense, -# and/or sell copies of the Software, and to permit
> > > > persons to whom the -# Software is furnished to do so, subject
> > > > to the following conditions: -#
> > > > -# The above copyright notice and this permission notice shall
> > > > be included -# in all copies or substantial portions of the
> > > > Software. -#
> > > > -# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY
> > > > KIND, EXPRESS OR
> > > > -# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > > > MERCHANTABILITY, -# FITNESS FOR A PARTICULAR PURPOSE AND
> > > > NONINFRINGEMENT. IN NO EVENT SHALL -# THE AUTHORS OR COPYRIGHT
> > > > HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR -# OTHER LIABILITY,
> > > > WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, -# ARISING
> > > > FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR -#
> > > > OTHER DEALINGS IN THE SOFTWARE. -
> > > > -THISDIR = "${@os.path.dirname(d.getVar('FILE', True))}"
> > > > -
> > > > -die() {
> > > > -       bbfatal "$*"
> > > > -}
> > > > -
> > > > -bbnote() {
> > > > -       echo "NOTE:" "$*"
> > > > -}
> > > > -
> > > > -bbwarn() {
> > > > -       echo "WARNING:" "$*"
> > > > -}
> > > > -
> > > > -bbfatal() {
> > > > -       echo "FATAL:" "$*"
> > > > -       exit 1
> > > > -}
> > > > -
> > > > -bbdebug() {
> > > > -       test $# -ge 2 || {
> > > > -               echo "Usage: bbdebug level \"message\""
> > > > -               exit 1
> > > > -       }
> > > > -
> > > > -       test ${@bb.msg.debug_level['default']} -ge $1 && {
> > > > -               shift
> > > > -               echo "DEBUG:" $*
> > > > -       }
> > > > -}
> > > > -
> > > > -addtask showdata
> > > > -do_showdata[nostamp] = "1"
> > > > -python do_showdata() {
> > > > -       import sys
> > > > -       # emit variables and shell functions
> > > > -       bb.data.emit_env(sys.__stdout__, d, True)
> > > > -       # emit the metadata which isnt valid shell
> > > > -       for e in bb.data.keys(d):
> > > > -               if bb.data.getVarFlag(e, 'python', d):
> > > > -                       sys.__stdout__.write("\npython %s ()
> > > > {\n%s}\n" % (e, bb.data.getVar(e, d, 1)))
> > > > -}
> > > > -
> > > > -addtask listtasks
> > > > -do_listtasks[nostamp] = "1"
> > > > -python do_listtasks() {
> > > > -       import sys
> > > > -       for e in bb.data.keys(d):
> > > > -               if bb.data.getVarFlag(e, 'task', d):
> > > > -                       sys.__stdout__.write("%s\n" % e)
> > > > -}
> > > > -
> > > > -addtask build
> > > > -do_build[dirs] = "${TOPDIR}"
> > > > -python base_do_build () {
> > > > -       bb.note("The included, default BB base.bbclass does not
> > > > define a useful default task.")
> > > > -       bb.note("Try running the 'listtasks' task against a .bb
> > > > to see what tasks are defined.")
> > > > -}
> > > > -
> > > > -EXPORT_FUNCTIONS do_clean do_mrproper do_build
> > > > diff --git a/meta/classes/base.bbclass
> > > > b/meta/classes/base.bbclass new file mode 120000
> > > > index 0000000..11fe0a4
> > > > --- /dev/null
> > > > +++ b/meta/classes/base.bbclass
> > > > @@ -0,0 +1 @@
> > > > +../../bitbake/classes/base.bbclass
> > > > \ No newline at end of file
> > > > diff --git a/meta/classes/dpkg.bbclass
> > > > b/meta/classes/dpkg.bbclass index a0446d7..3d7aafb 100644
> > > > --- a/meta/classes/dpkg.bbclass
> > > > +++ b/meta/classes/dpkg.bbclass
> > > > @@ -1,6 +1,8 @@
> > > >  # This software is a part of ISAR.
> > > >  # Copyright (C) 2015-2016 ilbers GmbH
> > > >
> > > > +inherit isar-base
> > > > +
> > > >  # Add dependency from buildchroot creation
> > > >  DEPENDS += "buildchroot"
> > > >  do_unpack[deptask] = "do_build"
> > > > diff --git a/meta/classes/ext4-img.bbclass
> > > > b/meta/classes/ext4-img.bbclass index 65d4c11..1486c3e 100644
> > > > --- a/meta/classes/ext4-img.bbclass
> > > > +++ b/meta/classes/ext4-img.bbclass
> > > > @@ -1,6 +1,8 @@
> > > >  # This software is a part of ISAR.
> > > >  # Copyright (C) 2015-2016 ilbers GmbH
> > > >
> > > > +inherit isar-base
> > > > +
> > > >  # Extra space for rootfs in MB
> > > >  ROOTFS_EXTRA ?= "64"
> > > >
> > > > diff --git a/meta/classes/isar-base.bbclass
> > > > b/meta/classes/isar-base.bbcla ss
> > > > new file mode 100644
> > > > index 0000000..67d38a1
> > > > --- /dev/null
> > > > +++ b/meta/classes/isar-base.bbclass
> > > > @@ -0,0 +1,36 @@
> > > > +# This software is a part of ISAR.
> > > > +# Copyright (C) 2017  Siemens AG
> > > > +#
> > > > +# Permission is hereby granted, free of charge, to any person
> > > > obtaining a +# copy of this software and associated
> > > > documentation files (the "Software"),
> > > > +# to deal in the Software without restriction, including
> > > > without limitation
> > > > +# the rights to use, copy, modify, merge, publish, distribute,
> > > > sublicense, +# and/or sell copies of the Software, and to permit
> > > > persons to whom the +# Software is furnished to do so, subject
> > > > to the following conditions: +#
> > > > +# The above copyright notice and this permission notice shall
> > > > be included +# in all copies or substantial portions of the
> > > > Software. +#
> > > > +# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY
> > > > KIND, EXPRESS OR
> > > > +# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > > > MERCHANTABILITY, +# FITNESS FOR A PARTICULAR PURPOSE AND
> > > > NONINFRINGEMENT. IN NO EVENT SHALL +# THE AUTHORS OR COPYRIGHT
> > > > HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR +# OTHER LIABILITY,
> > > > WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, +# ARISING
> > > > FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR +#
> > > > OTHER DEALINGS IN THE SOFTWARE. +
> > > > +THISDIR = "${@os.path.dirname(d.getVar('FILE', True))}"
> > > > +
> > > > +bbdebug() {
> > > > +       test $# -ge 2 || {
> > > > +               echo "Usage: bbdebug level \"message\""
> > > > +               exit 1
> > > > +       }
> > > > +
> > > > +       test ${@bb.msg.debug_level['default']} -ge $1 && {
> > > > +               shift
> > > > +               echo "DEBUG:" $*
> > > > +       }
> > > > +}
> > > > +
> > > > +do_build[nostamp] = "0"
> > > > --
> > > > 2.13.0
> > > >
> > > >  
> >
> >