Thread (83 messages) flat view 83 messages, 6 authors, 2021-07-06

[Buildroot] [PATCH 04/15] package/pkg-generic.mk: Fix .la files overwrite detection

From: Herve Codina <herve.codina@bootlin.com>
Date: 2021-06-22 18:01:45

On Mon, 21 Jun 2021 22:54:11 +0200
Thomas Petazzoni [off-list ref] wrote:
On Mon, 21 Jun 2021 16:11:19 +0200
Herve Codina [off-list ref] wrote:
quoted
During per-package build, original .la files are modified by
fixup-libtool-files calls.
But since fixup-libtool-files modifies files using sed --in-place,
these modification are done using a temporary file and a call to
rename. Rename breaks the hardlink to the original file and leave the
temporary file in per-package TARGET dir.
As the original file is not modified, this is no longer considered as
an overwrite.

To fix this detection, this patch simply considers the what is done  
"the what" -> "that what"
Fixed.
quoted
by fixup-libtool-files is part of the original snapshot used to
detect overwrites. And so, the original snapshot is taken after
fixup-libtool-files call.

Signed-off-by: Herve Codina <herve.codina@bootlin.com>  
With that:

Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>

Thanks for this first review.

Herv?

-- 
Herv? Codina, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help