Skip to content

make it work on WINNT with '/reference' on WSL #46

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

refack
Copy link

@refack refack commented Apr 25, 2025

Three parts:

  1. demangle filenames re.sub(r'[\uf000-\uf0ff]', lambda m: chr(ord(m.group(0)) & 0xff), filename).
    Apperently WSL1's drvfs will turn incompatible ASCII characters in filenames (* : ? |) into unicode \u0xf0XX.
    For maximal tarball name preservation I exploded it to a WSL1 path, but read it from native windows, so we need to demangle before processing, but keep mangled name for renaming, so we map both to the new name
  2. replace uses of os.path with import posixpath as urlpath for most URL path manipulations
  3. Use pathlib.Path when constructing URLs to the live website.

If there's interest (or if it makes my workflow simpler) I'll figure out how to handle a tarball exploded in native WIN32.

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.

1 participant