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.

Sign up for GitHub

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

Jump to bottom

Substitution for os.sep #1700

Closed
jayvdb opened this issue Oct 20, 2020 · 0 comments · Fixed by #1701
Closed

Substitution for os.sep #1700

jayvdb opened this issue Oct 20, 2020 · 0 comments · Fixed by #1701
Labels
feature:new something does not exist yet, but should

Comments

@jayvdb
Copy link

jayvdb commented Oct 20, 2020 •
edited
Loading

There is {:} for os.pathsep , but none for os.sep (/ vs \ on Windows). I propose {/} for this.

This is probably because most of the time / works ok on Windows, but that is not universal.

However the lack of os.sep support also impacts the ability of config tests to do meaningful tests of paths that are cross platform without horrible contortions for Windows.
@jayvdb jayvdb added the feature:new something does not exist yet, but should label Oct 20, 2020
jayvdb added a commit to jayvdb/tox that referenced this issue Oct 20, 2020
@jayvdb
Add substitution {/} to os.sep
c5fb611 
Closes tox-dev#1700
@jayvdb jayvdb mentioned this issue Oct 20, 2020
Merged
6 tasks
jayvdb added a commit to jayvdb/tox that referenced this issue Oct 20, 2020
@jayvdb
Add substitution {/} to os.sep
60cda3c 
Closes tox-dev#1700
gaborbernat pushed a commit that referenced this issue Oct 21, 2020
@jayvdb
Add substitution {/} to os.sep (#1701)
7c103c8 
Closes #1700
gaborbernat pushed a commit that referenced this issue Oct 23, 2020
@jayvdb @gaborbernat
Add substitution {/} to os.sep
afefabb 
Closes #1700
gaborbernat pushed a commit that referenced this issue Oct 23, 2020
@jayvdb @gaborbernat
Fix escaping brackets in setenv
58336d1 
Closes #1700

Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
@tox-dev tox-dev locked and limited conversation to collaborators Jan 14, 2021
ssbarnea pushed a commit to ssbarnea/tox that referenced this issue Apr 19, 2021
@jayvdb @ssbarnea
Add substitution {/} to os.sep (tox-dev#1701)
bfea74b 
Closes tox-dev#1700
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
feature:new something does not exist yet, but should
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add substitution {/} to os.sep jayvdb/tox
1 participant
@jayvdb