[coreboot] flashrom: Remove Makefile pciutils check, and don't rm *~ in clean

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Wed Jul 2 04:40:34 CEST 2008


On 02.07.2008 00:18, Peter Stuge wrote:
> Patch attached.
>
> I have on two occasions in a short period of time run into systems
> where the pciutils check fails, while both packages are in fact
> installed. In one case, no compiler was installed. The other case
> has not been resolved yet, but the fact is that the check is not
> precise enough.
>
> It is also a waste of time to do the check on every single make
> invocation.
>   

Stefan alread commented on this.

> The other hunk is about make clean. I think make clean should only
> remove files that were created by other targets in the Makefile.
> On principle I don't like make clean removing backups that someone
> may want to keep. (I haven't gotten bit by this yet, but I would be
> upset if it should happen.)
>
> Hope you agree on these! :)
>   

Fully agreed on the non-removal of backup files.

> flashrom: Remove Makefile pciutils check, and don't rm *~ in clean
>
> Signed-off-by: Peter Stuge <peter at stuge.se>

The "don't rm *~ in clean" part is
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Regards,
Carl-Daniel




More information about the coreboot mailing list