[PATCH] Do not delete make check-abi symlist files
Roland McGrath
roland@hack.frob.com
Thu May 3 18:40:00 GMT 2012
> Now it's really short, thanks.
>
> Ok now?
Looks fine if you remove the commented-out lines and add a clearer comment.
> 2012-05-05 Andreas Jaeger <aj@suse.de>,
> Roland McGrath <roland@hack.frob.com>
Normally no comma for this case. The second name should
be lined up under the first name, not just indented by a tab.
(In Emacs, just hitting TAB twice at bol dtrt.)
> * Makerules (.PRECIOUS): Add .symlist files.
> (generated): Add .symlist files.
For .PRECIOUS, say "Add %.symlist pattern to prevent intermediate file
deletion" or something like that.
> diff --git a/Makerules b/Makerules
> index de87ad6..ab734b4 100644
> --- a/Makerules
> +++ b/Makerules
> @@ -1158,6 +1158,12 @@ ifeq ($(versioning),yes)
>
> vpath %.abilist $(+sysdep_dirs)
>
> +# Make sure that symlist is not removed.
I'd say:
# The .PRECIOUS rule prevents the files built by an implicit rule whose
# target pattern is %.symlist from being considered "intermediate files"
# and automatically removed. We only want these files to be removed by
# 'make clean', which is handled by the 'generated' variable.
Thanks,
Roland
More information about the Libc-alpha
mailing list