QUERY · ISSUE

windows: os.sep should be backslash

openby jimmoopened 2022-09-12updated 2022-09-15

enhancement

This came up in https://github.com/micropython/micropython-lib/pull/527, and we have a workaround there, but getting this right will be required to make os.path work correctly.

The windows port current returns / from os.sep but should be \. We should fix that, as well as implementing os.altsep for unix/windows (return None on unix, / on windows).

CANDIDATE · PULL REQUEST

tools/mpremote: Fix "fs cp -r" on Windows.

closedby andrewleechopened 2022-02-01updated 2022-02-04

tools

Currently when using mpremote fs cp -r _manifest : on windows, it does not correctly handle the \ path separator, resulting in things like

cp _manifest\config_def.mpy :_manifest/_manifest\art.mpy

which gets even worse on-device:

os.listdir("/_manifest")
['_manifest\x07rt.mpy', ...]

Changing to a simply '/',join() appears to fix this problem cleanly.

Assessment

Q Qwen · medium

Both items address Windows path separator issues in MicroPython, but they target different components and problems. Issue #9286 is about fixing the core os.sep and os.altsep implementation in the Windows port to return correct separators. PR #8237 fixes a specific bug in the mpremote tool where os.path.join was incorrectly used, causing backslash interpretation issues. While related to the same general topic (Windows path handling), the PR does not resolve the core issue of os.sep returning the wrong value or implementing os.altsep. The PR is a workaround for a specific tool, not a fix for the underlying port issue.

Suggested action

Keep both items open. The PR addresses a specific tool bug, while the issue requests a fundamental fix to the Windows port's path separator implementation.

windows: os.sep should be backslash

tools/mpremote: Fix "fs cp -r" on Windows.

Keyboard