All right, thanks. -----Original Message----- From: Joshua Watt <[email protected]> Sent: Friday, October 28, 2022 5:45 PM To: Hamza, Muhammad <[email protected]> Cc: [email protected] Subject: Re: [meta-mingw][kirkstone][PATCH] toolchain-scripts-mingw32.bbclass: Remove trailing slash in SDKROOT
Ya, I queued it in master-next. Let me run it through the AB then we can merge it On Fri, Oct 28, 2022 at 7:00 AM Hamza, Muhammad <[email protected]> wrote: > > Hi, > Any update on this? > > -----Original Message----- > From: Hamza, Muhammad <[email protected]> > Sent: Monday, October 24, 2022 11:03 AM > To: [email protected] > Cc: [email protected]; Hamza, Muhammad <[email protected]> > Subject: [meta-mingw][kirkstone][PATCH] > toolchain-scripts-mingw32.bbclass: Remove trailing slash in SDKROOT > > Modify toolchain-scripts-mingw32.bbclass to add a check in environment-setup > script which removes trailing slash in path of SDKROOT. > This is needed to avoid multiple adjacent slashes in paths which are produced > by appending to SDKROOT. > In reference to > https://learn.microsoft.com/en-us/windows/win32/fileio/naming-a-file?r > edirectedfrom=MSDN naming convention used for paths and disk drives in > windows should use a single backslash. Even though in some cases windows > ignores double slashes in paths and it might work but it isn't documented as > a right naming convention and does fail in some cases eg. dir command cannot > interpret double slashes and fails. > > For example if my SDK is located in D: drive, the environment setup scripts > sets "SDKROOT=D:\" and hence SDKTARGETSYSROOT gets set as > "SDKTARGETSYSROOT=D:\\sysroots\armv8a-oe-linux" > The introduced check removes additional slash in SDKROOT to set it as > "SDKROOT=D:" so all other variables using SDKROOT get set without additional > slash. > > Signed-off-by: Muhammad Hamza <[email protected]> > --- > classes/toolchain-scripts-mingw32.bbclass | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/classes/toolchain-scripts-mingw32.bbclass > b/classes/toolchain-scripts-mingw32.bbclass > index d96cb40..8cb426a 100644 > --- a/classes/toolchain-scripts-mingw32.bbclass > +++ b/classes/toolchain-scripts-mingw32.bbclass > @@ -12,6 +12,7 @@ toolchain_create_sdk_env_script:sdkmingw32 () { > touch $script > # Be sure to use the 'short' path, so we can have deeper directories. > echo 'set SDKROOT=%~sdp0%' >> $script > + echo 'IF %SDKROOT:~-1%==\ set SDKROOT=%SDKROOT:~0,-1%' >> > + $script > > # Convert to mingw32 subpaths > sysroot='%SDKROOT%'${sysroot##${SDKPATH}} > -- > 2.25.1 >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#58432): https://lists.yoctoproject.org/g/yocto/message/58432 Mute This Topic: https://lists.yoctoproject.org/mt/94528644/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
