ms-iot / imx-iotcore

Windows 10 IoT Core BSP for NXP i.MX Processors
MIT License
64 stars 42 forks source link

Issue #125 fix breaks case handling for non-mount builds #135

Closed TGoodhew closed 4 years ago

TGoodhew commented 4 years ago

The changes introduced in #125 break case handling for local cloned builds in WSL via /mnt/c.

The repo is to follow the clones from the firmware documentation and then change to the /mnt/c/<>/build/firmware/HummingBoardEdge_iMX6Q_2GB directory and run make.

dmcilvaney commented 4 years ago

Thanks, I'll take a look at it.

TGoodhew commented 4 years ago

Thanks.

TonyG

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10


From: Daniel McIlvaney notifications@github.com Sent: Monday, September 16, 2019 10:22:40 AM To: ms-iot/imx-iotcore imx-iotcore@noreply.github.com Cc: Tony Goodhew tony_goodhew@hotmail.com; Author author@noreply.github.com Subject: Re: [ms-iot/imx-iotcore] Issue #125 fix breaks case handling for non-mount builds (#135)

Thanks, I'll take a look at it.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fms-iot%2Fimx-iotcore%2Fissues%2F135%3Femail_source%3Dnotifications%26email_token%3DAB7GONAOQROAAXJXV66NUILQJ66GBA5CNFSM4IXE5WIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6Z4E7Q%23issuecomment-531874430&data=02%7C01%7C%7C51082eb7a68d48996fdf08d73aca7ad1%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637042513619795845&sdata=BroRNgZTY%2BHydWvWRis%2Bg2qMunRjskrj4hCUq4YG3Yo%3D&reserved=0, or mute the threadhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAB7GONG7N7X2QN6SUT4SYXLQJ66GBANCNFSM4IXE5WIA&data=02%7C01%7C%7C51082eb7a68d48996fdf08d73aca7ad1%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637042513619805856&sdata=j8k%2B4KIyQ0DaLelWNrXLq0WQRcFqcOBVG7ymh7oeeus%3D&reserved=0.

TGoodhew commented 4 years ago

Actually I can’t work around by setting the entire directory to case sensitive as that breaks the VS project ☹

TonyG

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10


From: Tony Goodhew tony_goodhew@hotmail.com Sent: Monday, September 16, 2019 10:24:39 AM To: ms-iot/imx-iotcore reply@reply.github.com; ms-iot/imx-iotcore imx-iotcore@noreply.github.com Cc: Author author@noreply.github.com Subject: RE: [ms-iot/imx-iotcore] Issue #125 fix breaks case handling for non-mount builds (#135)

Thanks.

TonyG

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10


From: Daniel McIlvaney notifications@github.com Sent: Monday, September 16, 2019 10:22:40 AM To: ms-iot/imx-iotcore imx-iotcore@noreply.github.com Cc: Tony Goodhew tony_goodhew@hotmail.com; Author author@noreply.github.com Subject: Re: [ms-iot/imx-iotcore] Issue #125 fix breaks case handling for non-mount builds (#135)

Thanks, I'll take a look at it.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fms-iot%2Fimx-iotcore%2Fissues%2F135%3Femail_source%3Dnotifications%26email_token%3DAB7GONAOQROAAXJXV66NUILQJ66GBA5CNFSM4IXE5WIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6Z4E7Q%23issuecomment-531874430&data=02%7C01%7C%7C51082eb7a68d48996fdf08d73aca7ad1%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637042513619795845&sdata=BroRNgZTY%2BHydWvWRis%2Bg2qMunRjskrj4hCUq4YG3Yo%3D&reserved=0, or mute the threadhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAB7GONG7N7X2QN6SUT4SYXLQJ66GBANCNFSM4IXE5WIA&data=02%7C01%7C%7C51082eb7a68d48996fdf08d73aca7ad1%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637042513619805856&sdata=j8k%2B4KIyQ0DaLelWNrXLq0WQRcFqcOBVG7ymh7oeeus%3D&reserved=0.

dmcilvaney commented 4 years ago

Set the /build/firmware/<board>/u-boot and /build/firmware/<board>/optee folders to case sensitive, those are the only two which matter.

TGoodhew commented 4 years ago

Thanks – That appears to have worked.

TonyG

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10


From: Daniel McIlvaney notifications@github.com Sent: Monday, September 16, 2019 10:27:34 AM To: ms-iot/imx-iotcore imx-iotcore@noreply.github.com Cc: Tony Goodhew tony_goodhew@hotmail.com; Author author@noreply.github.com Subject: Re: [ms-iot/imx-iotcore] Issue #125 fix breaks case handling for non-mount builds (#135)

Set the /build/firmware//u-boot and /build/firmware//optee folders to case sensitive, those are the only two which matter.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fms-iot%2Fimx-iotcore%2Fissues%2F135%3Femail_source%3Dnotifications%26email_token%3DAB7GONCQSM3WVYCQ7KRMDLTQJ66YNA5CNFSM4IXE5WIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6Z4TYY%23issuecomment-531876323&data=02%7C01%7C%7C69db91e704544ae9c0eb08d73acb2a24%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637042516561763333&sdata=FypxSvgompTWG6%2BpFJfgPb%2BifmHWwzIwkd%2F8owXzZx4%3D&reserved=0, or mute the threadhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAB7GONGKJPPQPPJ4EYAKMNTQJ66YNANCNFSM4IXE5WIA&data=02%7C01%7C%7C69db91e704544ae9c0eb08d73acb2a24%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637042516561773344&sdata=oPSmmChPNdxOsdurqCr3KYvJ4VPBPBvpkUaGEbHyTAI%3D&reserved=0.

dmcilvaney commented 4 years ago

Should be fixed in the makefile as well, it snuck past our CI since we run those in native Linux vms.

TGoodhew commented 4 years ago

Thanks again - Verified the make file fix on my machine