All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] meson: bump version to 0.48.1
Date: Sat, 20 Oct 2018 15:00:53 +0100	[thread overview]
Message-ID: <8dadf2d9-2f53-97cf-0d7f-4d1c323cc2f2@mind.be> (raw)
In-Reply-To: <f6178cb2-5035-a8b7-6a8c-428195fb1eda@mind.be>



On 20/10/2018 13:57, Arnout Vandecappelle wrote:
>  Hi Eric,
>
> On 19/10/2018 22:10, Eric Le Bihan wrote:
>> Hi!
>>
>> On 2018-10-19 20:37, Peter Seiderer wrote:
>>> On Fri, 19 Oct 2018 20:30:50 +0200, Peter Seiderer <ps.report@gmx.net> wrote:
> [snip]
>>>> With the patch applied I get the following error:
>>>>
>>>>>>> host-meson 0.48.1 Building
>>>> (cd .../build/host-meson-0.48.1//; PATH=".../host/bin:.../host/sbin:/usr/local/bin:/usr/bin:/bin:/usr/lib/mit/sbin" PYTHONNOUSERSITE=1  .../host/bin/python3 setup.py build  )
>>>> Traceback (most recent call last):
>>>>   File "setup.py", line 26, in <module>
>>>>     from setuptools import setup
>>>> ModuleNotFoundError: No module named 'setuptools'>>>
>>>> 	$ ls -d build/host-python*
>>>> build/host-python-2.7.15
>>>> build/host-python3-3.7.0
>>>> build/host-python-setuptools-40.0.0
>>> graph-depends.pdf shows the following host-meson dependencies:
>>>
>>> - host-meson --> host-python3
>>> - host-meson --> host-python-setuptools --> host-python
>>> - host-meson --> host-ninja --> host-python
>>>
>>> Maybe host-python-setuptools are build for the wrong python version (python2 instead of python3)?
>  That is correct. When target python or python3 is selected, then the
> corresponding host python is also selected, and *all* host python packages are
> built for that specific python version. But if only target python is selected,
> then you will get host-python as well, but you can also still depend on
> host-python3. In that case, however, all the host python packages will be built
> only for python2, not for python3. Hence the breakage.
>
>  I'm surprised though that we don't see this in the autobuilders... Your patch
> doesn't change anything there, does it?

?It actually does... meson 0.47.1 still had a fallback on distutils if
setuptools wasn't available, but 0.48.1 removed that fallback.


?Regards,
?Arnout


>
>> I got the same issue and thought that was related to using an old build
>> environment and not a fresh one.
>>
>> The package host-python-setuptools was using $(HOST_DIR)/bin/python,
>> symlinked to $(HOST_DIR)/bin/python2.7. Symlinking to python3 and
>> rebuilding host-python-setuptools solves host-meson build issue, but
>> that is not a proper solution.
>>
>> I'm a bit puzzled... If for some reason host-meson had been built before
>> a package requiring host-python2 and host-python-setuptools, would the build
>> of this package fail due to host-python-setuptools having been built
>> for host-python3?
>  No, it purely depends on what you have selected for the target.
>
>  Hm, actually, if you have *nothing* selected for the target, it gets even more
> confusing, because any host package that depends on python 2-or-3 will default
> to python2. I guess it then depends on the build order which version gets
> used... Bad!
>
>  Yann is working on a series to support side-by-side python2 and python3 on the
> host and target. That would solve this issue.
>
>  Regards,
>  Arnout
>
>> Thanks for the review.
>>
>> Regards,
>>
>> --
>> ELB
>> _______________________________________________
>> buildroot mailing list
>> buildroot at busybox.net
>> http://lists.busybox.net/mailman/listinfo/buildroot
>>

  reply	other threads:[~2018-10-20 14:00 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-18 20:44 [Buildroot] [PATCH] meson: bump version to 0.48.1 Eric Le Bihan
2018-10-19  8:08 ` Thomas Petazzoni
2018-10-19 18:30 ` Peter Seiderer
2018-10-19 18:37   ` Peter Seiderer
2018-10-19 21:10     ` Eric Le Bihan
2018-10-20  7:17       ` Peter Seiderer
2018-10-20 12:57       ` Arnout Vandecappelle
2018-10-20 14:00         ` Arnout Vandecappelle [this message]
2018-10-20 15:52           ` Arnout Vandecappelle
2018-10-23 11:17             ` Baruch Siach
2018-10-23 16:44               ` Peter Seiderer
2018-10-23 17:34                 ` Baruch Siach
2018-10-23 18:34                   ` Arnout Vandecappelle
2018-10-21 17:48           ` Eric Le Bihan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8dadf2d9-2f53-97cf-0d7f-4d1c323cc2f2@mind.be \
    --to=arnout@mind.be \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.