Elia Pinto <[email protected]> writes:

> Add some missing phony target to Makefile.
>
> Signed-off-by: Elia Pinto <[email protected]>
> Helped-by: Matthieu Moy <[email protected]>
> ---
> This is the third version of this patch.

Thanks. I double-checked with

grep .PHONY Makefile | sed 's/.PHONY: //; s/#.*//' | tr ' ' '\n' | sort | uniq 
-c | sort -n

there are no duplicate .PHONY, and all .PHONY targets are indeed .PHONY
ones, so the patch is now

Reviewed-by: Matthieu Moy <[email protected]>

> Compared to the previous I have added only the missing phony 
> target as suggested by Matthieu Moy

Note: my comments were not meant to be an objection to the additional
refactoring. At least some of the hunks definitely made sense to me, but
they are better done in a separate patch.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to