-
Notifications
You must be signed in to change notification settings - Fork 3
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
does not compile because of an error on disk/mraid1x_linux.c #1
Comments
Thanks for the feedback. In the readme, it mentions that this was originally obtained from grub-2.04.tar.gz at https://git.savannah.gnu.org/cgit/grub.git I redownloaded the file from there just now and it is bitwise identical to the one I originally uploaded here 2.5 years ago. As for compiling the code, did you use the instructions in the readme? I have only verified this working on x64 Ubuntu 14.04, but others have been forking and building this code in recent years. I'm not really actively involved in this anymore. |
P.S. I assume before you are referring to "grub-2.04\grub-core\disk\mdraid1x_linux.c", as there doesn't seem to be a file called mraid1x_linux.c here. |
Comparing Grub-2.04 vs. Grub-2.06, there is a & present at 181:15 in 2.04 that is not present in 2.06 (actually there are a several code changes in that line). Nevertheless this is a RAID disk related item, the Surface doesn't have that type of hardware, so we probably don't need to build mdraid1x. Can you try building again using the instructions in the readme but leaving out mdraid1x from the final command? |
Hi thanks for taking the time to come back to this. |
If you have a suggestion on how I can compile leaving this module out I'd be grateful 😅 |
When trying to compile on raspbian (Linux 5.17.0) I get the following error
disk/mraid1x_linux.c: 181:15 Werror=array-bounds
Im not a C expert but I compared disk/mraid1x_linux.c to the original codebase version and it seems there is a typo at 181:15
The text was updated successfully, but these errors were encountered: