<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Hello Ross, <br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Thank you for reviewing. <br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
I am not sure where you saw the commit message say that pyvenv has been removed.<br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
What it says about pyvenv is that it is deprecated in python3.6 (to explain why it has been provided via the newly created venv package).<br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Regards<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Hugues<br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Get <a href="https://aka.ms/ghei36">Outlook for Android</a></div>
<br>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Burton, Ross <ross.burton@intel.com><br>
<b>Sent:</b> Tuesday, October 30, 2018 4:06:40 PM<br>
<b>To:</b> Hugues Kamba<br>
<b>Cc:</b> OE-core<br>
<b>Subject:</b> Re: [OE-core] [PATCH v2] python3: Fix python3-pyvenv run-time dependency</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:11pt;">
<div class="PlainText">Patch looks good, but the commit message contradicts itself a few<br>
times. python3-pyvenv is removed, but:<br>
<br>
" This patch adds the python3-venv module as a self-contained package which<br>
python3-pyvenv must depend on at run-time."<br>
This patch specifies the newly created<br>
python3-venv as a run-time dependency of python3-pyvenv."<br>
<br>
Ross<br>
On Tue, 30 Oct 2018 at 13:08, Hugues Kamba <Hugues.Kamba@arm.com> wrote:<br>
><br>
> Pyvenv is just a small script that uses venv to create virtual<br>
> environments.<br>
> <a href="https://www.python.org/dev/peps/pep-0405/#creating-virtual-environments">
https://www.python.org/dev/peps/pep-0405/#creating-virtual-environments</a><br>
><br>
> This patch adds the python3-venv module as a self-contained package which<br>
> python3-pyvenv must depend on at run-time.<br>
><br>
> The patch also provides the package python3-pyvenv from the pyhton3-venv<br>
> package.This is good for future-proofing since python3-pyvenv has been<br>
> deprecated and only python3-venv is now available in Python 3.6.<br>
> <a href="https://docs.python.org/3/library/venv.html">https://docs.python.org/3/library/venv.html</a>.<br>
><br>
> Without this patch python3-pyvenv is broken because it is missing the<br>
> venv module at run-time. This patch specifies the newly created<br>
> python3-venv as a run-time dependency of python3-pyvenv.<br>
><br>
> Signed-off-by: Hugues Kamba <hugues.kamba@arm.com><br>
> ---<br>
> .../python/python3/python3-manifest.json | 16 ++++++++++++++++<br>
> meta/recipes-devtools/python/python3_3.5.6.bb | 4 ++--<br>
> 2 files changed, 18 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/meta/recipes-devtools/python/python3/python3-manifest.json b/meta/recipes-devtools/python/python3/python3-manifest.json<br>
> index f922561368..ef7fa4ecaf 100644<br>
> --- a/meta/recipes-devtools/python/python3/python3-manifest.json<br>
> +++ b/meta/recipes-devtools/python/python3/python3-manifest.json<br>
> @@ -1107,6 +1107,22 @@<br>
> "${libdir}/python${PYTHON_MAJMIN}/__pycache__/getpass.*.pyc"<br>
> ]<br>
> },<br>
> + "venv": {<br>
> + "summary": "Provides support for creating lightweight virtual environments with their own site directories, optionally isolated from system site directories.",<br>
> + "rdepends": [<br>
> + "compression",<br>
> + "core",<br>
> + "logging",<br>
> + "shell",<br>
> + "stringold",<br>
> + "unixadmin"<br>
> + ],<br>
> + "files": [<br>
> + "${libdir}/python${PYTHON_MAJMIN}/venv",<br>
> + "${bindir}/pyvenv*"<br>
> + ],<br>
> + "cached": []<br>
> + },<br>
> "xml": {<br>
> "summary": "Python basic XML support",<br>
> "rdepends": [<br>
> diff --git a/meta/recipes-devtools/python/python3_3.5.6.bb b/meta/recipes-devtools/python/python3_3.5.6.bb<br>
> index 2a4547694a..31f8eada11 100644<br>
> --- a/meta/recipes-devtools/python/python3_3.5.6.bb<br>
> +++ b/meta/recipes-devtools/python/python3_3.5.6.bb<br>
> @@ -226,8 +226,8 @@ FILES_${PN}-2to3 += "${bindir}/2to3-${PYTHON_MAJMIN}"<br>
> FILES_${PN}-pydoc += "${bindir}/pydoc${PYTHON_MAJMIN} ${bindir}/pydoc3"<br>
> FILES_${PN}-idle += "${bindir}/idle3 ${bindir}/idle${PYTHON_MAJMIN}"<br>
><br>
> -PACKAGES =+ "${PN}-pyvenv"<br>
> -FILES_${PN}-pyvenv += "${bindir}/pyvenv-${PYTHON_MAJMIN} ${bindir}/pyvenv"<br>
> +# provide python-pyvenv from python3-venv<br>
> +RPROVIDES_${PN}-venv += "${PN}-pyvenv"<br>
><br>
> # package libpython3<br>
> PACKAGES =+ "libpython3 libpython3-staticdev"<br>
> --<br>
> 2.18.0<br>
><br>
> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any
purpose, or store or copy the information in any medium. Thank you.<br>
> --<br>
> _______________________________________________<br>
> Openembedded-core mailing list<br>
> Openembedded-core@lists.openembedded.org<br>
> <a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><br>
</div>
</span></font>IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use
it for any purpose, or store or copy the information in any medium. Thank you.
</body>
</html>