All of lore.kernel.org
 help / color / mirror / Atom feed
* [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0
@ 2021-07-12  7:56 wangmy
  2021-07-15 14:43 ` Martin Jansa
  0 siblings, 1 reply; 8+ messages in thread
From: wangmy @ 2021-07-12  7:56 UTC (permalink / raw)
  To: openembedded-core; +Cc: Wang Mingyu

reproducibility.patch
deleted since it's been merged upstream.

v57.1.0
-------

Changes
^^^^^^^
* #2692: Globs are now sorted in 'license_files' restoring reproducibility by eliminating variance from disk order.
* #2714: Update to distutils at pypa/distutils@e2627b7.
* #2715: Removed reliance on deprecated ssl.match_hostname by removing the ssl support. Now any index operations rely on the native SSL implementation.

Documentation changes
^^^^^^^^^^^^^^^^^^^^^
* #2604: Revamped the backward/cross tool compatibility section to remove
  some confusion.
  Add some examples and the version since when ``entry_points`` are
  supported in declarative configuration.
  Tried to make the reading flow a bit leaner, gather some informations
  that were a bit dispersed.

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
---
 .../python3-setuptools/reproducibility.patch  | 40 -------------------
 ...57.0.0.bb => python3-setuptools_57.1.0.bb} |  5 +--
 2 files changed, 2 insertions(+), 43 deletions(-)
 delete mode 100644 meta/recipes-devtools/python/python3-setuptools/reproducibility.patch
 rename meta/recipes-devtools/python/{python3-setuptools_57.0.0.bb => python3-setuptools_57.1.0.bb} (88%)

diff --git a/meta/recipes-devtools/python/python3-setuptools/reproducibility.patch b/meta/recipes-devtools/python/python3-setuptools/reproducibility.patch
deleted file mode 100644
index 149d8ad5ce..0000000000
--- a/meta/recipes-devtools/python/python3-setuptools/reproducibility.patch
+++ /dev/null
@@ -1,40 +0,0 @@
-The License-File lines in PKG-INFO change ordering depending on the order on disk, 
-for example for python-packaging, one build shows:
-
-License-File: LICENSE
-License-File: LICENSE.APACHE
-License-File: LICENSE.BSD
-
-and the other shows:
-
-License-File: LICENSE
-License-File: LICENSE.BSD
-License-File: LICENSE.APACHE
-
-This is because glob uses os.listdir() which is unsorted. Sort the result to avoid this.
-
-Upstream-Status: Submitted [https://github.com/pypa/setuptools/issues/2691]
-Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-
-Index: setuptools-57.0.0/setuptools/dist.py
-===================================================================
---- setuptools-57.0.0.orig/setuptools/dist.py
-+++ setuptools-57.0.0/setuptools/dist.py
-@@ -15,7 +15,7 @@ import distutils.command
- from distutils.util import strtobool
- from distutils.debug import DEBUG
- from distutils.fancy_getopt import translate_longopt
--from glob import iglob
-+from glob import glob
- import itertools
- import textwrap
- from typing import List, Optional, TYPE_CHECKING
-@@ -603,7 +603,7 @@ class Distribution(_Distribution):
-         return (
-             path
-             for pattern in patterns
--            for path in iglob(pattern)
-+            for path in sorted(glob(pattern))
-             if not path.endswith('~')
-             and os.path.isfile(path)
-         )
diff --git a/meta/recipes-devtools/python/python3-setuptools_57.0.0.bb b/meta/recipes-devtools/python/python3-setuptools_57.1.0.bb
similarity index 88%
rename from meta/recipes-devtools/python/python3-setuptools_57.0.0.bb
rename to meta/recipes-devtools/python/python3-setuptools_57.1.0.bb
index a15b51e31c..c066d8209f 100644
--- a/meta/recipes-devtools/python/python3-setuptools_57.0.0.bb
+++ b/meta/recipes-devtools/python/python3-setuptools_57.1.0.bb
@@ -8,10 +8,9 @@ inherit pypi setuptools3
 
 SRC_URI_append_class-native = " file://0001-conditionally-do-not-fetch-code-by-easy_install.patch"
 
-SRC_URI += "file://0001-change-shebang-to-python3.patch \
-            file://reproducibility.patch"
+SRC_URI += "file://0001-change-shebang-to-python3.patch"
 
-SRC_URI[sha256sum] = "401cbf33a7bf817d08014d51560fc003b895c4cdc1a5b521ad2969e928a07535"
+SRC_URI[sha256sum] = "cfca9c97e7eebbc8abe18d5e5e962a08dcad55bb63afddd82d681de4d22a597b"
 
 DEPENDS += "${PYTHON_PN}"
 
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0
  2021-07-12  7:56 [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0 wangmy
@ 2021-07-15 14:43 ` Martin Jansa
  2021-07-15 17:22   ` Khem Raj
  0 siblings, 1 reply; 8+ messages in thread
From: Martin Jansa @ 2021-07-15 14:43 UTC (permalink / raw)
  To: wangmy, Leon Anavi, Derek Straka, Trevor Gamblin
  Cc: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 1815 bytes --]

On Mon, Jul 12, 2021 at 9:57 AM wangmy <wangmy@fujitsu.com> wrote:

> reproducibility.patch
> deleted since it's been merged upstream.
>
> v57.1.0
> -------
>
> Changes
> ^^^^^^^
> * #2692: Globs are now sorted in 'license_files' restoring reproducibility
> by eliminating variance from disk order.
> * #2714: Update to distutils at pypa/distutils@e2627b7.
> * #2715: Removed reliance on deprecated ssl.match_hostname by removing the
> ssl support. Now any index operations rely on the native SSL implementation.
>

FWIW: this change
breaks  meta-openembedded/meta-python/recipes-devtools/python/python3-cassandra-driver_3.25.0.bb:
do_install:

+ people who might be interested in python3-cassandra-driver.

| Installed
/jenkins/mjansa/build/ros/ros2-rolling-honister/tmp-glibc/work/core2-64-oe-linux/python3-cassandra-driver/3.25.0-r0/recipe-sysroot-native/usr/lib/python3.9/site-packages/cassandra_driver-3.25.0-py3.9-linux-x86_64.egg
| Processing dependencies for cassandra-driver==3.25.0
| Searching for geomet<0.3,>=0.1
| Reading https://pypi.org/simple/geomet/
| Download error on https://pypi.org/simple/geomet/: [SSL:
CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local
issuer certificate (_ssl.c:1129) -- Some packages may not be found!
| Couldn't find index page for 'geomet' (maybe misspelled?)
| Scanning index of all packages (this may take a while)
| Reading https://pypi.org/simple/
| Download error on https://pypi.org/simple/: [SSL:
CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local
issuer certificate (_ssl.c:1129) -- Some packages may not be found!
| No local packages or working download links found for geomet<0.3,>=0.1
| error: Could not find suitable distribution for
Requirement.parse('geomet<0.3,>=0.1')

[-- Attachment #2: Type: text/html, Size: 2420 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0
  2021-07-15 14:43 ` Martin Jansa
@ 2021-07-15 17:22   ` Khem Raj
  2021-07-15 17:57     ` Konrad Weihmann
  0 siblings, 1 reply; 8+ messages in thread
From: Khem Raj @ 2021-07-15 17:22 UTC (permalink / raw)
  To: Martin Jansa, wangmy, Leon Anavi, Derek Straka, Trevor Gamblin
  Cc: Patches and discussions about the oe-core layer



On 7/15/21 7:43 AM, Martin Jansa wrote:
> On Mon, Jul 12, 2021 at 9:57 AM wangmy <wangmy@fujitsu.com 
> <mailto:wangmy@fujitsu.com>> wrote:
> 
>     reproducibility.patch
>     deleted since it's been merged upstream.
> 
>     v57.1.0
>     -------
> 
>     Changes
>     ^^^^^^^
>     * #2692: Globs are now sorted in 'license_files' restoring
>     reproducibility by eliminating variance from disk order.
>     * #2714: Update to distutils at pypa/distutils@e2627b7.
>     * #2715: Removed reliance on deprecated ssl.match_hostname by
>     removing the ssl support. Now any index operations rely on the
>     native SSL implementation.
> 
> 
> FWIW: this change 
> breaks  meta-openembedded/meta-python/recipes-devtools/python/python3-cassandra-driver_3.25.0.bb:do_install:
> 
> + people who might be interested in python3-cassandra-driver.
> 
> | Installed 
> /jenkins/mjansa/build/ros/ros2-rolling-honister/tmp-glibc/work/core2-64-oe-linux/python3-cassandra-driver/3.25.0-r0/recipe-sysroot-native/usr/lib/python3.9/site-packages/cassandra_driver-3.25.0-py3.9-linux-x86_64.egg
> | Processing dependencies for cassandra-driver==3.25.0
> | Searching for geomet<0.3,>=0.1
> | Reading https://pypi.org/simple/geomet/ <https://pypi.org/simple/geomet/>
> | Download error on https://pypi.org/simple/geomet/ 
> <https://pypi.org/simple/geomet/>: [SSL: CERTIFICATE_VERIFY_FAILED] 
> certificate verify failed: unable to get local issuer certificate 
> (_ssl.c:1129) -- Some packages may not be found!
> | Couldn't find index page for 'geomet' (maybe misspelled?)
> | Scanning index of all packages (this may take a while)
> | Reading https://pypi.org/simple/ <https://pypi.org/simple/>
> | Download error on https://pypi.org/simple/ <https://pypi.org/simple/>: 
> [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to 
> get local issuer certificate (_ssl.c:1129) -- Some packages may not be 
> found!
> | No local packages or working download links found for geomet<0.3,>=0.1
> | error: Could not find suitable distribution for 
> Requirement.parse('geomet<0.3,>=0.1')
> 
> 

yes its reproduced on AB as well.
https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/1397

> 
> 
> 
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0
  2021-07-15 17:22   ` Khem Raj
@ 2021-07-15 17:57     ` Konrad Weihmann
  2021-07-15 18:40       ` Trevor Gamblin
  2021-07-15 18:55       ` Khem Raj
  0 siblings, 2 replies; 8+ messages in thread
From: Konrad Weihmann @ 2021-07-15 17:57 UTC (permalink / raw)
  To: Khem Raj, Martin Jansa, wangmy, Leon Anavi, Derek Straka, Trevor Gamblin
  Cc: Patches and discussions about the oe-core layer

On 15.07.21 19:22, Khem Raj wrote:
> 
> 
> On 7/15/21 7:43 AM, Martin Jansa wrote:
>> On Mon, Jul 12, 2021 at 9:57 AM wangmy <wangmy@fujitsu.com 
>> <mailto:wangmy@fujitsu.com>> wrote:
>>
>>     reproducibility.patch
>>     deleted since it's been merged upstream.
>>
>>     v57.1.0
>>     -------
>>
>>     Changes
>>     ^^^^^^^
>>     * #2692: Globs are now sorted in 'license_files' restoring
>>     reproducibility by eliminating variance from disk order.
>>     * #2714: Update to distutils at pypa/distutils@e2627b7.
>>     * #2715: Removed reliance on deprecated ssl.match_hostname by
>>     removing the ssl support. Now any index operations rely on the
>>     native SSL implementation.
>>
>>
>> FWIW: this change 
>> breaks  meta-openembedded/meta-python/recipes-devtools/python/python3-cassandra-driver_3.25.0.bb:do_install: 
>>
>>
>> + people who might be interested in python3-cassandra-driver.
>>
>> | Installed 
>> /jenkins/mjansa/build/ros/ros2-rolling-honister/tmp-glibc/work/core2-64-oe-linux/python3-cassandra-driver/3.25.0-r0/recipe-sysroot-native/usr/lib/python3.9/site-packages/cassandra_driver-3.25.0-py3.9-linux-x86_64.egg 
>>
>> | Processing dependencies for cassandra-driver==3.25.0
>> | Searching for geomet<0.3,>=0.1
>> | Reading https://pypi.org/simple/geomet/ 
>> <https://pypi.org/simple/geomet/>
>> | Download error on https://pypi.org/simple/geomet/ 
>> <https://pypi.org/simple/geomet/>: [SSL: CERTIFICATE_VERIFY_FAILED] 
>> certificate verify failed: unable to get local issuer certificate 
>> (_ssl.c:1129) -- Some packages may not be found!
>> | Couldn't find index page for 'geomet' (maybe misspelled?)
>> | Scanning index of all packages (this may take a while)
>> | Reading https://pypi.org/simple/ <https://pypi.org/simple/>
>> | Download error on https://pypi.org/simple/ 
>> <https://pypi.org/simple/>: [SSL: CERTIFICATE_VERIFY_FAILED] 
>> certificate verify failed: unable to get local issuer certificate 
>> (_ssl.c:1129) -- Some packages may not be found!
>> | No local packages or working download links found for geomet<0.3,>=0.1

But that not directly an issue of the setuptools upgrade, but a not well 
written recipe here.
I'm more than lucky that this update revealed the issue, because 
otherwise no one would have really noticed that the original recipe of 
cassandra-driver is pulling sources while installing.
So just in case I would rather pull in this upgrade than to mind the 
broken recipe from meta-python here (just my 2c)

>> | error: Could not find suitable distribution for 
>> Requirement.parse('geomet<0.3,>=0.1')
>>
>>
> 
> yes its reproduced on AB as well.
> https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/1397
> 
>>
>>
>>
>>
> 
> 
> 
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0
  2021-07-15 17:57     ` Konrad Weihmann
@ 2021-07-15 18:40       ` Trevor Gamblin
  2021-07-15 18:55       ` Khem Raj
  1 sibling, 0 replies; 8+ messages in thread
From: Trevor Gamblin @ 2021-07-15 18:40 UTC (permalink / raw)
  To: Konrad Weihmann, Khem Raj, Martin Jansa, wangmy, Leon Anavi,
	Derek Straka
  Cc: Patches and discussions about the oe-core layer


On 2021-07-15 1:57 p.m., Konrad Weihmann wrote:
> [Please note: This e-mail is from an EXTERNAL e-mail address]
>
> On 15.07.21 19:22, Khem Raj wrote:
>>
>>
>> On 7/15/21 7:43 AM, Martin Jansa wrote:
>>> On Mon, Jul 12, 2021 at 9:57 AM wangmy <wangmy@fujitsu.com
>>> <mailto:wangmy@fujitsu.com>> wrote:
>>>
>>>     reproducibility.patch
>>>     deleted since it's been merged upstream.
>>>
>>>     v57.1.0
>>>     -------
>>>
>>>     Changes
>>>     ^^^^^^^
>>>     * #2692: Globs are now sorted in 'license_files' restoring
>>>     reproducibility by eliminating variance from disk order.
>>>     * #2714: Update to distutils at pypa/distutils@e2627b7.
>>>     * #2715: Removed reliance on deprecated ssl.match_hostname by
>>>     removing the ssl support. Now any index operations rely on the
>>>     native SSL implementation.
>>>
>>>
>>> FWIW: this change
>>> breaks 
>>> meta-openembedded/meta-python/recipes-devtools/python/python3-cassandra-driver_3.25.0.bb:do_install:
>>>
>>>
>>> + people who might be interested in python3-cassandra-driver.
>>>
>>> | Installed
>>> /jenkins/mjansa/build/ros/ros2-rolling-honister/tmp-glibc/work/core2-64-oe-linux/python3-cassandra-driver/3.25.0-r0/recipe-sysroot-native/usr/lib/python3.9/site-packages/cassandra_driver-3.25.0-py3.9-linux-x86_64.egg 
>>>
>>>
>>> | Processing dependencies for cassandra-driver==3.25.0
>>> | Searching for geomet<0.3,>=0.1
>>> | Reading https://pypi.org/simple/geomet/
>>> <https://pypi.org/simple/geomet/>
>>> | Download error on https://pypi.org/simple/geomet/
>>> <https://pypi.org/simple/geomet/>: [SSL: CERTIFICATE_VERIFY_FAILED]
>>> certificate verify failed: unable to get local issuer certificate
>>> (_ssl.c:1129) -- Some packages may not be found!
>>> | Couldn't find index page for 'geomet' (maybe misspelled?)
>>> | Scanning index of all packages (this may take a while)
>>> | Reading https://pypi.org/simple/ <https://pypi.org/simple/>
>>> | Download error on https://pypi.org/simple/
>>> <https://pypi.org/simple/>: [SSL: CERTIFICATE_VERIFY_FAILED]
>>> certificate verify failed: unable to get local issuer certificate
>>> (_ssl.c:1129) -- Some packages may not be found!
>>> | No local packages or working download links found for 
>>> geomet<0.3,>=0.1
>
> But that not directly an issue of the setuptools upgrade, but a not well
> written recipe here.
> I'm more than lucky that this update revealed the issue, because
> otherwise no one would have really noticed that the original recipe of
> cassandra-driver is pulling sources while installing.
> So just in case I would rather pull in this upgrade than to mind the
> broken recipe from meta-python here (just my 2c)

FYI, I am working on a recipe for python3-geomat to make this work 
better in the future.

- Trevor

>
>>> | error: Could not find suitable distribution for
>>> Requirement.parse('geomet<0.3,>=0.1')
>>>
>>>
>>
>> yes its reproduced on AB as well.
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/1397
>>
>>>
>>>
>>>
>>>
>>
>>
>> 
>>

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0
  2021-07-15 17:57     ` Konrad Weihmann
  2021-07-15 18:40       ` Trevor Gamblin
@ 2021-07-15 18:55       ` Khem Raj
  2021-07-16 13:32         ` Trevor Gamblin
  1 sibling, 1 reply; 8+ messages in thread
From: Khem Raj @ 2021-07-15 18:55 UTC (permalink / raw)
  To: Konrad Weihmann
  Cc: Martin Jansa, wangmy, Leon Anavi, Derek Straka, Trevor Gamblin,
	Patches and discussions about the oe-core layer

On Thu, Jul 15, 2021 at 10:57 AM Konrad Weihmann <kweihmann@outlook.com> wrote:
>
> On 15.07.21 19:22, Khem Raj wrote:
> >
> >
> > On 7/15/21 7:43 AM, Martin Jansa wrote:
> >> On Mon, Jul 12, 2021 at 9:57 AM wangmy <wangmy@fujitsu.com
> >> <mailto:wangmy@fujitsu.com>> wrote:
> >>
> >>     reproducibility.patch
> >>     deleted since it's been merged upstream.
> >>
> >>     v57.1.0
> >>     -------
> >>
> >>     Changes
> >>     ^^^^^^^
> >>     * #2692: Globs are now sorted in 'license_files' restoring
> >>     reproducibility by eliminating variance from disk order.
> >>     * #2714: Update to distutils at pypa/distutils@e2627b7.
> >>     * #2715: Removed reliance on deprecated ssl.match_hostname by
> >>     removing the ssl support. Now any index operations rely on the
> >>     native SSL implementation.
> >>
> >>
> >> FWIW: this change
> >> breaks  meta-openembedded/meta-python/recipes-devtools/python/python3-cassandra-driver_3.25.0.bb:do_install:
> >>
> >>
> >> + people who might be interested in python3-cassandra-driver.
> >>
> >> | Installed
> >> /jenkins/mjansa/build/ros/ros2-rolling-honister/tmp-glibc/work/core2-64-oe-linux/python3-cassandra-driver/3.25.0-r0/recipe-sysroot-native/usr/lib/python3.9/site-packages/cassandra_driver-3.25.0-py3.9-linux-x86_64.egg
> >>
> >> | Processing dependencies for cassandra-driver==3.25.0
> >> | Searching for geomet<0.3,>=0.1
> >> | Reading https://pypi.org/simple/geomet/
> >> <https://pypi.org/simple/geomet/>
> >> | Download error on https://pypi.org/simple/geomet/
> >> <https://pypi.org/simple/geomet/>: [SSL: CERTIFICATE_VERIFY_FAILED]
> >> certificate verify failed: unable to get local issuer certificate
> >> (_ssl.c:1129) -- Some packages may not be found!
> >> | Couldn't find index page for 'geomet' (maybe misspelled?)
> >> | Scanning index of all packages (this may take a while)
> >> | Reading https://pypi.org/simple/ <https://pypi.org/simple/>
> >> | Download error on https://pypi.org/simple/
> >> <https://pypi.org/simple/>: [SSL: CERTIFICATE_VERIFY_FAILED]
> >> certificate verify failed: unable to get local issuer certificate
> >> (_ssl.c:1129) -- Some packages may not be found!
> >> | No local packages or working download links found for geomet<0.3,>=0.1
>
> But that not directly an issue of the setuptools upgrade, but a not well
> written recipe here.
> I'm more than lucky that this update revealed the issue, because
> otherwise no one would have really noticed that the original recipe of
> cassandra-driver is pulling sources while installing.
> So just in case I would rather pull in this upgrade than to mind the
> broken recipe from meta-python here (just my 2c)
>

i am fine with whatever the right fix is. Its normal that some base
infra component dependency
drives a change in another recipe if thats inline with expected design
then change the component
to work with new update.

> >> | error: Could not find suitable distribution for
> >> Requirement.parse('geomet<0.3,>=0.1')
> >>
> >>
> >
> > yes its reproduced on AB as well.
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/1397
> >
> >>
> >>
> >>
> >>
> >
> >
> > 
> >

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0
  2021-07-15 18:55       ` Khem Raj
@ 2021-07-16 13:32         ` Trevor Gamblin
  2021-07-16 13:38           ` Martin Jansa
  0 siblings, 1 reply; 8+ messages in thread
From: Trevor Gamblin @ 2021-07-16 13:32 UTC (permalink / raw)
  To: Khem Raj, Konrad Weihmann
  Cc: Martin Jansa, wangmy, Leon Anavi, Derek Straka,
	Patches and discussions about the oe-core layer


On 2021-07-15 2:55 p.m., Khem Raj wrote:
> [Please note: This e-mail is from an EXTERNAL e-mail address]
>
> On Thu, Jul 15, 2021 at 10:57 AM Konrad Weihmann <kweihmann@outlook.com> wrote:
>> On 15.07.21 19:22, Khem Raj wrote:
>>>
>>> On 7/15/21 7:43 AM, Martin Jansa wrote:
>>>> On Mon, Jul 12, 2021 at 9:57 AM wangmy <wangmy@fujitsu.com
>>>> <mailto:wangmy@fujitsu.com>> wrote:
>>>>
>>>>      reproducibility.patch
>>>>      deleted since it's been merged upstream.
>>>>
>>>>      v57.1.0
>>>>      -------
>>>>
>>>>      Changes
>>>>      ^^^^^^^
>>>>      * #2692: Globs are now sorted in 'license_files' restoring
>>>>      reproducibility by eliminating variance from disk order.
>>>>      * #2714: Update to distutils at pypa/distutils@e2627b7.
>>>>      * #2715: Removed reliance on deprecated ssl.match_hostname by
>>>>      removing the ssl support. Now any index operations rely on the
>>>>      native SSL implementation.
>>>>
>>>>
>>>> FWIW: this change
>>>> breaks  meta-openembedded/meta-python/recipes-devtools/python/python3-cassandra-driver_3.25.0.bb:do_install:
>>>>
>>>>
>>>> + people who might be interested in python3-cassandra-driver.
>>>>
>>>> | Installed
>>>> /jenkins/mjansa/build/ros/ros2-rolling-honister/tmp-glibc/work/core2-64-oe-linux/python3-cassandra-driver/3.25.0-r0/recipe-sysroot-native/usr/lib/python3.9/site-packages/cassandra_driver-3.25.0-py3.9-linux-x86_64.egg
>>>>
>>>> | Processing dependencies for cassandra-driver==3.25.0
>>>> | Searching for geomet<0.3,>=0.1
>>>> | Reading https://pypi.org/simple/geomet/
>>>> <https://pypi.org/simple/geomet/>
>>>> | Download error on https://pypi.org/simple/geomet/
>>>> <https://pypi.org/simple/geomet/>: [SSL: CERTIFICATE_VERIFY_FAILED]
>>>> certificate verify failed: unable to get local issuer certificate
>>>> (_ssl.c:1129) -- Some packages may not be found!
>>>> | Couldn't find index page for 'geomet' (maybe misspelled?)
>>>> | Scanning index of all packages (this may take a while)
>>>> | Reading https://pypi.org/simple/ <https://pypi.org/simple/>
>>>> | Download error on https://pypi.org/simple/
>>>> <https://pypi.org/simple/>: [SSL: CERTIFICATE_VERIFY_FAILED]
>>>> certificate verify failed: unable to get local issuer certificate
>>>> (_ssl.c:1129) -- Some packages may not be found!
>>>> | No local packages or working download links found for geomet<0.3,>=0.1
>> But that not directly an issue of the setuptools upgrade, but a not well
>> written recipe here.
>> I'm more than lucky that this update revealed the issue, because
>> otherwise no one would have really noticed that the original recipe of
>> cassandra-driver is pulling sources while installing.
>> So just in case I would rather pull in this upgrade than to mind the
>> broken recipe from meta-python here (just my 2c)
>>
> i am fine with whatever the right fix is. Its normal that some base
> infra component dependency
> drives a change in another recipe if thats inline with expected design
> then change the component
> to work with new update.

I've sent a patch to the openembedded-devel mailing list that removes 
the --no-libev flags and adds python3-geomet (which I also sent a patch 
to add to the layer) to the RDEPENDS, allowing python3-cassandra-driver 
to build/import OK in an image without trying to pull stuff during 
do_install.

- Trevor

>
>>>> | error: Could not find suitable distribution for
>>>> Requirement.parse('geomet<0.3,>=0.1')
>>>>
>>>>
>>> yes its reproduced on AB as well.
>>> https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/1397
>>>
>>>>
>>>>
>>>>
>>>
>>> 
>>>

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0
  2021-07-16 13:32         ` Trevor Gamblin
@ 2021-07-16 13:38           ` Martin Jansa
  0 siblings, 0 replies; 8+ messages in thread
From: Martin Jansa @ 2021-07-16 13:38 UTC (permalink / raw)
  To: Trevor Gamblin
  Cc: Khem Raj, Konrad Weihmann, wangmy, Leon Anavi, Derek Straka,
	Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 4019 bytes --]

Thanks Trevor! my next world build will be green again (maybe) :).

On Fri, Jul 16, 2021 at 3:32 PM Trevor Gamblin <trevor.gamblin@windriver.com>
wrote:

>
> On 2021-07-15 2:55 p.m., Khem Raj wrote:
> > [Please note: This e-mail is from an EXTERNAL e-mail address]
> >
> > On Thu, Jul 15, 2021 at 10:57 AM Konrad Weihmann <kweihmann@outlook.com>
> wrote:
> >> On 15.07.21 19:22, Khem Raj wrote:
> >>>
> >>> On 7/15/21 7:43 AM, Martin Jansa wrote:
> >>>> On Mon, Jul 12, 2021 at 9:57 AM wangmy <wangmy@fujitsu.com
> >>>> <mailto:wangmy@fujitsu.com>> wrote:
> >>>>
> >>>>      reproducibility.patch
> >>>>      deleted since it's been merged upstream.
> >>>>
> >>>>      v57.1.0
> >>>>      -------
> >>>>
> >>>>      Changes
> >>>>      ^^^^^^^
> >>>>      * #2692: Globs are now sorted in 'license_files' restoring
> >>>>      reproducibility by eliminating variance from disk order.
> >>>>      * #2714: Update to distutils at pypa/distutils@e2627b7.
> >>>>      * #2715: Removed reliance on deprecated ssl.match_hostname by
> >>>>      removing the ssl support. Now any index operations rely on the
> >>>>      native SSL implementation.
> >>>>
> >>>>
> >>>> FWIW: this change
> >>>> breaks
> meta-openembedded/meta-python/recipes-devtools/python/python3-cassandra-driver_3.25.0.bb:
> do_install:
> >>>>
> >>>>
> >>>> + people who might be interested in python3-cassandra-driver.
> >>>>
> >>>> | Installed
> >>>>
> /jenkins/mjansa/build/ros/ros2-rolling-honister/tmp-glibc/work/core2-64-oe-linux/python3-cassandra-driver/3.25.0-r0/recipe-sysroot-native/usr/lib/python3.9/site-packages/cassandra_driver-3.25.0-py3.9-linux-x86_64.egg
> >>>>
> >>>> | Processing dependencies for cassandra-driver==3.25.0
> >>>> | Searching for geomet<0.3,>=0.1
> >>>> | Reading https://pypi.org/simple/geomet/
> >>>> <https://pypi.org/simple/geomet/>
> >>>> | Download error on https://pypi.org/simple/geomet/
> >>>> <https://pypi.org/simple/geomet/>: [SSL: CERTIFICATE_VERIFY_FAILED]
> >>>> certificate verify failed: unable to get local issuer certificate
> >>>> (_ssl.c:1129) -- Some packages may not be found!
> >>>> | Couldn't find index page for 'geomet' (maybe misspelled?)
> >>>> | Scanning index of all packages (this may take a while)
> >>>> | Reading https://pypi.org/simple/ <https://pypi.org/simple/>
> >>>> | Download error on https://pypi.org/simple/
> >>>> <https://pypi.org/simple/>: [SSL: CERTIFICATE_VERIFY_FAILED]
> >>>> certificate verify failed: unable to get local issuer certificate
> >>>> (_ssl.c:1129) -- Some packages may not be found!
> >>>> | No local packages or working download links found for
> geomet<0.3,>=0.1
> >> But that not directly an issue of the setuptools upgrade, but a not well
> >> written recipe here.
> >> I'm more than lucky that this update revealed the issue, because
> >> otherwise no one would have really noticed that the original recipe of
> >> cassandra-driver is pulling sources while installing.
> >> So just in case I would rather pull in this upgrade than to mind the
> >> broken recipe from meta-python here (just my 2c)
> >>
> > i am fine with whatever the right fix is. Its normal that some base
> > infra component dependency
> > drives a change in another recipe if thats inline with expected design
> > then change the component
> > to work with new update.
>
> I've sent a patch to the openembedded-devel mailing list that removes
> the --no-libev flags and adds python3-geomet (which I also sent a patch
> to add to the layer) to the RDEPENDS, allowing python3-cassandra-driver
> to build/import OK in an image without trying to pull stuff during
> do_install.
>
> - Trevor
>
> >
> >>>> | error: Could not find suitable distribution for
> >>>> Requirement.parse('geomet<0.3,>=0.1')
> >>>>
> >>>>
> >>> yes its reproduced on AB as well.
> >>> https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/1397
> >>>
> >>>>
> >>>>
> >>>>
> >>>
> >>>
> >>>
>
> 
>
>

[-- Attachment #2: Type: text/html, Size: 6272 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2021-07-16 13:38 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-12  7:56 [OE-core] [PATCH v4] python3-setuptools: upgrade 57.0.0 -> 57.1.0 wangmy
2021-07-15 14:43 ` Martin Jansa
2021-07-15 17:22   ` Khem Raj
2021-07-15 17:57     ` Konrad Weihmann
2021-07-15 18:40       ` Trevor Gamblin
2021-07-15 18:55       ` Khem Raj
2021-07-16 13:32         ` Trevor Gamblin
2021-07-16 13:38           ` Martin Jansa

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.