<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p>Hello Huges,</p>
    <p><br>
    </p>
    <p>I cannot reply inline because you sent the patch as an
      attachment, please send it using git send-email next time.</p>
    <p><br>
    </p>
    <p>Also, something doesnt add up here:</p>
    <p><br>
    </p>
    <p><br>
    </p>
    <pre class="moz-quote-pre" wrap="">+    "pyvenv": {
+        "summary": "Allow you to create virtual environments so we can isolate our project dependencies. Deprecated in Python 3.6",
+        "rdepends": [
+            "core",
+            "venv"


This doesnt make sense, since you are saying that the python3-pyvenv package depends on python3-venv?, shouldnt they be the same package?
 

+        ],
+        "files": [
+            "${bindir}/pyvenv*"



Again, this should probably on one single package.




 +        ],
+        "cached": []
+    },
     "resource": {
         "summary": "Python resource control interface",
         "rdepends": [
@@ -1107,6 +1118,21 @@
             "${libdir}/python${PYTHON_MAJMIN}/__pycache__/getpass.*.pyc"
         ]




This is wrong because, cached files shouldnt be on rdepends.



Everything from there to the bottom is probably ok.

 },
+    "venv": {
+        "summary": "Provides support for creating lightweight virtual environments with their own site directories, optionally isolated from system site directories",
+        "rdepends": [
+            "compression",
+            "core",
+            "logging",
+            "shell",
+            "stringold",
+            "unixadmin"
+        ],
+        "files": [
+            "${libdir}/python${PYTHON_MAJMIN}/venv"
+        ],
+        "cached": []
+    },
     "xml": {
         "summary": "Python basic XML support",
         "rdepends": [</pre>
    <p><br>
    </p>
    <p><br>
    </p>
    <p><br>
    </p>
    <p>Also, this patch is missing the changes to the python3 recipe
      file where the python3-pyvenv package was being previously
      created, if were now doing it through the manifest, then we dont
      need it in the recipe.</p>
    <p><br>
    </p>
    <p>Thanks!</p>
    <p><br>
    </p>
    <p>Alejandro<br>
    </p>
    <p><br>
    </p>
    <p><br>
    </p>
    <p><br>
    </p>
    <div class="moz-cite-prefix">On 10/22/2018 11:48 AM, Hugues Kamba
      wrote:<br>
    </div>
    <blockquote type="cite"
cite="mid:VI1PR08MB36317EE05BE84D857C630F2398F40@VI1PR08MB3631.eurprd08.prod.outlook.com">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      <style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
      <div id="divtagdefaultwrapper" dir="ltr" style="font-size:12pt;
        color:rgb(0,0,0);
        font-family:Calibri,Helvetica,sans-serif,EmojiFont,&quot;Apple
        Color Emoji&quot;,&quot;Segoe UI
        Emoji&quot;,NotoColorEmoji,&quot;Segoe UI
        Symbol&quot;,&quot;Android Emoji&quot;,EmojiSymbols">
        <p style="margin-top:0; margin-bottom:0">Dear All,</p>
        <p style="margin-top:0; margin-bottom:0"><br>
        </p>
        <p style="margin-top:0; margin-bottom:0">As it currently is,
          python3-pyvenv is broken because it is missing the venv module
          run-time dependency.</p>
        <p style="margin-top:0; margin-bottom:0"><br>
        </p>
        <p style="margin-top:0; margin-bottom:0">The patch I am
          submitting is to add the venv module dependency to
          python3-pyvenv. </p>
        <p style="margin-top:0; margin-bottom:0">The dependency has
          been <span style="font-size:12pt">added </span><span
            style="font-size:12pt">using the newly added venv package
            (python3-venv).</span></p>
        <p style="margin-top:0; margin-bottom:0"><span
            style="font-size:12pt">The python3-venv package has been
            added for future proofing since pyvenv has been deprecated
            in Python 3.6 and creation of virtual environment is done
            using the venv module directly (i.e python3 -m venv
            my_venv).</span></p>
        <p style="margin-top:0; margin-bottom:0"><span
            style="font-size:12pt"><br>
          </span></p>
        <p style="margin-top:0; margin-bottom:0"><span
            style="font-size:12pt">Regards,</span></p>
        <p style="margin-top:0; margin-bottom:0"><span
            style="font-size:12pt"><br>
          </span></p>
        <p style="margin-top:0; margin-bottom:0"><span
            style="font-size:12pt">Hugues Kamba</span></p>
        <div id="Signature">
          <div id="divtagdefaultwrapper" dir="ltr" style="">
            <p style="color:rgb(0,0,0);
              font-family:Calibri,Helvetica,sans-serif,EmojiFont,&quot;Apple
              Color Emoji&quot;,&quot;Segoe UI
              Emoji&quot;,NotoColorEmoji,&quot;Segoe UI
              Symbol&quot;,&quot;Android Emoji&quot;,EmojiSymbols;
              font-size:12pt">
            </p>
          </div>
        </div>
      </div>
      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>
      <fieldset class="mimeAttachmentHeader"></fieldset>
    </blockquote>
  </body>
</html>