All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-python][PATCH] python-attrs: Adjust the RDEPENDS to include crypt rather than cryptography
@ 2018-04-12 13:33 jan vermaete
  2018-04-12 14:12 ` Derek Straka
  0 siblings, 1 reply; 6+ messages in thread
From: jan vermaete @ 2018-04-12 13:33 UTC (permalink / raw)
  To: openembedded-devel

I'm afraid this patch just revert the file to the previous state.
As far as I can see, there is no need at all to have something of
crypto nor crypotgraphy needed for this python package.

-- 
Jan Vermaete
"For every complex problem there is an answer that is clear, simple,
and wrong." - H.L. Mencken


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

* Re: [meta-python][PATCH] python-attrs: Adjust the RDEPENDS to include crypt rather than cryptography
  2018-04-12 13:33 [meta-python][PATCH] python-attrs: Adjust the RDEPENDS to include crypt rather than cryptography jan vermaete
@ 2018-04-12 14:12 ` Derek Straka
  2018-04-12 15:09   ` jan vermaete
  0 siblings, 1 reply; 6+ messages in thread
From: Derek Straka @ 2018-04-12 14:12 UTC (permalink / raw)
  To: jan vermaete; +Cc: openembeded-devel

There is a usage of hashlib (provided by ${PYTHON_PN}-crypt) in _make.py.

derek@sparta:/tmp$ dpkg -c
upstream/build/tmp/deploy/ipk/core2-64/python-crypt_2.7.14-r1_core2-64.ipk
drwxrwxrwx root/root         0 2018-03-24 09:28 ./
drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/
drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/lib/
drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/lib/python2.7/
-rw-r--r-- root/root      7842 2018-03-24 09:28
./usr/lib/python2.7/hashlib.py
-rw-r--r-- root/root      6891 2018-03-24 09:28
./usr/lib/python2.7/hashlib.pyc
drwxr-xr-x root/root         0 2018-03-24 09:28
./usr/lib/python2.7/lib-dynload/
-rwxr-xr-x root/root     25184 2018-03-24 09:28
./usr/lib/python2.7/lib-dynload/_hashlib.so
-rwxr-xr-x root/root      6520 2018-03-24 09:28
./usr/lib/python2.7/lib-dynload/crypt.so
-rw-r--r-- root/root       358 2018-03-24 09:28 ./usr/lib/python2.7/md5.py
-rw-r--r-- root/root       376 2018-03-24 09:28 ./usr/lib/python2.7/md5.pyc
-rw-r--r-- root/root       393 2018-03-24 09:28 ./usr/lib/python2.7/sha.py
-rw-r--r-- root/root       419 2018-03-24 09:28 ./usr/lib/python2.7/sha.pyc

derek@sparta:/tmp$ grep -r hashlib attrs/
attrs/src/attr/_make.py:import hashlib
attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
attrs/src/attr/_make.py:    sha1 = hashlib.sha1()

On Thu, Apr 12, 2018 at 9:33 AM, jan vermaete <jan.vermaete@gmail.com>
wrote:

> I'm afraid this patch just revert the file to the previous state.
> As far as I can see, there is no need at all to have something of
> crypto nor crypotgraphy needed for this python package.
>
> --
> Jan Vermaete
> "For every complex problem there is an answer that is clear, simple,
> and wrong." - H.L. Mencken
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>


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

* Re: [meta-python][PATCH] python-attrs: Adjust the RDEPENDS to include crypt rather than cryptography
  2018-04-12 14:12 ` Derek Straka
@ 2018-04-12 15:09   ` jan vermaete
  2018-04-12 15:24     ` Derek Straka
  0 siblings, 1 reply; 6+ messages in thread
From: jan vermaete @ 2018-04-12 15:09 UTC (permalink / raw)
  To: Derek Straka; +Cc: openembeded-devel

my mistake...
Thanks for the info

On Thu, Apr 12, 2018 at 4:12 PM, Derek Straka <derek@asterius.io> wrote:
> There is a usage of hashlib (provided by ${PYTHON_PN}-crypt) in _make.py.
>
> derek@sparta:/tmp$ dpkg -c
> upstream/build/tmp/deploy/ipk/core2-64/python-crypt_2.7.14-r1_core2-64.ipk
> drwxrwxrwx root/root         0 2018-03-24 09:28 ./
> drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/
> drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/lib/
> drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/lib/python2.7/
> -rw-r--r-- root/root      7842 2018-03-24 09:28
> ./usr/lib/python2.7/hashlib.py
> -rw-r--r-- root/root      6891 2018-03-24 09:28
> ./usr/lib/python2.7/hashlib.pyc
> drwxr-xr-x root/root         0 2018-03-24 09:28
> ./usr/lib/python2.7/lib-dynload/
> -rwxr-xr-x root/root     25184 2018-03-24 09:28
> ./usr/lib/python2.7/lib-dynload/_hashlib.so
> -rwxr-xr-x root/root      6520 2018-03-24 09:28
> ./usr/lib/python2.7/lib-dynload/crypt.so
> -rw-r--r-- root/root       358 2018-03-24 09:28 ./usr/lib/python2.7/md5.py
> -rw-r--r-- root/root       376 2018-03-24 09:28 ./usr/lib/python2.7/md5.pyc
> -rw-r--r-- root/root       393 2018-03-24 09:28 ./usr/lib/python2.7/sha.py
> -rw-r--r-- root/root       419 2018-03-24 09:28 ./usr/lib/python2.7/sha.pyc
>
> derek@sparta:/tmp$ grep -r hashlib attrs/
> attrs/src/attr/_make.py:import hashlib
> attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
> attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
> attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
>
> On Thu, Apr 12, 2018 at 9:33 AM, jan vermaete <jan.vermaete@gmail.com>
> wrote:
>>
>> I'm afraid this patch just revert the file to the previous state.
>> As far as I can see, there is no need at all to have something of
>> crypto nor crypotgraphy needed for this python package.
>>
>> --
>> Jan Vermaete
>> "For every complex problem there is an answer that is clear, simple,
>> and wrong." - H.L. Mencken
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
>



-- 
Jan Vermaete
"For every complex problem there is an answer that is clear, simple,
and wrong." - H.L. Mencken


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

* Re: [meta-python][PATCH] python-attrs: Adjust the RDEPENDS to include crypt rather than cryptography
  2018-04-12 15:09   ` jan vermaete
@ 2018-04-12 15:24     ` Derek Straka
  2018-11-26 20:49       ` jan vermaete
  0 siblings, 1 reply; 6+ messages in thread
From: Derek Straka @ 2018-04-12 15:24 UTC (permalink / raw)
  To: jan vermaete; +Cc: openembeded-devel

No worries.  Does this resolve the original github issue you created?

On Thu, Apr 12, 2018 at 11:09 AM, jan vermaete <jan.vermaete@gmail.com>
wrote:

> my mistake...
> Thanks for the info
>
> On Thu, Apr 12, 2018 at 4:12 PM, Derek Straka <derek@asterius.io> wrote:
> > There is a usage of hashlib (provided by ${PYTHON_PN}-crypt) in _make.py.
> >
> > derek@sparta:/tmp$ dpkg -c
> > upstream/build/tmp/deploy/ipk/core2-64/python-crypt_2.7.14-
> r1_core2-64.ipk
> > drwxrwxrwx root/root         0 2018-03-24 09:28 ./
> > drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/
> > drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/lib/
> > drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/lib/python2.7/
> > -rw-r--r-- root/root      7842 2018-03-24 09:28
> > ./usr/lib/python2.7/hashlib.py
> > -rw-r--r-- root/root      6891 2018-03-24 09:28
> > ./usr/lib/python2.7/hashlib.pyc
> > drwxr-xr-x root/root         0 2018-03-24 09:28
> > ./usr/lib/python2.7/lib-dynload/
> > -rwxr-xr-x root/root     25184 2018-03-24 09:28
> > ./usr/lib/python2.7/lib-dynload/_hashlib.so
> > -rwxr-xr-x root/root      6520 2018-03-24 09:28
> > ./usr/lib/python2.7/lib-dynload/crypt.so
> > -rw-r--r-- root/root       358 2018-03-24 09:28
> ./usr/lib/python2.7/md5.py
> > -rw-r--r-- root/root       376 2018-03-24 09:28
> ./usr/lib/python2.7/md5.pyc
> > -rw-r--r-- root/root       393 2018-03-24 09:28
> ./usr/lib/python2.7/sha.py
> > -rw-r--r-- root/root       419 2018-03-24 09:28
> ./usr/lib/python2.7/sha.pyc
> >
> > derek@sparta:/tmp$ grep -r hashlib attrs/
> > attrs/src/attr/_make.py:import hashlib
> > attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
> > attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
> > attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
> >
> > On Thu, Apr 12, 2018 at 9:33 AM, jan vermaete <jan.vermaete@gmail.com>
> > wrote:
> >>
> >> I'm afraid this patch just revert the file to the previous state.
> >> As far as I can see, there is no need at all to have something of
> >> crypto nor crypotgraphy needed for this python package.
> >>
> >> --
> >> Jan Vermaete
> >> "For every complex problem there is an answer that is clear, simple,
> >> and wrong." - H.L. Mencken
> >> --
> >> _______________________________________________
> >> Openembedded-devel mailing list
> >> Openembedded-devel@lists.openembedded.org
> >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> >
> >
>
>
>
> --
> Jan Vermaete
> "For every complex problem there is an answer that is clear, simple,
> and wrong." - H.L. Mencken
>


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

* Re: [meta-python][PATCH] python-attrs: Adjust the RDEPENDS to include crypt rather than cryptography
  2018-04-12 15:24     ` Derek Straka
@ 2018-11-26 20:49       ` jan vermaete
  0 siblings, 0 replies; 6+ messages in thread
From: jan vermaete @ 2018-11-26 20:49 UTC (permalink / raw)
  To: Derek Straka; +Cc: openembeded-devel

At the Thud branch of meta-python, I see the python3-attrs
(meta-python/recipes-devtools/python/python3-attrs_18.1.0.bb) recipe is
still depending on '${PYTHON_PN}-cryptography'.
I think depending on '${PYTHON_PN}-crypto' is enough.

With 'crypto' i.s.o. 'cryptography' the following packages where not
installed (-18%):
  python3-distutils was removed
  python3-cffi was removed
  python3-asn1crypto was removed
  cpp was removed
  python3-setuptools was removed
  python3-ply was removed
  libmpfr6 was removed
  python3-compile was removed
  cpp-symlinks was removed
  python3-pycparser was removed
  libmpc3 was removed
  python3-plistlib was removed
  python3-cryptography was removed

Br,

On Thu, Apr 12, 2018 at 5:24 PM Derek Straka <derek@asterius.io> wrote:

> No worries.  Does this resolve the original github issue you created?
>
> On Thu, Apr 12, 2018 at 11:09 AM, jan vermaete <jan.vermaete@gmail.com>
> wrote:
>
>> my mistake...
>> Thanks for the info
>>
>> On Thu, Apr 12, 2018 at 4:12 PM, Derek Straka <derek@asterius.io> wrote:
>> > There is a usage of hashlib (provided by ${PYTHON_PN}-crypt) in
>> _make.py.
>> >
>> > derek@sparta:/tmp$ dpkg -c
>> >
>> upstream/build/tmp/deploy/ipk/core2-64/python-crypt_2.7.14-r1_core2-64.ipk
>> > drwxrwxrwx root/root         0 2018-03-24 09:28 ./
>> > drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/
>> > drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/lib/
>> > drwxr-xr-x root/root         0 2018-03-24 09:28 ./usr/lib/python2.7/
>> > -rw-r--r-- root/root      7842 2018-03-24 09:28
>> > ./usr/lib/python2.7/hashlib.py
>> > -rw-r--r-- root/root      6891 2018-03-24 09:28
>> > ./usr/lib/python2.7/hashlib.pyc
>> > drwxr-xr-x root/root         0 2018-03-24 09:28
>> > ./usr/lib/python2.7/lib-dynload/
>> > -rwxr-xr-x root/root     25184 2018-03-24 09:28
>> > ./usr/lib/python2.7/lib-dynload/_hashlib.so
>> > -rwxr-xr-x root/root      6520 2018-03-24 09:28
>> > ./usr/lib/python2.7/lib-dynload/crypt.so
>> > -rw-r--r-- root/root       358 2018-03-24 09:28
>> ./usr/lib/python2.7/md5.py
>> > -rw-r--r-- root/root       376 2018-03-24 09:28
>> ./usr/lib/python2.7/md5.pyc
>> > -rw-r--r-- root/root       393 2018-03-24 09:28
>> ./usr/lib/python2.7/sha.py
>> > -rw-r--r-- root/root       419 2018-03-24 09:28
>> ./usr/lib/python2.7/sha.pyc
>> >
>> > derek@sparta:/tmp$ grep -r hashlib attrs/
>> > attrs/src/attr/_make.py:import hashlib
>> > attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
>> > attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
>> > attrs/src/attr/_make.py:    sha1 = hashlib.sha1()
>> >
>> > On Thu, Apr 12, 2018 at 9:33 AM, jan vermaete <jan.vermaete@gmail.com>
>> > wrote:
>> >>
>> >> I'm afraid this patch just revert the file to the previous state.
>> >> As far as I can see, there is no need at all to have something of
>> >> crypto nor crypotgraphy needed for this python package.
>> >>
>> >> --
>> >> Jan Vermaete
>> >> "For every complex problem there is an answer that is clear, simple,
>> >> and wrong." - H.L. Mencken
>> >> --
>> >> _______________________________________________
>> >> Openembedded-devel mailing list
>> >> Openembedded-devel@lists.openembedded.org
>> >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>> >
>> >
>>
>>
>>
>> --
>> Jan Vermaete
>> "For every complex problem there is an answer that is clear, simple,
>> and wrong." - H.L. Mencken
>>
>
>

-- 
Jan Vermaete
"For every complex problem there is an answer that is clear, simple, and
wrong." - H.L. Mencken


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

* [meta-python][PATCH] python-attrs: Adjust the RDEPENDS to include crypt rather than cryptography
@ 2018-04-11 12:30 Derek Straka
  0 siblings, 0 replies; 6+ messages in thread
From: Derek Straka @ 2018-04-11 12:30 UTC (permalink / raw)
  To: openembedded-devel

Signed-off-by: Derek Straka <derek@asterius.io>
---
 meta-python/recipes-devtools/python/python-attrs.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-python/recipes-devtools/python/python-attrs.inc b/meta-python/recipes-devtools/python/python-attrs.inc
index c29f058..ad76852 100644
--- a/meta-python/recipes-devtools/python/python-attrs.inc
+++ b/meta-python/recipes-devtools/python/python-attrs.inc
@@ -9,6 +9,6 @@ SRC_URI[sha256sum] = "1c7960ccfd6a005cd9f7ba884e6316b5e430a3f1a6c37c5f87d8b43f83
 inherit pypi
 
 RDEPENDS_${PN}_class-target += " \
-    ${PYTHON_PN}-cryptography \
+    ${PYTHON_PN}-crypt \
     ${PYTHON_PN}-ctypes \
 "    
-- 
2.7.4



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

end of thread, other threads:[~2018-11-26 20:50 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-12 13:33 [meta-python][PATCH] python-attrs: Adjust the RDEPENDS to include crypt rather than cryptography jan vermaete
2018-04-12 14:12 ` Derek Straka
2018-04-12 15:09   ` jan vermaete
2018-04-12 15:24     ` Derek Straka
2018-11-26 20:49       ` jan vermaete
  -- strict thread matches above, loose matches on Subject: below --
2018-04-11 12:30 Derek Straka

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.