Source-Changes-D archive

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

Re: CVS commit: src/sys/arch/evbarm/conf



On 2018/08/19 0:06, Jason Thorpe wrote:
On 2018/08/18 20:08, Rin Okuyama wrote:
Ah, then it should be "if ARM32". It was "if arm32" in rev. 1.31.

Sorry, not 1.31 but 1.30.

That's because all options are normalized to lower-case.

Oops, not "if" but "ifdef".

"ifdef" statements refer "attrtab", whose entries are not normalized to
lower-case. Therefore, "ifdef arm32" becomes false even if we have
"options ARM32", as I wrote in the previous message:

http://mail-index.netbsd.org/source-changes-d/2018/08/18/msg010463.html

Well, this is a very confusing behavior of config(1). We should normalize
everything to lower-case, provided no one depends on this behavior...

rin


Home | Main Index | Thread Index | Old Index