Skip to content

Resolve relative paths to absolute before Windows path conversion - #968

Open
miles-read wants to merge 3 commits into
winapps-org:mainfrom
miles-read:fix-relative-file-path
Open

miles-read wants to merge 3 commits into
winapps-org:mainfrom
miles-read:fix-relative-file-path

Conversation

@miles-read

Copy link
Copy Markdown

Small change to fix opening files using relative paths.

The existing sed substitution only matches paths already prefixed with $HOME or $REMOVABLE_MEDIA, so a relative path falls through unconverted.

[ "${2#/}" = "${2}" ] && set -- "${1}" "$(pwd)/${2}"

@oskardotglobal

Copy link
Copy Markdown
Member

Hi, please sign off on your commits (see the DCO check on how to do that). Also, please indent the if statement and the comment by another 4 spaces. Then, this will be ready to merge.

pre-commit-ci Bot and others added 2 commits September 16, 2026 10:53
for more information, see https://pre-commit.ci

Signed-off-by: miles-read <readmiles00@gmail.com>
Signed-off-by: miles-read <readmiles00@gmail.com>
@miles-read
miles-read force-pushed the fix-relative-file-path branch from 963c919 to e8fd503 Compare September 16, 2026 09:54
@oskardotglobal

Copy link
Copy Markdown
Member

Your first commit still isn't signed off.

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.

2 participants