<?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>5725</bug_id>
          
          <creation_ts>2014-01-14 17:56:55 +0000</creation_ts>
          <short_desc>Add QA check for installing Python Eggs</short_desc>
          <delta_ts>2023-04-25 09:45:32 +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>1.6</version>
          <rep_platform>All</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>OBSOLETE</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard>NEWCOMER</status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>4.99</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ross Burton">ross.burton</reporter>
          <assigned_to name="Loc Nguyen">loc.nguyen.wt</assigned_to>
          <cc>akuster808</cc>
    
    <cc>loc.nguyen.wt</cc>
    
    <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>mhalstead</cc>
    
    <cc>randy.macleod</cc>
    
    <cc>richard.purdie</cc>
    
    <cc>tim.orling</cc>
    
    <cc>tvgamblin</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>39807</commentid>
    <comment_count>0</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2014-01-14 17:56:55 +0000</bug_when>
    <thetext>Packages that install Python modules shouldn&apos;t be installing Eggs, as distro-managed modules should be &quot;bare&quot;.  It shouldn&apos;t be hard to detect this happening in a sanity test, and warning the user.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>93374</commentid>
    <comment_count>1</comment_count>
    <who name="akuster">akuster808</who>
    <bug_when>2022-06-02 14:51:37 +0000</bug_when>
    <thetext>Since we moved to wheels, is this still needed?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>93401</commentid>
    <comment_count>2</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2022-06-04 17:47:17 +0000</bug_when>
    <thetext>Might be worth having the test still, to catch people installing with legacy stuff.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>95290</commentid>
    <comment_count>3</comment_count>
    <who name="Loc Nguyen">loc.nguyen.wt</who>
    <bug_when>2023-04-05 03:07:15 +0000</bug_when>
    <thetext>Dear all,
I&apos;m a newcomer, I have submitted a patch following instruction at: http://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded
Please help to review.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>95291</commentid>
    <comment_count>4</comment_count>
      <attachid>4944</attachid>
    <who name="Loc Nguyen">loc.nguyen.wt</who>
    <bug_when>2023-04-05 03:07:55 +0000</bug_when>
    <thetext>Created attachment 4944
proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>95376</commentid>
    <comment_count>5</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2023-04-18 14:16:51 +0000</bug_when>
    <thetext>I can&apos;t see the patch on the list, did you submit it to openembedded-core@lists.openembedded.org?

I&apos;m not sure that patch will work, as it depends on the recipe explicitly RDEPENDing on python3-eggs.  A recipe which is installing an egg inadvertently won&apos;t do that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>95449</commentid>
    <comment_count>6</comment_count>
    <who name="Loc Nguyen">loc.nguyen.wt</who>
    <bug_when>2023-04-25 04:18:19 +0000</bug_when>
    <thetext>Hi Mr. Ross Burton

&gt; I&apos;m not sure that patch will work,(In reply to Ross Burton from comment #5)
&gt; I can&apos;t see the patch on the list, did you submit it to
&gt; openembedded-core@lists.openembedded.org?
I&apos;ve submitted to openembedded-core@lists.openembedded.org.
You can search for the mail subject: setuptools3: adding sanity check to detect python-eggs install

&gt; I&apos;m not sure that patch will work, as it depends on the recipe explicitly
&gt; RDEPENDing on python3-eggs.  A recipe which is installing an egg
&gt; inadvertently won&apos;t do that.
I&apos;m sorry, but can you give me an example. My understanding is:
- Each python module will be managed by a separate recipe. 
- To install python-eggs, user will specify IMAGE_INSTALL += &apos; python-eggs&apos;, 
- or it will be installed when RDEPENDing of a recipe have &apos;python-eggs&apos;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>95454</commentid>
    <comment_count>7</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2023-04-25 09:45:32 +0000</bug_when>
    <thetext>&gt; I&apos;m sorry, but can you give me an example. My understanding is:
&gt; - Each python module will be managed by a separate recipe. 
&gt; - To install python-eggs, user will specify IMAGE_INSTALL += &apos; python-eggs&apos;, 
&gt; - or it will be installed when RDEPENDing of a recipe have &apos;python-eggs&apos;

That&apos;s not right.

A &apos;python egg&apos; is .zip file which contains a module (where as a bare module is just a directory of files). There is no file called &apos;python-eggs&apos;.

However since this bug was created the Python world has moved on a lot and I&apos;m not convinced it&apos;s worth checking for eggs anymore.  They were used by easy_install which has been entirely replaced by pip.

Sorry, but I think I&apos;ll close this: the check will be obsolete.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>4944</attachid>
            <date>2023-04-05 03:07:55 +0000</date>
            <delta_ts>2023-04-05 03:07:55 +0000</delta_ts>
            <desc>proposed patch</desc>
            <filename>0001-setuptools3-adding-sanity-check-to-detect-python-egg.patch</filename>
            <type>text/plain</type>
            <size>1419</size>
            <attacher name="Loc Nguyen">loc.nguyen.wt</attacher>
            
              <data encoding="base64">RnJvbSA0MWQ0OTBjNTFhOTdiZDdhYWRlMjI2YzE0NTlkMzIzOWJkNTYxNDJjIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBMb2MgTmd1eWVuIDxsb2Mubmd1eWVuLnd0QHJlbmVzYXMuY29t
PgpEYXRlOiBGcmksIDE3IE1hciAyMDIzIDEwOjQ1OjA4ICswNzAwClN1YmplY3Q6IFtQQVRDSF0g
c2V0dXB0b29sczM6IGFkZGluZyBzYW5pdHkgY2hlY2sgdG8gZGV0ZWN0IHB5dGhvbi1lZ2dzCiBp
bnN0YWxsCgpQYWNrYWdlcyB0aGF0IGluc3RhbGwgUHl0aG9uIG1vZHVsZXMgc2hvdWxkbid0IGJl
IGluc3RhbGxpbmcgRWdncywKYXMgZGlzdHJvLW1hbmFnZWQgbW9kdWxlcyBzaG91bGQgYmUgImJh
cmUiCgpBZGRlZCBhIHNhbml0eSBjaGVjayB0byB2ZXJpZnkgaWYgcHl0aG9uIG1vZHVsZXMgd2ls
bCBpbnN0YWxsIEVnZ3MuCklmIHllcywgZGlzcGxheSBhIHdhcm5pbmcgbWVzc2FnZSB0byB1c2Vy
LgoKU2VlIFtZT0NUTyAjNTcyNV0gZm9yIG1vcmUgZGV0YWlscwoKU2lnbmVkLW9mZi1ieTogTG9j
IE5ndXllbiA8bG9jLm5ndXllbi53dEByZW5lc2FzLmNvbT4KLS0tCiBtZXRhL2NsYXNzZXMtcmVj
aXBlL3NldHVwdG9vbHMzLmJiY2xhc3MgfCAxMCArKysrKysrKysrCiAxIGZpbGUgY2hhbmdlZCwg
MTAgaW5zZXJ0aW9ucygrKQoKZGlmZiAtLWdpdCBhL21ldGEvY2xhc3Nlcy1yZWNpcGUvc2V0dXB0
b29sczMuYmJjbGFzcyBiL21ldGEvY2xhc3Nlcy1yZWNpcGUvc2V0dXB0b29sczMuYmJjbGFzcwpp
bmRleCA0YzZlNzllZTlhLi41ZDhlZmE4NmVhIDEwMDY0NAotLS0gYS9tZXRhL2NsYXNzZXMtcmVj
aXBlL3NldHVwdG9vbHMzLmJiY2xhc3MKKysrIGIvbWV0YS9jbGFzc2VzLXJlY2lwZS9zZXR1cHRv
b2xzMy5iYmNsYXNzCkBAIC0zMyw2ICszMywxNiBAQCBzZXR1cHRvb2xzM19kb19pbnN0YWxsKCkg
ewogICAgICAgICBweXRob25fcGVwNTE3X2RvX2luc3RhbGwKIH0KIAorcHl0aG9uIGRvX3Nhbml0
eV9jaGVjaygpIHsKKyAgICAjIFB5dGhvbiBtb2R1bGVzIHNob3VsZG4ndCBiZSBpbnN0YWxsaW5n
IEVnZ3MsIGFzIGRpc3Ryby1tYW5hZ2VkIG1vZHVsZXMgc2hvdWxkIGJlICJiYXJlIgorICAgIHBr
Z05hbWUgPSBkLmdldFZhcignQlBOJykKKyAgICByZGVwcyA9IGQuZ2V0VmFyKCdSREVQRU5EU18n
ICsgcGtnTmFtZSkgb3IgIiIKKyAgICBpZiAoJ3B5dGhvbi1lZ2dzJyBpbiByZGVwcykgb3IgKCdw
eXRob24zLWVnZ3MnIGluIHJkZXBzKToKKyAgICAgICAgYmIud2Fybihwa2dOYW1lICsgJyBpcyBp
bnN0YWxsaW5nIEVnZ3MuJykKK30KKworYWRkdGFzayBzYW5pdHlfY2hlY2sgYmVmb3JlIGRvX2Zl
dGNoCisKIEVYUE9SVF9GVU5DVElPTlMgZG9fY29uZmlndXJlIGRvX2NvbXBpbGUgZG9faW5zdGFs
bAogCiBleHBvcnQgTERTSEFSRUQ9IiR7Q0NMRH0gLXNoYXJlZCIKLS0gCjIuMjUuMQoK
</data>

          </attachment>
      

    </bug>

</bugzilla>