* re: [RESEND] [PATCH] tools/python: Pass linker to Python build process
@ 2020-09-29 2:27 Pry Mar
2020-09-29 5:56 ` Jan Beulich
0 siblings, 1 reply; 3+ messages in thread
From: Pry Mar @ 2020-09-29 2:27 UTC (permalink / raw)
To: xen-devel; +Cc: ehem+xen, ijackson
>Unexpectedly the environment variable which needs to be passed is
>$LDSHARED and not $LD. Otherwise Python may find the build `ld` instead
>of the host `ld`.
>
>Replace $(LDFLAGS) with $(SHLIB_LDFLAGS) as Python needs shared objects
>it can load at runtime, not executables.
>
>This uses $(CC) instead of $(LD) since Python distutils appends $CFLAGS
>to $LDFLAGS which breaks many linkers.
>
>Signed-off-by: Elliott Mitchell <ehem+xen@xxxxxxx>
Tested-by: Mark Pryor <pryorm09@xxxxxxxx>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [RESEND] [PATCH] tools/python: Pass linker to Python build process
2020-09-29 2:27 [RESEND] [PATCH] tools/python: Pass linker to Python build process Pry Mar
@ 2020-09-29 5:56 ` Jan Beulich
0 siblings, 0 replies; 3+ messages in thread
From: Jan Beulich @ 2020-09-29 5:56 UTC (permalink / raw)
To: Pry Mar; +Cc: xen-devel, ehem+xen, ijackson
On 29.09.2020 04:27, Pry Mar wrote:
>> Unexpectedly the environment variable which needs to be passed is
>> $LDSHARED and not $LD. Otherwise Python may find the build `ld` instead
>> of the host `ld`.
>>
>> Replace $(LDFLAGS) with $(SHLIB_LDFLAGS) as Python needs shared objects
>> it can load at runtime, not executables.
>>
>> This uses $(CC) instead of $(LD) since Python distutils appends $CFLAGS
>> to $LDFLAGS which breaks many linkers.
>>
>> Signed-off-by: Elliott Mitchell <ehem+xen@xxxxxxx>
>
> Tested-by: Mark Pryor <pryorm09@xxxxxxxx>
Just fyi: With this zapping of the domain that looks to have appeared
on your side (list archive has the same), I'm afraid such a tag is
unusable. While one might infer it from From: of the mail, there's no
guarantee that what's intended.
Jan
^ permalink raw reply [flat|nested] 3+ messages in thread
* [RESEND] [PATCH] tools/python: Pass linker to Python build process
2020-09-11 16:37 Elliott Mitchell
@ 2020-09-28 1:26 ` Elliott Mitchell
0 siblings, 0 replies; 3+ messages in thread
From: Elliott Mitchell @ 2020-09-28 1:26 UTC (permalink / raw)
To: xen-devel
Cc: Ian Jackson, Wei Liu, Marek =?unknown-8bit?Q?Marczykowski-G=C3=B3recki?=
Unexpectedly the environment variable which needs to be passed is
$LDSHARED and not $LD. Otherwise Python may find the build `ld` instead
of the host `ld`.
Replace $(LDFLAGS) with $(SHLIB_LDFLAGS) as Python needs shared objects
it can load at runtime, not executables.
This uses $(CC) instead of $(LD) since Python distutils appends $CFLAGS
to $LDFLAGS which breaks many linkers.
Signed-off-by: Elliott Mitchell <ehem+xen@m5p.com>
---
Simply resending this.
Having looked around a bit, I believe this is a Python 2/3 compatibility
issue. "distutils" for Python 2 likely lacked a separate $LDSHARED or
$LD variable, whereas Python 3 does have this. Alas this is pointless
due to the above (unless you can cause distutils.py to do the final link
step separately).
---
tools/pygrub/Makefile | 9 +++++----
tools/python/Makefile | 9 +++++----
2 files changed, 10 insertions(+), 8 deletions(-)
diff --git a/tools/pygrub/Makefile b/tools/pygrub/Makefile
index 3063c4998f..37b2146214 100644
--- a/tools/pygrub/Makefile
+++ b/tools/pygrub/Makefile
@@ -3,20 +3,21 @@ XEN_ROOT = $(CURDIR)/../..
include $(XEN_ROOT)/tools/Rules.mk
PY_CFLAGS = $(CFLAGS) $(PY_NOOPT_CFLAGS)
-PY_LDFLAGS = $(LDFLAGS) $(APPEND_LDFLAGS)
+PY_LDFLAGS = $(SHLIB_LDFLAGS) $(APPEND_LDFLAGS)
INSTALL_LOG = build/installed_files.txt
.PHONY: all
all: build
.PHONY: build
build:
- CC="$(CC)" CFLAGS="$(PY_CFLAGS)" LDFLAGS="$(PY_LDFLAGS)" $(PYTHON) setup.py build
+ CC="$(CC)" CFLAGS="$(PY_CFLAGS)" LDSHARED="$(CC)" LDFLAGS="$(PY_LDFLAGS)" $(PYTHON) setup.py build
.PHONY: install
install: all
$(INSTALL_DIR) $(DESTDIR)/$(bindir)
- CC="$(CC)" CFLAGS="$(PY_CFLAGS)" LDFLAGS="$(PY_LDFLAGS)" $(PYTHON) \
- setup.py install --record $(INSTALL_LOG) $(PYTHON_PREFIX_ARG) \
+ CC="$(CC)" CFLAGS="$(PY_CFLAGS)" LDSHARED="$(CC)" \
+ LDFLAGS="$(PY_LDFLAGS)" $(PYTHON) setup.py install \
+ --record $(INSTALL_LOG) $(PYTHON_PREFIX_ARG) \
--root="$(DESTDIR)" --install-scripts=$(LIBEXEC_BIN) --force
set -e; if [ $(bindir) != $(LIBEXEC_BIN) -a \
"`readlink -f $(DESTDIR)/$(bindir)`" != \
diff --git a/tools/python/Makefile b/tools/python/Makefile
index 8d22c03676..b675f5b4de 100644
--- a/tools/python/Makefile
+++ b/tools/python/Makefile
@@ -5,19 +5,20 @@ include $(XEN_ROOT)/tools/Rules.mk
all: build
PY_CFLAGS = $(CFLAGS) $(PY_NOOPT_CFLAGS)
-PY_LDFLAGS = $(LDFLAGS) $(APPEND_LDFLAGS)
+PY_LDFLAGS = $(SHLIB_LDFLAGS) $(APPEND_LDFLAGS)
INSTALL_LOG = build/installed_files.txt
.PHONY: build
build:
- CC="$(CC)" CFLAGS="$(PY_CFLAGS)" $(PYTHON) setup.py build
+ CC="$(CC)" CFLAGS="$(PY_CFLAGS)" LDSHARED="$(CC)" LDFLAGS="$(PY_LDFLAGS)" $(PYTHON) setup.py build
.PHONY: install
install:
$(INSTALL_DIR) $(DESTDIR)$(LIBEXEC_BIN)
- CC="$(CC)" CFLAGS="$(PY_CFLAGS)" LDFLAGS="$(PY_LDFLAGS)" $(PYTHON) \
- setup.py install --record $(INSTALL_LOG) $(PYTHON_PREFIX_ARG) \
+ CC="$(CC)" CFLAGS="$(PY_CFLAGS)" LDSHARED="$(CC)" \
+ LDFLAGS="$(PY_LDFLAGS)" $(PYTHON) setup.py install \
+ --record $(INSTALL_LOG) $(PYTHON_PREFIX_ARG) \
--root="$(DESTDIR)" --force
$(INSTALL_PYTHON_PROG) scripts/convert-legacy-stream $(DESTDIR)$(LIBEXEC_BIN)
--
2.20.1
--
(\___(\___(\______ --=> 8-) EHM <=-- ______/)___/)___/)
\BS ( | ehem+sigmsg@m5p.com PGP 87145445 | ) /
\_CS\ | _____ -O #include <stddisclaimer.h> O- _____ | / _/
8A19\___\_|_/58D2 7E3D DDF4 7BA6 <-PGP-> 41D1 B375 37D0 8714\_|_/___/5445
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-09-29 5:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-29 2:27 [RESEND] [PATCH] tools/python: Pass linker to Python build process Pry Mar
2020-09-29 5:56 ` Jan Beulich
-- strict thread matches above, loose matches on Subject: below --
2020-09-11 16:37 Elliott Mitchell
2020-09-28 1:26 ` [RESEND] " Elliott Mitchell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).