All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alejandro Enedino Hernandez Samaniego <alejandro.enedino.hernandez-samaniego@xilinx.com>
To: Jens Rehsack <sno@netbsd.org>,
	Alejandro Enedino Hernandez Samaniego
	<alejandro.enedino.hernandez-samaniego@xilinx.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH v2] python3{,-native}: update to 3.7.0
Date: Mon, 10 Sep 2018 14:33:50 -0700	[thread overview]
Message-ID: <badf35f0-449f-d632-c016-509dfb3f0f01@xilinx.com> (raw)
In-Reply-To: <F9FC0A02-BAE6-45CB-A38D-7AE4E39E5587@netbsd.org>

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

Hey Jens,


As I explained before, when you create a manifest for python (target), 
it uses the native build as base (it literally runs the native python 
that was just built), it is assumed its the same version as target and 
contains all the modules provided by upstream, otherwise the missing 
modules cannot be checked for dependencies, and the manifest becomes 
incoherent, so its not an option to have an incomplete python native build.


Yes you probably need a patch to look at the correct directories for the 
h files, as well as a dependency to make the h files available on 
recipe-sysroot-native.

Alejandro


On 09/10/2018 02:05 PM, Jens Rehsack wrote:
> Hey Alejandro,
>
> I fixed that for cross-compile only, since I would need add a patch 
> and a dependency python3-native for one thing: calculate uuids.
> When you can explain to me why the python-native needs that, I'll 
> change that from -target & nativesdk to all.
>
> From my point of view it's not a question of having every (unneeded) 
> python module being built for the native python, which is used for 
> cross-compiling python and some modules only.
>
> Cheers,
> Jens
>
>> Am 10.09.2018 um 22:17 schrieb Alejandro Enedino Hernandez Samaniego 
>> <alejandro.enedino.hernandez-samaniego@xilinx.com 
>> <mailto:alejandro.enedino.hernandez-samaniego@xilinx.com>>:
>>
>> Hey Jens,
>>
>> The compilation log for python3-native still shows that it didn't 
>> build the uuid module
>>
>>
>> Python build finished successfully!
>> The necessary bits to build these optional modules were not found:
>> _uuid
>>
>> Please look at my previous reply to find how this can be solved (its 
>> likely a missing DEPENDS).
>>
>>
>> Also, this patch is missing the new python3 manifest for this 
>> release, there appears to be a few new modules that we need to decide 
>> which package they belong to, this is the output of bitbake python3 
>> -c create_manifest:
>>
>>
>> | ERROR:
>> | The following files are repeated (contained in more than one package),
>> | this is likely to happen when new files are introduced after an 
>> upgrade,
>> | please check which package should get it,
>> |  modify the manifest accordingly and re-run the create_manifest task:
>> | ${libdir}/python${PYTHON_MAJMIN}/lib-dynload/_blake2.*.so
>> | ${libdir}/python${PYTHON_MAJMIN}/lib-dynload/_sha3.*.so
>> | ${libdir}/python${PYTHON_MAJMIN}/lib-dynload/_contextvars.*.so
>> | ${libdir}/python${PYTHON_MAJMIN}/contextvars.py
>> | ${libdir}/python${PYTHON_MAJMIN}/__pycache__/contextvars.*.pyc
>> | ${libdir}/python${PYTHON_MAJMIN}/lib-dynload/_queue.*.so
>>
>>
>> Cheers,
>>
>>
>> Alejandro
>>
>>
>> On 09/10/2018 09:38 AM, Burton, Ross wrote:
>>> One thing to be aware of is that I've been fixing up Python's PGO
>>> support and there's a slew of patches in master-next and more just
>>> posted that this needs to be rebased on top of.  Good news is that my
>>> patches remove two of the patches we've been carrying!
>>>
>>> Ross
>>>
>>> On 10 September 2018 at 17:36, Jens Rehsack <sno@netbsd.org 
>>> <mailto:sno@netbsd.org>> wrote:
>>>>
>>>> Am 10.09.2018 um 11:35 schrieb Alexander Kanavin 
>>>> <alex.kanavin@gmail.com <mailto:alex.kanavin@gmail.com>>:
>>>>
>>>> Large parts of dnf and friends have been rewritten in c++. I have not
>>>> yet updated and reviewed that, that will happen in the next cycle.
>>>>
>>>>
>>>> If I can prepare something for you - drop me a note.
>>>> Otherwise - the perl-5.28 update ("." in @INC, regex buffer 
>>>> overflow, ...)
>>>> is also
>>>> awaiting some progress (I can keep "myself" busy).
>>>>
>>>> There's already enough disruption to deal with (postinsts errors,
>>>> openssl 1.1, both caused by me :)
>>>>
>>>>
>>>> You know, corner, ash, ... things happen. But there is progress! Great!
>>>>
>>>> Good that we got all the way to do_rootfs though with 3.7.
>>>>
>>>>
>>>> Yeah, but than came postinst (coreutils :P) :D
>>>>
>>>> Cheers
>>>>
>>>> Alex
>>>>
>>>> 2018-09-10 0:38 GMT+02:00 Tim Orling <ticotimo@gmail.com 
>>>> <mailto:ticotimo@gmail.com>>:
>>>>
>>>> I did not review the patches closely, but I did try to build
>>>> core-image-full-cmdline with the tip of poky and these patches applied.
>>>> Everything was fine until do_rootfs... I've attached the log.
>>>>
>>>> Essentially, there are some bits of dnf and so on which are not 
>>>> ready for
>>>> Python 3.7. We have dnf version 2.7.5, but the latest upstream 
>>>> release is
>>>> 3.4.0 (with a 3.5.0 just 3 days ago). Not sure yet if that would have
>>>> helped.
>>>>
>>>> --
>>>> _______________________________________________
>>>> Openembedded-core mailing list
>>>> Openembedded-core@lists.openembedded.org 
>>>> <mailto:Openembedded-core@lists.openembedded.org>
>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>>>
>>>>
>>>> --
>>>> Jens Rehsack - rehsack@gmail.com
>>>>
>>>>
>>>> --
>>>> _______________________________________________
>>>> Openembedded-core mailing list
>>>> Openembedded-core@lists.openembedded.org
>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>>>
>>
>> -- 
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org 
>> <mailto:Openembedded-core@lists.openembedded.org>
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
> -- 
> Jens Rehsack - rehsack@gmail.com <mailto:rehsack@gmail.com>
>


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

  reply	other threads:[~2018-09-10 21:34 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-07 14:55 [PATCH v2] python3{,-native}: update to 3.7.0 Jens Rehsack
2018-09-07 14:59 ` Jens Rehsack
2018-09-09 22:38   ` Tim Orling
2018-09-10  9:35     ` Alexander Kanavin
2018-09-10 16:36       ` Jens Rehsack
2018-09-10 16:38         ` Burton, Ross
2018-09-10 20:17           ` Alejandro Enedino Hernandez Samaniego
2018-09-10 21:05             ` Jens Rehsack
2018-09-10 21:33               ` Alejandro Enedino Hernandez Samaniego [this message]
2018-09-11  6:58                 ` Jens Rehsack
2018-09-11 18:56                   ` Alejandro Enedino Hernandez Samaniego
2018-09-12 11:53                     ` Jens Rehsack
2018-09-12 21:02                       ` Alejandro Hernandez
2018-09-17 19:00                         ` Jens Rehsack
2018-09-19  6:11                           ` Alejandro Hernandez
2018-09-11  7:09           ` Jens Rehsack
2018-09-10 16:44         ` Alexander Kanavin
2018-09-10 16:42       ` Jens Rehsack
2018-09-10 18:10       ` Jens Rehsack
2018-09-07 15:03 ` ✗ patchtest: failure for python3{,-native}: update to 3.7.0 (rev2) Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=badf35f0-449f-d632-c016-509dfb3f0f01@xilinx.com \
    --to=alejandro.enedino.hernandez-samaniego@xilinx.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=sno@netbsd.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.