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

Backslash character in compile window escapes dollar signs #1587

Closed
christopher-rieder opened this issue Dec 28, 2016 · 0 comments
Closed

Backslash character in compile window escapes dollar signs #1587

christopher-rieder opened this issue Dec 28, 2016 · 0 comments
Assignees
Labels
Prio:2 Medium priority: Non crash bugs that impede the user, features that add new functionality. Type:Bug Errors and problems

Comments

@christopher-rieder
Copy link

christopher-rieder commented Dec 28, 2016

with this line
"D:\Juegos\quake dp\ID1\maps\${MAP_BASE_NAME}-compile.bsp"
i get this:

Copying 'D:\TrenchBroom\quakespasm\id1\maps\qump-topher-compile.bsp' to 'D:\Juegos\quake dp\ID1\maps\${MAP_BASE_NAME}-compile.bsp'

with this line other line
D:/Juegos/quake dp/ID1/maps/${MAP_BASE_NAME}-compile.bsp
i get this:

Copying 'D:\TrenchBroom\quakespasm\id1\maps\qump-topher-compile.bsp' to 'D:\Juegos\quake dp\ID1\maps\qump-topher-compile.bsp'

Windows use this slash for folders: '\'
linux use this slash for folders: '/'

changing '\' with '/' manually solves the problem, but windows by default use '\'
so, when someone in windows copy the full path of a folder to the clipboard, the path contains '\' slashes instead of '/' slashes.

@kduske kduske changed the title Slash character in compile window Backslash character in compile window escapes dollar signs Dec 28, 2016
@kduske kduske added Platform:Windows Prio:2 Medium priority: Non crash bugs that impede the user, features that add new functionality. Type:Bug Errors and problems labels Dec 28, 2016
@kduske kduske added this to the TrenchBroom 2.0.0 milestone Dec 28, 2016
@kduske kduske self-assigned this Dec 28, 2016
kduske added a commit that referenced this issue Dec 28, 2016
kduske added a commit that referenced this issue Dec 29, 2016
* #1587: Add failing test case.

* #1587: Don't use backslash as an escape character by default.
@kduske kduske closed this as completed Dec 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Prio:2 Medium priority: Non crash bugs that impede the user, features that add new functionality. Type:Bug Errors and problems
Projects
None yet
Development

No branches or pull requests

2 participants