<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <div class="moz-cite-prefix">Hi Mark,</div>
    <div class="moz-cite-prefix"><br>
    </div>
    <div class="moz-cite-prefix">You're certainly right : WolfSSH writes
      here <a class="moz-txt-link-freetext" href="https://github.com/wolfSSL/wolfssh">https://github.com/wolfSSL/wolfssh</a> : "wolfSSH is dependent on
      wolfCrypt, found as a part of wolfSSL".</div>
    <div class="moz-cite-prefix"><br>
    </div>
    <div class="moz-cite-prefix">So it seems we need to bundle it still,
      and it's a good thing we managed to have it more or less compile
      with ESP-IDF 5+ (a few bugs are still being ironed out -
      especially the OPENSSL compatibility layer, which may not be used
      if mongoose does use mbedTLS instead).<br>
    </div>
    <div class="moz-cite-prefix"><br>
    </div>
    <div class="moz-cite-prefix">Regarding the integration, I tried (my
      best :-)) to have it build on both ESP-IDF 3, ESP-IDF 5 (and also
      ESP-IDF 4 with a small patch to the ESP-IDF build system itself).
      Of course I need help for testing / reproducing the builds and I'm
      grateful to those of you sparing some time to do exactly that.</div>
    <div class="moz-cite-prefix"><br>
    </div>
    <div class="moz-cite-prefix">I have another round of PRs waiting for
      some peer-review :
<a class="moz-txt-link-freetext" href="https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/pulls?q=is%3Aopen+is%3Apr+author%3Allange+draft%3Afalse">https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/pulls?q=is%3Aopen+is%3Apr+author%3Allange+draft%3Afalse</a>
      <br>
    </div>
    <div class="moz-cite-prefix"><br>
    </div>
    <div class="moz-cite-prefix">-> @List, do not hesitate to have a
      look - especially for the ones that are a little bit touchy:</div>
    <div class="moz-cite-prefix">
      <ul>
        <li><a class="moz-txt-link-freetext" href="https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/pull/831">https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/pull/831</a>
          is trying to guess whether the multiple switch case
          falls-through are wanted, or not. It impacts the code of
          multiple people, so it would be interesting if those people
          could have a look at it.</li>
        <li><a class="moz-txt-link-freetext" href="https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/pull/835">https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/pull/835</a>
          is also a big piece and would benefit from a peer-review.</li>
      </ul>
      <p>Thanks in advance !<br>
      </p>
    </div>
    <div class="moz-cite-prefix">Regards,<br>
    </div>
    <div class="moz-cite-prefix"><br>
    </div>
    <div class="moz-cite-prefix">Le 20/02/2023 à 08:07, Mark
      Webb-Johnson a écrit :<br>
    </div>
    <blockquote type="cite"
      cite="mid:826742D4-5C7F-402A-B1FD-4389D2F1F5A3@webb-johnson.net">
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      Michael, Ludovic,
      <div><br>
      </div>
      <div>
        <blockquote type="cite"><span style="caret-color: rgb(0, 0, 0);
            color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size:
            14px;">Actually one of the next things I wanted to
            try/determine is why you want/need to use wolfSSL instead of
            mbedTLS, which we've been using since wolfSSL failed so
            miserably about the Let's encrypt root certificate
            transition. Is there an issue with mbedTLS in idf5?</span><br
            style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);
            font-family: Menlo-Regular; font-size: 14px;">
        </blockquote>
        <div><br>
        </div>
        My memory was that wolfssh required wolfssl? But I may be
        mistaken.</div>
      <div><br>
        <blockquote type="cite"><span style="caret-color: rgb(0, 0, 0);
            color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size:
            14px;">We still need to decide on the integration way.
            Mark's suggestion was doing the idf5 transition in a
            separate branch that will later become the new "master".
            That would keep the idf3 build clean from any regressions,
            but need merging any work done on that branch into the idf5
            branch, which may become a lot of additional work, depending
            on the time needed to finish the transition.</span></blockquote>
        <div><br>
        </div>
        If this can cleanly be applied to the existing work, to allow
        building under either v3 or v5 IDF, that would be ideal. IMHO,
        the separate branch would only be required if the changes were
        very extensive and breaking to v3.</div>
      <div><br>
      </div>
      <div>Regards, Mark.<br>
      </div>
    </blockquote>
    <br>
    <div id="grammalecte_menu_main_button_shadow_host" style="width:
      0px; height: 0px;"></div>
  </body>
</html>