Pkg xen devel - Sep 2020 - [PATCH 12/12] Partially revert "Cross-compilation fixes."

Elliott Mitchell writes ("[PATCH 12/12] Partially revert
"Cross-compilation fixes.""):
> This partially reverts commit 16504669c5cbb8b195d20412aadc838da5c428f7.
Wow, that is an upsteam commit from 2005.

However, I would like some kind of explanation.  Is it in fact now
false that
  | # These don't cross-compile
?

Should this patch go upstream ?

Ian.

-- 
Ian Jackson <ijackson at chiark.greenend.org.uk>   These opinions are my
own.

Pronouns: they/he.  If I emailed you from @fyvzl.net or @evade.org.uk,
that is a private address which bypasses my fierce spamfilter.

On 9/17/20 6:21 PM, Ian Jackson wrote:
> Elliott Mitchell writes ("[PATCH 12/12] Partially revert
"Cross-compilation fixes.""):
>> This partially reverts commit 16504669c5cbb8b195d20412aadc838da5c428f7.
> 
> Wow, that is an upsteam commit from 2005.
> 
> However, I would like some kind of explanation.  Is it in fact now
> false that
>   | # These don't cross-compile
> ?
> 
> Should this patch go upstream ?
Yes, that has been done.

It's 69953e28563

Hans

On 9/17/20 6:23 PM, Hans van Kranenburg wrote:
> On 9/17/20 6:21 PM, Ian Jackson wrote:
>> Elliott Mitchell writes ("[PATCH 12/12] Partially revert
"Cross-compilation fixes.""):
>>> This partially reverts commit
16504669c5cbb8b195d20412aadc838da5c428f7.
>>
>> Wow, that is an upsteam commit from 2005.
>>
>> However, I would like some kind of explanation.  Is it in fact now
>> false that
>>   | # These don't cross-compile
>> ?
>>
>> Should this patch go upstream ?
> 
> Yes, that has been done.
> 
> It's 69953e28563
Picked into knorrie/4.14 now, already, so we can't forget it later.

K

On Thu, Sep 17, 2020 at 05:21:32PM +0100, Ian Jackson
wrote:
> Elliott Mitchell writes ("[PATCH 12/12] Partially revert
"Cross-compilation fixes.""):
> > This partially reverts commit
16504669c5cbb8b195d20412aadc838da5c428f7.
> 
> Wow, that is an upsteam commit from 2005.
> 
> However, I would like some kind of explanation.  Is it in fact now
> false that
>   | # These don't cross-compile
> ?
I had the Python portion cross-compiling reliably and so passed it
upstream.  More recently I was testing Han's 4.14 branch and discovered
a bit of breakage which hadn't previously shown up.

The patch for that is "tools/python: Pass linker to Python build
process"
which should already be in your @xenproject.org mailbox...   So far it
has been almost a week and there hasn't been any commentary.


-- 
(\___(\___(\______          --=> 8-) EHM <=--          ______/)___/)___/)
 \BS (    |         ehem+sigmsg at m5p.com  PGP 87145445         |    )   /
  \_CS\   |  _____  -O #include <stddisclaimer.h> O-   _____  |   /  _/
8A19\___\_|_/58D2 7E3D DDF4 7BA6 <-PGP-> 41D1 B375 37D0 8714\_|_/___/5445