GayPizzaSpecifications / mac-linux-kdk

Build ARM Linux Kernels natively on macOS hosts
GNU Lesser General Public License v3.0
60 stars 8 forks source link

script lacks case-sensitive filesystem check #4

Open hailmikhail opened 6 months ago

hailmikhail commented 6 months ago

Should the script enforce the requirement for a case-sensitive filesystem?

While kernel build workflow already offers appropriate guidance, I would have benefited from a script-enforced check. In a rush to build the kernel, I skipped over the following.

Enforcement example:

touch casE.tmp case.tmp
case_sensitivity="$(find . -name 'cas*.tm*' 2>/dev/null | wc -l)"
rm -f 'cas*.tmp' || true
if [ "$case_sensitivity" != 2 ]
then
  echo "Error. File system is not case sensitive.

Please use a case-sensitive file system."
  exit 1
fi
azenla commented 6 months ago

Yep, we can add this in the new implementation.