Re: [PATCH v1 kbuild for-next 1/2] makefiles: add config option to force all cc warnings to errors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jonathan Toppins schreef op di 17-03-2015 om 19:15 [-0400]:
> So default to "n"?

"n" is the default anyhow, so I think you never need it.

> Kconfig bases paragraph continuation on indention, 
> correct?

Yes. It's a syntax I quite dislike, but we're stuck with it.

> Appreciate the review thanks.

My review ended up a bit blunt. But the patch defaults to y for every
configuration (because, I think, all of them source lib/Kconfig.debug)
for an option that would stop the build at the first warning it sees.
That's not really a considerate thing to do.


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux