Skip to content
This repository has been archived by the owner on Oct 28, 2023. It is now read-only.

.xml extension must be lower case #349

Open
steve6375 opened this issue May 18, 2022 · 1 comment
Open

.xml extension must be lower case #349

steve6375 opened this issue May 18, 2022 · 1 comment

Comments

@steve6375
Copy link

.XML files are not listed, .xml must be lower case.

@a1ive
Copy link
Owner

a1ive commented May 26, 2022

if [ -f "(${grubfm_device})${grubfm_dir}"*.xml ];

I used wildcards to match xml files, which are case-sensitive.

https://github.com/a1ive/grub/blob/5b20b2f521f32e8b170581854cc7c80f0c0bc159/grub-core/commands/wildcard.c#L180
According to https://man7.org/linux/man-pages/man3/regcomp.3.html,
if (regcomp (regexp, buffer, RE_SYNTAX_GNU_AWK)) should be if (regcomp (regexp, buffer, RE_SYNTAX_GNU_AWK | REG_ICASE))

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

No branches or pull requests

2 participants