Re: Rework of makefiles v5

2018-02-22 Thread Marcus Folkesson
On Wed, Feb 21, 2018 at 10:46:45PM +0100, Nicolas Iooss wrote: > On Thu, Feb 15, 2018 at 2:04 PM, Marcus Folkesson > wrote: > > Hi Nicolas, > > > > First of all, thank you for your review. > > > > On Wed, Feb 14, 2018 at 08:19:44PM +0100, Nicolas Iooss wrote: > >> On Wed, Feb 14, 2018 at 10:57 AM,

Re: Rework of makefiles v5

2018-02-21 Thread Nicolas Iooss
On Thu, Feb 15, 2018 at 2:04 PM, Marcus Folkesson wrote: > Hi Nicolas, > > First of all, thank you for your review. > > On Wed, Feb 14, 2018 at 08:19:44PM +0100, Nicolas Iooss wrote: >> On Wed, Feb 14, 2018 at 10:57 AM, Marcus Folkesson >> wrote: >> > I have updated the patchset. >> > >> > The bi

Re: Rework of makefiles v5

2018-02-15 Thread Marcus Folkesson
Hi Nicolas, First of all, thank you for your review. On Wed, Feb 14, 2018 at 08:19:44PM +0100, Nicolas Iooss wrote: > On Wed, Feb 14, 2018 at 10:57 AM, Marcus Folkesson > wrote: > > I have updated the patchset. > > > > The biggest change is that $(DESTDIR) is now used in the > > install stage on

[PATCH Rework of makefiles v5 16/15] Travis-CI: do not duplicate $DESTDIR in $PYSITEDIR

2018-02-14 Thread Nicolas Iooss
Recent commits removed $DESTDIR from $PYSITEDIR in libselinux and libsemanage: -PYSITEDIR ?= $(DESTDIR)$(shell $(PYTHON) -c 'import site; print(site.getsitepackages()[0])') +PYSITEDIR ?= $(shell $(PYTHON) -c 'import site; print(site.getsitepackages()[0])') As "site.getsitepackages

Re: Rework of makefiles v5

2018-02-14 Thread Nicolas Iooss
On Wed, Feb 14, 2018 at 10:57 AM, Marcus Folkesson wrote: > I have updated the patchset. > > The biggest change is that $(DESTDIR) is now used in the > install stage only. > > Also some overidden CFLAGS/LDFLAGS has been removed since we now have > explicit build rules. > > I have moved the changel

Rework of makefiles v5

2018-02-14 Thread Marcus Folkesson
I have updated the patchset. The biggest change is that $(DESTDIR) is now used in the install stage only. Also some overidden CFLAGS/LDFLAGS has been removed since we now have explicit build rules. I have moved the changelog into patches. Please test to compile with: make DESTDIR=/tmp/myroot PR