Hi On Mon, Sep 21, 2020 at 10:46 AM David Gibson <david@xxxxxxxxxxxxxxxxxxxxx> wrote: > > On Tue, Sep 15, 2020 at 11:27:03PM +0400, marcandre.lureau@xxxxxxxxxx wrote: > > From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx> > > > > setup.py build_ext is run from top_srcdir with Makefile. > > > > ../pylibfdt will produce output files in parent directory. > > Uh.. can you elaborate a bit on what exactly ends up where, and why > the new behaviour you're setting up is better. ~/src/dtc $ make ... ~/src/dtc $ ls ../pylibfdt _libfdt.cpython-38-x86_64-linux-gnu.so It "escapes" the source directory > > > Note that setup.py install will rebuild it with the default 'build' > > directory. There doesn't seem to be a way to override that. > That rings a bell. ISTR it being a real PITA. > It's a minor inconvenience imho. You just end up with an extra build/ directory during install. > > > > Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx> > > > --- > > pylibfdt/Makefile.pylibfdt | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/pylibfdt/Makefile.pylibfdt b/pylibfdt/Makefile.pylibfdt > > index 32ae1c5..1b5f236 100644 > > --- a/pylibfdt/Makefile.pylibfdt > > +++ b/pylibfdt/Makefile.pylibfdt > > @@ -18,7 +18,7 @@ endif > > > > $(PYMODULE): $(PYLIBFDT_srcs) $(LIBFDT_archive) $(SETUP) $(VERSION_FILE) > > @$(VECHO) PYMOD $@ > > - $(PYTHON) $(SETUP) $(SETUPFLAGS) build_ext --build-lib=../$(PYLIBFDT_dir) > > + $(PYTHON) $(SETUP) $(SETUPFLAGS) build_ext --build-lib=$(PYLIBFDT_dir) > > > > install_pylibfdt: $(PYMODULE) > > @$(VECHO) INSTALL-PYLIB > > -- > David Gibson | I'll have my music baroque, and my code > david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ > | _way_ _around_! > http://www.ozlabs.org/~dgibson