Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

script lacks case-sensitive filesystem check #4

Open
hailmikhail opened this issue Mar 27, 2024 · 1 comment
Open

script lacks case-sensitive filesystem check #4

hailmikhail opened this issue Mar 27, 2024 · 1 comment

Comments

@hailmikhail
Copy link

hailmikhail commented Mar 27, 2024

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.

  • In README.md, the Usage section already directs the user to a case-sensitive filesystem.
  • git clone of a linux kernel source repo. to a filesystem that is not case-sensitive will return an error or warning.

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
Copy link
Member

azenla commented Apr 4, 2024

Yep, we can add this in the new implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants