<div dir="ltr">Asaf, Thomas,<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Well, only when you wrote that I noticed the package is named<br>python-pybind on Buildroot, but the real package name is actually<br>pybind11. So yes, I think it should be changed to python-pybind11.</blockquote><div><br></div><div>I proposed a patch to rename pybind11 so a potential cmake support in the future</div><div>with $(eval(cmake-package)) does not face problems. <br>The C++11 check is not needed until we make the package a cmake package,</div><div>I prefered to push only the name changes at the moment, we will look into the cmake thing afterwards</div><div><br></div><div>Thomas,</div><div>I will revise the patch I have in standby once this package has been effectively renamed, it needs its dependency</div><div>to 'host-python-pybind' updated</div><div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><div><div><font face="arial,helvetica,sans-serif">Guillaume W. Bres<br></font></div><font face="arial,helvetica,sans-serif">Software engineer</font><br></div><font face="arial,helvetica,sans-serif"></font></div><font face="arial,helvetica,sans-serif"><<a href="mailto:guillaume.bressaix@gmail.com" target="_blank">guillaume.bressaix@gmail.com</a>><br></font></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le ven. 12 juin 2020 à 16:55, Asaf Kahlon <<a href="mailto:asafka7@gmail.com">asafka7@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
<br>
On Fri, Jun 12, 2020 at 10:43 AM Guillaume Bres<br>
<<a href="mailto:guillaume.bressaix@gmail.com" target="_blank">guillaume.bressaix@gmail.com</a>> wrote:<br>
><br>
> Hello,<br>
><br>
> If I understand correctly, this patch only covers the first use case (C++/Python). I only added this one because my following patch requires host-python-pybind, but I agree it certainly lacks a depends on !BR2_INSTALL_LIBSTDCPP. I'll wait for people's point of view before going any further anyway. If you agree on this, I will add the dependency and push a v2<br>
><br>
This patch includes only the installation of pybind11 to the target -<br>
which isn't worth much since it includes only headers. It should be<br>
installed to the staging dir instead. As far as I understand, there's<br>
no point in installing this package to the target at all.<br>
In addition, pay attention you don't only need C++, but at least C++11.<br>
<br>
> The other use case that people might need in the future, is not allowed at the moment until we unlock it with $(eval $(cmake-package)) but requires further work (I am also not familiar with cmake at the moment). Maybe you could proceed with adding the cmake support because you are knowledgeable and already got it working?<br>
><br>
Actually, the cmake part is already implemented in the patch I sent in<br>
the previous mail. Feel free to take any parts you need from it.<br>
<br>
> One last question though: "For example, this project won't find pybind11 because it uses find_package(pybind11...)", so we should rename the package to python-pybind11? so we don't have any other problems when unlocking the cmake option. If you agree on that point, my v2 will be renamed to python-pybind11 & and I will also adapt my following work to this new name<br>
><br>
Well, only when you wrote that I noticed the package is named<br>
python-pybind on Buildroot, but the real package name is actually<br>
pybind11. So yes, I think it should be changed to python-pybind11.<br>
<br>
> Guillaume W. Bres<br>
> Software engineer<br>
> <<a href="mailto:guillaume.bressaix@gmail.com" target="_blank">guillaume.bressaix@gmail.com</a>><br>
><br>
><br>
> Le jeu. 11 juin 2020 à 20:14, Asaf Kahlon <<a href="mailto:asafka7@gmail.com" target="_blank">asafka7@gmail.com</a>> a écrit :<br>
>><br>
>> Hello,<br>
>><br>
>> I'm not sure pybind11 is just a "regular" python package.<br>
>> First, it needs C++11 so we have to make sure the toolchain supports<br>
>> that in order to be able to actually compile the generated pybind11<br>
>> code.<br>
>> Secondly, the installation through setup.py just installs the pybind11<br>
>> headers and it doesn't give any way for C++ projects to locate the<br>
>> package. For example, this project won't find pybind11 because it uses<br>
>> find_package(pybind11...) in it's CMakeLists.txt:<br>
>> <a href="https://github.com/pybind/pybind11_json" rel="noreferrer" target="_blank">https://github.com/pybind/pybind11_json</a>. The installation of pybind11<br>
>> using CMake gives a solution to this problem.<br>
>><br>
>> Those considerations have been taken into account in a previous patch<br>
>> I sent to the mailing list few weeks ago:<br>
>> <a href="https://patchwork.ozlabs.org/project/buildroot/patch/20200513183903.8656-1-asafka7@gmail.com/" rel="noreferrer" target="_blank">https://patchwork.ozlabs.org/project/buildroot/patch/20200513183903.8656-1-asafka7@gmail.com/</a><br>
>><br>
>> Regards,<br>
>> Asaf.<br>
Best regards,<br>
Asaf.<br>
</blockquote></div>