<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>12258</bug_id>
          
          <creation_ts>2017-10-20 13:42:54 +0000</creation_ts>
          <short_desc>LICENSE in packagegroup?</short_desc>
          <delta_ts>2017-11-06 17:38:52 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>core</component>
          <version>2.4</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>2.5</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Robert Berger">pokylinux</reporter>
          <assigned_to name="Ross Burton">ross.burton</assigned_to>
          <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>ross.burton</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>77730</commentid>
    <comment_count>0</comment_count>
    <who name="Robert Berger">pokylinux</who>
    <bug_when>2017-10-20 13:42:54 +0000</bug_when>
    <thetext>in poky:

$find | grep packagegroup | grep .bb | xargs grep LICENSE

./meta/classes/packagegroup.bbclass:LICENSE ?= &quot;MIT&quot;
./meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb:LICENSE = &quot;MIT&quot;

I am not sure what meaning e.g. in nativesdk-packagegroup-sdk-host.bb the LICENSE field has.

I guess it could be removed.

Also please check the bbclass. Is LICENSE needed there?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77733</commentid>
    <comment_count>1</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2017-10-20 14:25:23 +0000</bug_when>
    <thetext>All recipes need a license and the default is INVALID so the class sets a default value that is likely good (as packagegroups are just metapackages).

Yes the LICENS in sdk-host can be removed as it is redundant.  Want to send a patch?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77828</commentid>
    <comment_count>2</comment_count>
    <who name="Stephen K Jolley">sjolley.yp.pm</who>
    <bug_when>2017-10-26 14:36:31 +0000</bug_when>
    <thetext>Please answer Ross&apos;s question.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77837</commentid>
    <comment_count>3</comment_count>
      <attachid>4105</attachid>
    <who name="Robert Berger">pokylinux</who>
    <bug_when>2017-10-26 15:31:42 +0000</bug_when>
    <thetext>Created attachment 4105
proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77855</commentid>
    <comment_count>4</comment_count>
    <who name="Stephen K Jolley">sjolley.yp.pm</who>
    <bug_when>2017-10-27 18:27:55 +0000</bug_when>
    <thetext>Ross,

It appears the question has been answered.  If not please restate the question and put it back in NEEDINFO.

Thanks,</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77861</commentid>
    <comment_count>5</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2017-10-30 12:00:05 +0000</bug_when>
    <thetext>Patches go to the mailing list, please.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77862</commentid>
    <comment_count>6</comment_count>
    <who name="Robert Berger">pokylinux</who>
    <bug_when>2017-10-30 13:52:25 +0000</bug_when>
    <thetext>Just sent it to the mailing list.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77950</commentid>
    <comment_count>7</comment_count>
    <who name="Robert Berger">pokylinux</who>
    <bug_when>2017-11-03 17:08:45 +0000</bug_when>
    <thetext>It made it into oe patchwork:

https://patchwork.openembedded.org/series/9539/

Now it&apos;s resting there;)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77997</commentid>
    <comment_count>8</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2017-11-06 17:38:52 +0000</bug_when>
    <thetext>Merged in oe-core c04ae17f439ffd5fd70d8564430a94582e2cf688.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>4105</attachid>
            <date>2017-10-26 15:31:42 +0000</date>
            <delta_ts>2017-10-26 15:31:42 +0000</delta_ts>
            <desc>proposed patch</desc>
            <filename>0001-poky-LICENSE-in-packagegroup-makes-no-sense-removed.patch</filename>
            <type>text/plain</type>
            <size>920</size>
            <attacher name="Robert Berger">pokylinux</attacher>
            
              <data encoding="base64">RnJvbSA2NDExNzRhZTNhMzQ1M2FjMDRhMzI1OGZlMTNkNDJlZmJlNDk2ZjllIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBSb2JlcnQgQmVyZ2VyIDxyb2JlcnQuYmVyZ2VyQFJlbGlhYmxl
RW1iZWRkZWRTeXN0ZW1zLmNvbT4KRGF0ZTogVGh1LCAyNiBPY3QgMjAxNyAxODoyOToyNSArMDMw
MApTdWJqZWN0OiBbUEFUQ0hdIHBva3k6IExJQ0VOU0UgaW4gcGFja2FnZWdyb3VwIG1ha2VzIG5v
IHNlbnNlIC0gcmVtb3ZlZAoKU2lnbmVkLW9mZi1ieTogUm9iZXJ0IEJlcmdlciA8cm9iZXJ0LmJl
cmdlckBSZWxpYWJsZUVtYmVkZGVkU3lzdGVtcy5jb20+Ci0tLQogbWV0YS9yZWNpcGVzLWNvcmUv
cGFja2FnZWdyb3Vwcy9uYXRpdmVzZGstcGFja2FnZWdyb3VwLXNkay1ob3N0LmJiIHwgMSAtCiAx
IGZpbGUgY2hhbmdlZCwgMSBkZWxldGlvbigtKQoKZGlmZiAtLWdpdCBhL21ldGEvcmVjaXBlcy1j
b3JlL3BhY2thZ2Vncm91cHMvbmF0aXZlc2RrLXBhY2thZ2Vncm91cC1zZGstaG9zdC5iYiBiL21l
dGEvcmVjaXBlcy1jb3JlL3BhY2thZ2Vncm91cHMvbmF0aXZlc2RrLXBhY2thZ2Vncm91cC1zZGst
aG9zdC5iYgppbmRleCAyY2E2MzkyLi5hZWU0YTAzIDEwMDY0NAotLS0gYS9tZXRhL3JlY2lwZXMt
Y29yZS9wYWNrYWdlZ3JvdXBzL25hdGl2ZXNkay1wYWNrYWdlZ3JvdXAtc2RrLWhvc3QuYmIKKysr
IGIvbWV0YS9yZWNpcGVzLWNvcmUvcGFja2FnZWdyb3Vwcy9uYXRpdmVzZGstcGFja2FnZWdyb3Vw
LXNkay1ob3N0LmJiCkBAIC00LDcgKzQsNiBAQAogCiBTVU1NQVJZID0gIkhvc3QgcGFja2FnZXMg
Zm9yIHRoZSBzdGFuZGFsb25lIFNESyBvciBleHRlcm5hbCB0b29sY2hhaW4iCiBQUiA9ICJyMTIi
Ci1MSUNFTlNFID0gIk1JVCIKIAogaW5oZXJpdCBwYWNrYWdlZ3JvdXAgbmF0aXZlc2RrCiAKLS0g
CjIuNy40Cgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>