* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 @ 2018-06-26 21:43 Fabrice Fontaine 2018-06-27 3:36 ` Baruch Siach 2018-08-05 12:39 ` Thomas Petazzoni 0 siblings, 2 replies; 14+ messages in thread From: Fabrice Fontaine @ 2018-06-26 21:43 UTC (permalink / raw) To: buildroot gnuradio needs host-python2 however there is no way to enforce this so add a dependency on !BR2_PACKAGE_PYTHON3. Indeed, if BR2_PACKAGE_PYTHON3 is selected, then buildroot will only build host-python3. This issue was not raised in the previous version of gnuradio as host-python-cheetah was the only dependency and host-python-cheetah enforces python2 through HOST_PYTHON_CHEETAH_NEEDS_HOST_PYTHON however we now have host-python-six and host-python-mako dependencies and it does not seem right to enforces python2 on those packages Fixes: - http://autobuild.buildroot.net/results/a502fef3e2a40f38ca63332440b0b6bd2ed11d5a Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> --- package/gnuradio/Config.in | 2 ++ 1 file changed, 2 insertions(+) diff --git a/package/gnuradio/Config.in b/package/gnuradio/Config.in index f748b902a7..25131fb105 100644 --- a/package/gnuradio/Config.in +++ b/package/gnuradio/Config.in @@ -1,5 +1,6 @@ comment "gnuradio needs a toolchain w/ C++, NPTL, wchar, dynamic library" depends on BR2_USE_MMU + depends on !BR2_PACKAGE_PYTHON3 depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \ !BR2_TOOLCHAIN_HAS_THREADS_NPTL || BR2_STATIC_LIBS @@ -10,6 +11,7 @@ config BR2_PACKAGE_GNURADIO depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL depends on BR2_USE_MMU # use fork() depends on BR2_USE_WCHAR # boost + depends on !BR2_PACKAGE_PYTHON3 select BR2_PACKAGE_BOOST select BR2_PACKAGE_BOOST_DATE_TIME select BR2_PACKAGE_BOOST_FILESYSTEM -- 2.14.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-26 21:43 [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 Fabrice Fontaine @ 2018-06-27 3:36 ` Baruch Siach 2018-06-27 5:35 ` Fabrice Fontaine 2018-08-05 12:39 ` Thomas Petazzoni 1 sibling, 1 reply; 14+ messages in thread From: Baruch Siach @ 2018-06-27 3:36 UTC (permalink / raw) To: buildroot Hi Fabrice, On Tue, Jun 26, 2018 at 11:43:26PM +0200, Fabrice Fontaine wrote: > gnuradio needs host-python2 however there is no way to enforce this so > add a dependency on !BR2_PACKAGE_PYTHON3. > Indeed, if BR2_PACKAGE_PYTHON3 is selected, then buildroot will only > build host-python3. Why not just add host-python to _DEPENDENCIES? baruch > This issue was not raised in the previous version of gnuradio as > host-python-cheetah was the only dependency and host-python-cheetah > enforces python2 through HOST_PYTHON_CHEETAH_NEEDS_HOST_PYTHON however > we now have host-python-six and host-python-mako dependencies and it > does not seem right to enforces python2 on those packages > > Fixes: > - http://autobuild.buildroot.net/results/a502fef3e2a40f38ca63332440b0b6bd2ed11d5a > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > --- > package/gnuradio/Config.in | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/package/gnuradio/Config.in b/package/gnuradio/Config.in > index f748b902a7..25131fb105 100644 > --- a/package/gnuradio/Config.in > +++ b/package/gnuradio/Config.in > @@ -1,5 +1,6 @@ > comment "gnuradio needs a toolchain w/ C++, NPTL, wchar, dynamic library" > depends on BR2_USE_MMU > + depends on !BR2_PACKAGE_PYTHON3 > depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \ > !BR2_TOOLCHAIN_HAS_THREADS_NPTL || BR2_STATIC_LIBS > > @@ -10,6 +11,7 @@ config BR2_PACKAGE_GNURADIO > depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL > depends on BR2_USE_MMU # use fork() > depends on BR2_USE_WCHAR # boost > + depends on !BR2_PACKAGE_PYTHON3 > select BR2_PACKAGE_BOOST > select BR2_PACKAGE_BOOST_DATE_TIME > select BR2_PACKAGE_BOOST_FILESYSTEM -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-27 3:36 ` Baruch Siach @ 2018-06-27 5:35 ` Fabrice Fontaine 2018-06-27 16:24 ` Fabrice Fontaine 0 siblings, 1 reply; 14+ messages in thread From: Fabrice Fontaine @ 2018-06-27 5:35 UTC (permalink / raw) To: buildroot Dear Baruch, 2018-06-27 5:36 GMT+02:00 Baruch Siach <baruch@tkos.co.il>: > Hi Fabrice, > > On Tue, Jun 26, 2018 at 11:43:26PM +0200, Fabrice Fontaine wrote: > > gnuradio needs host-python2 however there is no way to enforce this so > > add a dependency on !BR2_PACKAGE_PYTHON3. > > Indeed, if BR2_PACKAGE_PYTHON3 is selected, then buildroot will only > > build host-python3. > > Why not just add host-python to _DEPENDENCIES? > Indeed, I will update my patch. Thanks for your help. > > baruch > > > This issue was not raised in the previous version of gnuradio as > > host-python-cheetah was the only dependency and host-python-cheetah > > enforces python2 through HOST_PYTHON_CHEETAH_NEEDS_HOST_PYTHON however > > we now have host-python-six and host-python-mako dependencies and it > > does not seem right to enforces python2 on those packages > > > > Fixes: > > - http://autobuild.buildroot.net/results/a502fef3e2a40f38ca63332440b0b6 > bd2ed11d5a > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > --- > > package/gnuradio/Config.in | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/package/gnuradio/Config.in b/package/gnuradio/Config.in > > index f748b902a7..25131fb105 100644 > > --- a/package/gnuradio/Config.in > > +++ b/package/gnuradio/Config.in > > @@ -1,5 +1,6 @@ > > comment "gnuradio needs a toolchain w/ C++, NPTL, wchar, dynamic > library" > > depends on BR2_USE_MMU > > + depends on !BR2_PACKAGE_PYTHON3 > > depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \ > > !BR2_TOOLCHAIN_HAS_THREADS_NPTL || BR2_STATIC_LIBS > > > > @@ -10,6 +11,7 @@ config BR2_PACKAGE_GNURADIO > > depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL > > depends on BR2_USE_MMU # use fork() > > depends on BR2_USE_WCHAR # boost > > + depends on !BR2_PACKAGE_PYTHON3 > > select BR2_PACKAGE_BOOST > > select BR2_PACKAGE_BOOST_DATE_TIME > > select BR2_PACKAGE_BOOST_FILESYSTEM > > -- > http://baruch.siach.name/blog/ ~. .~ Tk Open > Systems > =}------------------------------------------------ooO--U-- > Ooo------------{= > - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - > Best Regards, Fabrice -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180627/4a151f11/attachment.html> ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-27 5:35 ` Fabrice Fontaine @ 2018-06-27 16:24 ` Fabrice Fontaine 2018-06-28 3:25 ` Baruch Siach 0 siblings, 1 reply; 14+ messages in thread From: Fabrice Fontaine @ 2018-06-27 16:24 UTC (permalink / raw) To: buildroot Dear Baruch 2018-06-27 7:35 GMT+02:00 Fabrice Fontaine <fontaine.fabrice@gmail.com>: > Dear Baruch, > > 2018-06-27 5:36 GMT+02:00 Baruch Siach <baruch@tkos.co.il>: > >> Hi Fabrice, >> >> On Tue, Jun 26, 2018 at 11:43:26PM +0200, Fabrice Fontaine wrote: >> > gnuradio needs host-python2 however there is no way to enforce this so >> > add a dependency on !BR2_PACKAGE_PYTHON3. >> > Indeed, if BR2_PACKAGE_PYTHON3 is selected, then buildroot will only >> > build host-python3. >> >> Why not just add host-python to _DEPENDENCIES? >> > Indeed, I will update my patch. Thanks for your help. > After testing this option, this does not work. host-python2 is indeed built but host-python3 is also built. As a result, host-python-mako and host-python-six are built only for host-python3 and not for host-python2. I don't know if this the expected behavior,perhaps buildroot should always build the python2 and python3 host packages? > baruch >> >> > This issue was not raised in the previous version of gnuradio as >> > host-python-cheetah was the only dependency and host-python-cheetah >> > enforces python2 through HOST_PYTHON_CHEETAH_NEEDS_HOST_PYTHON however >> > we now have host-python-six and host-python-mako dependencies and it >> > does not seem right to enforces python2 on those packages >> > >> > Fixes: >> > - http://autobuild.buildroot.net/results/a502fef3e2a40f38ca633 >> 32440b0b6bd2ed11d5a >> > >> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> >> > --- >> > package/gnuradio/Config.in | 2 ++ >> > 1 file changed, 2 insertions(+) >> > >> > diff --git a/package/gnuradio/Config.in b/package/gnuradio/Config.in >> > index f748b902a7..25131fb105 100644 >> > --- a/package/gnuradio/Config.in >> > +++ b/package/gnuradio/Config.in >> > @@ -1,5 +1,6 @@ >> > comment "gnuradio needs a toolchain w/ C++, NPTL, wchar, dynamic >> library" >> > depends on BR2_USE_MMU >> > + depends on !BR2_PACKAGE_PYTHON3 >> > depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \ >> > !BR2_TOOLCHAIN_HAS_THREADS_NPTL || BR2_STATIC_LIBS >> > >> > @@ -10,6 +11,7 @@ config BR2_PACKAGE_GNURADIO >> > depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL >> > depends on BR2_USE_MMU # use fork() >> > depends on BR2_USE_WCHAR # boost >> > + depends on !BR2_PACKAGE_PYTHON3 >> > select BR2_PACKAGE_BOOST >> > select BR2_PACKAGE_BOOST_DATE_TIME >> > select BR2_PACKAGE_BOOST_FILESYSTEM >> >> -- >> http://baruch.siach.name/blog/ ~. .~ Tk Open >> Systems >> =}------------------------------------------------ooO--U--Oo >> o------------{= >> - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - >> > Best Regards, > > Fabrice > Best Regards, Fabrice -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180627/c7b36b0e/attachment.html> ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-27 16:24 ` Fabrice Fontaine @ 2018-06-28 3:25 ` Baruch Siach 2018-06-28 7:58 ` Yegor Yefremov 2018-06-28 18:46 ` Trent Piepho 0 siblings, 2 replies; 14+ messages in thread From: Baruch Siach @ 2018-06-28 3:25 UTC (permalink / raw) To: buildroot Hi Fabrice, On Wed, Jun 27, 2018 at 06:24:34PM +0200, Fabrice Fontaine wrote: > 2018-06-27 7:35 GMT+02:00 Fabrice Fontaine <fontaine.fabrice@gmail.com>: > > 2018-06-27 5:36 GMT+02:00 Baruch Siach <baruch@tkos.co.il>: > >> On Tue, Jun 26, 2018 at 11:43:26PM +0200, Fabrice Fontaine wrote: > >> > gnuradio needs host-python2 however there is no way to enforce this so > >> > add a dependency on !BR2_PACKAGE_PYTHON3. > >> > Indeed, if BR2_PACKAGE_PYTHON3 is selected, then buildroot will only > >> > build host-python3. > >> > >> Why not just add host-python to _DEPENDENCIES? > >> > > Indeed, I will update my patch. Thanks for your help. > > After testing this option, this does not work. host-python2 is indeed built > but host-python3 is also built. > As a result, host-python-mako and host-python-six are built only for > host-python3 and not for host-python2. I don't know if this the expected > behavior,perhaps buildroot should always build the python2 and python3 host > packages? Not sure what the right solution here is. Yegor, do you have an idea? baruch > >> > This issue was not raised in the previous version of gnuradio as > >> > host-python-cheetah was the only dependency and host-python-cheetah > >> > enforces python2 through HOST_PYTHON_CHEETAH_NEEDS_HOST_PYTHON however > >> > we now have host-python-six and host-python-mako dependencies and it > >> > does not seem right to enforces python2 on those packages > >> > > >> > Fixes: > >> > - http://autobuild.buildroot.net/results/a502fef3e2a40f38ca633 > >> 32440b0b6bd2ed11d5a > >> > > >> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > >> > --- > >> > package/gnuradio/Config.in | 2 ++ > >> > 1 file changed, 2 insertions(+) > >> > > >> > diff --git a/package/gnuradio/Config.in b/package/gnuradio/Config.in > >> > index f748b902a7..25131fb105 100644 > >> > --- a/package/gnuradio/Config.in > >> > +++ b/package/gnuradio/Config.in > >> > @@ -1,5 +1,6 @@ > >> > comment "gnuradio needs a toolchain w/ C++, NPTL, wchar, dynamic > >> library" > >> > depends on BR2_USE_MMU > >> > + depends on !BR2_PACKAGE_PYTHON3 > >> > depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \ > >> > !BR2_TOOLCHAIN_HAS_THREADS_NPTL || BR2_STATIC_LIBS > >> > > >> > @@ -10,6 +11,7 @@ config BR2_PACKAGE_GNURADIO > >> > depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL > >> > depends on BR2_USE_MMU # use fork() > >> > depends on BR2_USE_WCHAR # boost > >> > + depends on !BR2_PACKAGE_PYTHON3 > >> > select BR2_PACKAGE_BOOST > >> > select BR2_PACKAGE_BOOST_DATE_TIME > >> > select BR2_PACKAGE_BOOST_FILESYSTEM -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il - ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-28 3:25 ` Baruch Siach @ 2018-06-28 7:58 ` Yegor Yefremov 2018-06-28 8:23 ` Thomas Petazzoni 2018-06-28 18:46 ` Trent Piepho 1 sibling, 1 reply; 14+ messages in thread From: Yegor Yefremov @ 2018-06-28 7:58 UTC (permalink / raw) To: buildroot Hi Fabrice, Baruch, On Thu, Jun 28, 2018 at 5:25 AM, Baruch Siach <baruch@tkos.co.il> wrote: > Hi Fabrice, > > On Wed, Jun 27, 2018 at 06:24:34PM +0200, Fabrice Fontaine wrote: >> 2018-06-27 7:35 GMT+02:00 Fabrice Fontaine <fontaine.fabrice@gmail.com>: >> > 2018-06-27 5:36 GMT+02:00 Baruch Siach <baruch@tkos.co.il>: >> >> On Tue, Jun 26, 2018 at 11:43:26PM +0200, Fabrice Fontaine wrote: >> >> > gnuradio needs host-python2 however there is no way to enforce this so >> >> > add a dependency on !BR2_PACKAGE_PYTHON3. >> >> > Indeed, if BR2_PACKAGE_PYTHON3 is selected, then buildroot will only >> >> > build host-python3. >> >> >> >> Why not just add host-python to _DEPENDENCIES? >> >> >> > Indeed, I will update my patch. Thanks for your help. >> >> After testing this option, this does not work. host-python2 is indeed built >> but host-python3 is also built. >> As a result, host-python-mako and host-python-six are built only for >> host-python3 and not for host-python2. I don't know if this the expected >> behavior,perhaps buildroot should always build the python2 and python3 host >> packages? > > Not sure what the right solution here is. > > Yegor, do you have an idea? This patch did it for me. I've made "make defconfig", select ARM for A8 series, Linaro toolchain and python 2. Then I've just selected gnuradio and its python support. In this case no host-python3 will be built because we select general Python 2 support for the target (as gnuradio enforces) and hence only host-python will be built. @Fabrice have you performed "make clean" after adding host-python to the dependencies? diff --git a/package/gnuradio/gnuradio.mk b/package/gnuradio/gnuradio.mk index e79f848388..f2486a2fd2 100644 --- a/package/gnuradio/gnuradio.mk +++ b/package/gnuradio/gnuradio.mk @@ -14,6 +14,7 @@ GNURADIO_SUPPORTS_IN_SOURCE_BUILD = NO # host-python-cheetah, host-python-mako and host-python-six are needed for volk # to compile GNURADIO_DEPENDENCIES = \ + host-python \ host-python-cheetah \ host-python-mako \ host-python-six \ Yegor ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-28 7:58 ` Yegor Yefremov @ 2018-06-28 8:23 ` Thomas Petazzoni 2018-06-28 9:15 ` Yegor Yefremov 0 siblings, 1 reply; 14+ messages in thread From: Thomas Petazzoni @ 2018-06-28 8:23 UTC (permalink / raw) To: buildroot Hello Yegor, On Thu, 28 Jun 2018 09:58:21 +0200, Yegor Yefremov wrote: > This patch did it for me. I've made "make defconfig", select ARM for > A8 series, Linaro toolchain and python 2. Then I've just selected > gnuradio and its python support. > > In this case no host-python3 will be built because we select general > Python 2 support for the target (as gnuradio enforces) and hence only > host-python will be built. Did you test with python3 enabled for the target ? Then you will see the problem that Fabrice is facing. When python3 is enabled for the target, then the default host python version is python3. And due to this, all external host Python modules are installed for host python3, and not available for host python2. Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-28 8:23 ` Thomas Petazzoni @ 2018-06-28 9:15 ` Yegor Yefremov 2018-06-28 9:20 ` Thomas Petazzoni 0 siblings, 1 reply; 14+ messages in thread From: Yegor Yefremov @ 2018-06-28 9:15 UTC (permalink / raw) To: buildroot Hi Thomas, On Thu, Jun 28, 2018 at 10:23 AM, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > Hello Yegor, > > On Thu, 28 Jun 2018 09:58:21 +0200, Yegor Yefremov wrote: > >> This patch did it for me. I've made "make defconfig", select ARM for >> A8 series, Linaro toolchain and python 2. Then I've just selected >> gnuradio and its python support. >> >> In this case no host-python3 will be built because we select general >> Python 2 support for the target (as gnuradio enforces) and hence only >> host-python will be built. > > Did you test with python3 enabled for the target ? > > Then you will see the problem that Fabrice is facing. > > When python3 is enabled for the target, then the default host python > version is python3. And due to this, all external host Python modules > are installed for host python3, and not available for host python2. You're right. Then for now I see only one quick and dirty hack - install host versions of six and mako when installing host-python by default. Yegor ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-28 9:15 ` Yegor Yefremov @ 2018-06-28 9:20 ` Thomas Petazzoni 2018-06-28 10:02 ` Yegor Yefremov 0 siblings, 1 reply; 14+ messages in thread From: Thomas Petazzoni @ 2018-06-28 9:20 UTC (permalink / raw) To: buildroot Hello, On Thu, 28 Jun 2018 11:15:36 +0200, Yegor Yefremov wrote: > > When python3 is enabled for the target, then the default host python > > version is python3. And due to this, all external host Python modules > > are installed for host python3, and not available for host python2. > > You're right. Then for now I see only one quick and dirty hack - > install host versions of six and mako when installing host-python by > default. We also need to take into account the requirements of: http://patchwork.ozlabs.org/patch/906837/ http://patchwork.ozlabs.org/patch/906838/ when designing a solution to solve this problem. Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-28 9:20 ` Thomas Petazzoni @ 2018-06-28 10:02 ` Yegor Yefremov 2018-06-28 17:25 ` Fabrice Fontaine 0 siblings, 1 reply; 14+ messages in thread From: Yegor Yefremov @ 2018-06-28 10:02 UTC (permalink / raw) To: buildroot On Thu, Jun 28, 2018 at 11:20 AM, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > Hello, > > On Thu, 28 Jun 2018 11:15:36 +0200, Yegor Yefremov wrote: > >> > When python3 is enabled for the target, then the default host python >> > version is python3. And due to this, all external host Python modules >> > are installed for host python3, and not available for host python2. >> >> You're right. Then for now I see only one quick and dirty hack - >> install host versions of six and mako when installing host-python by >> default. > > We also need to take into account the requirements of: > > http://patchwork.ozlabs.org/patch/906837/ > http://patchwork.ozlabs.org/patch/906838/ > > when designing a solution to solve this problem. @Fabrice: could you try the latest next branch[1]? It has some Python 3 related fixes. Perhaps one doesn't need to enforce python 2 anymore? [1] https://github.com/gnuradio/gnuradio/commits/next Yegor ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-28 10:02 ` Yegor Yefremov @ 2018-06-28 17:25 ` Fabrice Fontaine 2018-07-19 15:02 ` Thomas Petazzoni 0 siblings, 1 reply; 14+ messages in thread From: Fabrice Fontaine @ 2018-06-28 17:25 UTC (permalink / raw) To: buildroot Dear Yegor, 2018-06-28 12:02 GMT+02:00 Yegor Yefremov <yegorslists@googlemail.com>: > On Thu, Jun 28, 2018 at 11:20 AM, Thomas Petazzoni > <thomas.petazzoni@bootlin.com> wrote: > > Hello, > > > > On Thu, 28 Jun 2018 11:15:36 +0200, Yegor Yefremov wrote: > > > >> > When python3 is enabled for the target, then the default host python > >> > version is python3. And due to this, all external host Python modules > >> > are installed for host python3, and not available for host python2. > >> > >> You're right. Then for now I see only one quick and dirty hack - > >> install host versions of six and mako when installing host-python by > >> default. > > > > We also need to take into account the requirements of: > > > > http://patchwork.ozlabs.org/patch/906837/ > > http://patchwork.ozlabs.org/patch/906838/ > > > > when designing a solution to solve this problem. > > @Fabrice: could you try the latest next branch[1]? It has some Python > 3 related fixes. Perhaps one doesn't need to enforce python 2 anymore? > > [1] https://github.com/gnuradio/gnuradio/commits/next Next branch builds fine however it seems there is outstanding issues: https://github.com/gnuradio/gnuradio/issues/1842. What should we do until an official release is made with python3 support? > > > Yegor > Best Regards, Fabrice -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180628/36e258ed/attachment.html> ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-28 17:25 ` Fabrice Fontaine @ 2018-07-19 15:02 ` Thomas Petazzoni 0 siblings, 0 replies; 14+ messages in thread From: Thomas Petazzoni @ 2018-07-19 15:02 UTC (permalink / raw) To: buildroot Hello, On Thu, 28 Jun 2018 19:25:32 +0200, Fabrice Fontaine wrote: > Next branch builds fine however it seems there is outstanding issues: > https://github.com/gnuradio/gnuradio/issues/1842. What should we do until > an official release is made with python3 support? In the mean time, I think I will just apply your patch that adds the "depends on !BR2_PACKAGE_PYTHON3". It's not great, but it's not the only simple approach to resolve the build failures. In today's Buildroot, if a package needs host Python modules in the Python version which is not the default one, this is not supported, and it will be difficult to support this. Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-28 3:25 ` Baruch Siach 2018-06-28 7:58 ` Yegor Yefremov @ 2018-06-28 18:46 ` Trent Piepho 1 sibling, 0 replies; 14+ messages in thread From: Trent Piepho @ 2018-06-28 18:46 UTC (permalink / raw) To: buildroot On Thu, 2018-06-28 at 06:25 +0300, Baruch Siach wrote: > Hi Fabrice, > > On Wed, Jun 27, 2018 at 06:24:34PM +0200, Fabrice Fontaine wrote: > > 2018-06-27 7:35 GMT+02:00 Fabrice Fontaine <fontaine.fabrice@gmail. > > com>: > > > 2018-06-27 5:36 GMT+02:00 Baruch Siach <baruch@tkos.co.il>: > > > > On Tue, Jun 26, 2018 at 11:43:26PM +0200, Fabrice Fontaine > > > > wrote: > > > > > gnuradio needs host-python2 however there is no way to > > > > > enforce this so > > > > > add a dependency on !BR2_PACKAGE_PYTHON3. > > > > > Indeed, if BR2_PACKAGE_PYTHON3 is selected, then buildroot > > > > > will only > > > > > build host-python3. > > > > > > > > Why not just add host-python to _DEPENDENCIES? > > > > > > > > > > Indeed, I will update my patch. Thanks for your help. > > > > After testing this option, this does not work. host-python2 is > > indeed built > > but host-python3 is also built. > > As a result, host-python-mako and host-python-six are built only > > for > > host-python3 and not for host-python2. I don't know if this the > > expected > > behavior,perhaps buildroot should always build the python2 and > > python3 host > > packages? > > Not sure what the right solution here is. > > Yegor, do you have an idea? I sent a patch series out some time ago that allows selecting the host python version to be used for creating python packages. It would allow selecting python2 to be used for host python packages, and still get the host-python3 *interpreter* built if something needs that too. However, it does not allow: Building host python packages for multiple python versions. Building host python packages for a different python version than is used on the target, if the target has python. So if we start with the requirement that we must have host-python-mako for python2, then from that follows that host-python-six and host- python-etcetc must be for python2 as well. And if the target has python, it must also be python2. Fixing this is hard to to do in buildroot. It ends up with the need to build a package twice, once for python2 and again for python3. Buildroot doesn't like two builds of the same package. Maybe turn host python packages into two packages? host-python2-mako and host-python3-mako. eval host-python-package creates one or both packages. Maybe that could fit in. Or maybe that's too complex. You get to pick one python version and all python packages are that version. If you want a python2 only package and a python3 only package, too bad. Port one of them to the other python version. As python3 support grows, this will be less and less of a problem. ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 2018-06-26 21:43 [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 Fabrice Fontaine 2018-06-27 3:36 ` Baruch Siach @ 2018-08-05 12:39 ` Thomas Petazzoni 1 sibling, 0 replies; 14+ messages in thread From: Thomas Petazzoni @ 2018-08-05 12:39 UTC (permalink / raw) To: buildroot Hello, On Tue, 26 Jun 2018 23:43:26 +0200, Fabrice Fontaine wrote: > gnuradio needs host-python2 however there is no way to enforce this so > add a dependency on !BR2_PACKAGE_PYTHON3. > Indeed, if BR2_PACKAGE_PYTHON3 is selected, then buildroot will only > build host-python3. > This issue was not raised in the previous version of gnuradio as > host-python-cheetah was the only dependency and host-python-cheetah > enforces python2 through HOST_PYTHON_CHEETAH_NEEDS_HOST_PYTHON however > we now have host-python-six and host-python-mako dependencies and it > does not seem right to enforces python2 on those packages > > Fixes: > - http://autobuild.buildroot.net/results/a502fef3e2a40f38ca63332440b0b6bd2ed11d5a > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > --- > package/gnuradio/Config.in | 2 ++ > 1 file changed, 2 insertions(+) Applied to master, thanks. Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2018-08-05 12:39 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-06-26 21:43 [Buildroot] [PATCH 1/1] gnuradio: needs host-python2 Fabrice Fontaine 2018-06-27 3:36 ` Baruch Siach 2018-06-27 5:35 ` Fabrice Fontaine 2018-06-27 16:24 ` Fabrice Fontaine 2018-06-28 3:25 ` Baruch Siach 2018-06-28 7:58 ` Yegor Yefremov 2018-06-28 8:23 ` Thomas Petazzoni 2018-06-28 9:15 ` Yegor Yefremov 2018-06-28 9:20 ` Thomas Petazzoni 2018-06-28 10:02 ` Yegor Yefremov 2018-06-28 17:25 ` Fabrice Fontaine 2018-07-19 15:02 ` Thomas Petazzoni 2018-06-28 18:46 ` Trent Piepho 2018-08-05 12:39 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox