Remove dead code in the cscope target

[def]

Summary

arm64 is the only currently supported architecture that has ${MACHINE_CPUARCH} set to a different value (aarch64) than ${MACHINE} (arm64), as described in arch(7). However, there is no source directory associated with arm64 that has a name set to ${MACHINE_CPUARCH}.

Remove the dead code that adds a directory with a name set to ${MACHINE_CPUARCH} to a list of directories indexed with cscope. This change allows to use the cscope target on arm64.

My branch:
https://github.com/kwitaszczyk/freebsd/tree/cscope-arm64

[imp]

[jhb]

Hmm, does cscope fail because the directory name doesn't exist? (I confess to always using ALL_ARCH=yes myself)

[def]

In D36402#836213, @jhb wrote:

Hmm, does cscope fail because the directory name doesn't exist? (I confess to always using ALL_ARCH=yes myself)

cscope fails because it looks for the aarch64 directory that doesn't exist.

[jhb]

sys/Makefile

10

FYI, the rest of this file does not use indentation for nested .if's, and in general we do not do that in src Makefiles AFAIK. It also adds a bit of noise to this commit for which the real changes are removing 3 lines.

[def]

Remove the style change.

[def]

sys/Makefile

10

I thought that the .if style change applied to freebsd-src as well. I've seen freebsd-ports doing so.

I removed the style change.