Github messages for voidlinux
 help / color / Atom feed
* Re: New packages: python3-breathe and python3-pybind11
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-20218@inbox.vuxu.org>
@ 2020-03-19 18:25 ` Chocimier
  2020-03-19 18:28 ` Chocimier
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 7+ messages in thread
From: Chocimier @ 2020-03-19 18:25 UTC (permalink / raw)
  To: ml

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

New comment by Chocimier on void-packages repository

https://github.com/void-linux/void-packages/pull/20218#issuecomment-601341767

Comment:
Some cross compilation fail. Approach to [use cmake build-style](https://github.com/void-linux/void-packages/pull/18873#issuecomment-597118155) shown by @yopito and @gspe have better chance to work, because this build step is more complex.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: New packages: python3-breathe and python3-pybind11
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-20218@inbox.vuxu.org>
  2020-03-19 18:25 ` New packages: python3-breathe and python3-pybind11 Chocimier
@ 2020-03-19 18:28 ` Chocimier
  2020-03-19 18:56 ` ahesford
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 7+ messages in thread
From: Chocimier @ 2020-03-19 18:28 UTC (permalink / raw)
  To: ml

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

New comment by Chocimier on void-packages repository

https://github.com/void-linux/void-packages/pull/20218#issuecomment-601343191

Comment:
python3-breathe builds fine without python-devel, and can be made noarch.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: New packages: python3-breathe and python3-pybind11
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-20218@inbox.vuxu.org>
  2020-03-19 18:25 ` New packages: python3-breathe and python3-pybind11 Chocimier
  2020-03-19 18:28 ` Chocimier
@ 2020-03-19 18:56 ` ahesford
  2020-03-19 19:01 ` ahesford
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 7+ messages in thread
From: ahesford @ 2020-03-19 18:56 UTC (permalink / raw)
  To: ml

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

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/20218#issuecomment-601358386

Comment:
> python3-breathe builds fine without python-devel, and can be made noarch.

The change has been made with the latest push.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: New packages: python3-breathe and python3-pybind11
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-20218@inbox.vuxu.org>
                   ` (2 preceding siblings ...)
  2020-03-19 18:56 ` ahesford
@ 2020-03-19 19:01 ` ahesford
  2020-03-20 19:56 ` Chocimier
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 7+ messages in thread
From: ahesford @ 2020-03-19 19:01 UTC (permalink / raw)
  To: ml

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

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/20218#issuecomment-601360449

Comment:
> Some cross compilation fail. Approach to [use cmake build-style](https://github.com/void-linux/void-packages/pull/18873#issuecomment-597118155) shown by @yopito and @gspe have better chance to work, because this build step is more complex.

There is no clean way around this. The problem is that the cmake modules invoke the Python interpreter to determine the size of a pointer on the target platform and compare it to the size of the pointer expected by the C++ compiler. The C++ (cross-) compiler returns a 4-byte pointer as expected. The Python interpreter is always native, and returns an 8-byte pointer. The test fails. We can never invoke the Python interpreter for the target arch because its binaries are not compatible with the host architecture.

The only useful thing the cmake build step does is prepare some configuration modules for cmake. I've just copied the relevant modules into `files/` and install them where they need to go in `post_install`. The modules do not appear to be architecture dependent and should be usable on all architectures supported by Void.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: New packages: python3-breathe and python3-pybind11
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-20218@inbox.vuxu.org>
                   ` (3 preceding siblings ...)
  2020-03-19 19:01 ` ahesford
@ 2020-03-20 19:56 ` Chocimier
  2020-03-23 14:07 ` ahesford
  2020-03-23 15:06 ` ahesford
  6 siblings, 0 replies; 7+ messages in thread
From: Chocimier @ 2020-03-20 19:56 UTC (permalink / raw)
  To: ml

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

New comment by Chocimier on void-packages repository

https://github.com/void-linux/void-packages/pull/20218#issuecomment-601884870

Comment:
This forces pybind to be installed on all archs, but if pybind is used by other package, cross build will use incorrect interpreter again, right? I think we should give upstream this as another argument to use [FindPython](https://cmake.org/cmake/help/latest/module/FindPython.html). Then if package is useful without FindPythonLibsNew.cmake, remove it, otherwise reconsider why do we need header only library in repo at all.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: New packages: python3-breathe and python3-pybind11
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-20218@inbox.vuxu.org>
                   ` (4 preceding siblings ...)
  2020-03-20 19:56 ` Chocimier
@ 2020-03-23 14:07 ` ahesford
  2020-03-23 15:06 ` ahesford
  6 siblings, 0 replies; 7+ messages in thread
From: ahesford @ 2020-03-23 14:07 UTC (permalink / raw)
  To: ml

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

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/20218#issuecomment-602615384

Comment:
> This forces pybind to be installed on all archs, but if pybind is used by other package, cross build will use incorrect interpreter again, right? I think we should give upstream this as another argument to use [FindPython](https://cmake.org/cmake/help/latest/module/FindPython.html). Then if package is useful without FindPythonLibsNew.cmake, remove it, otherwise reconsider why do we need header only library in repo at all.

This is a good point, and one I had overlooked. I'll file a problem report upstream and see what shakes out. It would be nice to package pybind11 even though it's a header-only library. We want it for the same reason we want anything in the repo: easy of deployment and management.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: New packages: python3-breathe and python3-pybind11
       [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-20218@inbox.vuxu.org>
                   ` (5 preceding siblings ...)
  2020-03-23 14:07 ` ahesford
@ 2020-03-23 15:06 ` ahesford
  6 siblings, 0 replies; 7+ messages in thread
From: ahesford @ 2020-03-23 15:06 UTC (permalink / raw)
  To: ml

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

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/20218#issuecomment-602615384

Comment:
> This forces pybind to be installed on all archs, but if pybind is used by other package, cross build will use incorrect interpreter again, right? I think we should give upstream this as another argument to use [FindPython](https://cmake.org/cmake/help/latest/module/FindPython.html). Then if package is useful without FindPythonLibsNew.cmake, remove it, otherwise reconsider why do we need header only library in repo at all.

This is a good point, and one I had overlooked. I've [reported the problem upstream](https://github.com/pybind/pybind11/issues/2139) and will see what shakes out. It would be nice to package pybind11 even though it's a header-only library. We want it for the same reason we want anything in the repo: easy of deployment and management.

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, back to index

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-20218@inbox.vuxu.org>
2020-03-19 18:25 ` New packages: python3-breathe and python3-pybind11 Chocimier
2020-03-19 18:28 ` Chocimier
2020-03-19 18:56 ` ahesford
2020-03-19 19:01 ` ahesford
2020-03-20 19:56 ` Chocimier
2020-03-23 14:07 ` ahesford
2020-03-23 15:06 ` ahesford

Github messages for voidlinux

Archives are clonable: git clone --mirror http://inbox.vuxu.org/voidlinux-github

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://inbox.vuxu.org/vuxu.github.voidlinux


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git