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

[FIX] fsInterface: Handle non-POSIX paths correctly #147

Merged
merged 2 commits into from
Jun 18, 2019

Conversation

RandomByte
Copy link
Member

@RandomByte RandomByte commented Jun 17, 2019

This resolves issues with the JSDoc build (especially the generateApiIndex task) on Windows.

fs functions can be called with non-POSIX paths on non-POSIX systems.
fsInterface should handle this correctly by converting any non-POSIX
paths to POSIX for further UI5 FS internal processing.

@coveralls
Copy link

coveralls commented Jun 17, 2019

Coverage Status

Coverage decreased (-0.05%) to 85.745% when pulling 7c08489 on fs-interface-non-posix into dc1eef1 on master.

@RandomByte RandomByte force-pushed the fs-interface-non-posix branch 3 times, most recently from 34cbb37 to 23d257d Compare June 17, 2019 14:24
@RandomByte RandomByte changed the title [WIP][INTERNAL] fsInterface: Handle non-posix paths correctly [INTERNAL] fsInterface: Handle non-posix paths correctly Jun 17, 2019
@RandomByte RandomByte changed the title [INTERNAL] fsInterface: Handle non-posix paths correctly [INTERNAL] fsInterface: Handle non-POSIX paths correctly Jun 17, 2019
@RandomByte RandomByte requested a review from matz3 June 17, 2019 14:26
@RandomByte RandomByte marked this pull request as ready for review June 17, 2019 14:26
fs functions can be called with non-POSIX paths on non-POSIX systems.
fsInterface should handle this correctly by converting any non-POSIX
paths to POSIX for further UI5 FS internal processing.
@RandomByte RandomByte changed the title [INTERNAL] fsInterface: Handle non-POSIX paths correctly [FIX] fsInterface: Handle non-POSIX paths correctly Jun 17, 2019
lib/fsInterface.js Outdated Show resolved Hide resolved
@matz3
Copy link
Member

matz3 commented Jun 18, 2019

@RandomByte squash required, as I've added a minor change

@RandomByte RandomByte merged commit 1ad0cc8 into master Jun 18, 2019
@RandomByte RandomByte deleted the fs-interface-non-posix branch June 18, 2019 08:23
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

Successfully merging this pull request may close these issues.

3 participants