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

CLI option --edit-level takes path relative to data directory, inconsistent with normal play invocation #1819

Open
Semphriss opened this issue Sep 15, 2021 · 1 comment · May be fixed by #1835
Open

CLI option --edit-level takes path relative to data directory, inconsistent with normal play invocation #1819

Semphriss opened this issue Sep 15, 2021 · 1 comment · May be fixed by #1835

Comments

@Semphriss
Copy link
Member

Playing a SuperTux level located in the current working directory is as easy as supertux2 ./level.stl. However, attempting to edit the same file does not work by simply appending --edit-level, since this option has a different file lookup mechanism.

Both situations should use the same lookup mechanism, that is, find the file using absolute path or relative path from current working directory.
tobbi added a commit that referenced this issue Oct 5, 2021
@tobbi
Check for file in filesystem for editor
7f3234c 
Addresses #1819
tobbi added a commit that referenced this issue Oct 5, 2021
@tobbi
Check for file in filesystem for editor
77f249b 
Addresses #1819
@tobbi
Copy link
Member

tobbi commented Oct 5, 2021

Check the commit above for a first draft. Saving doesn't yet work.

@tobbi tobbi linked a pull request Oct 5, 2021 that will close this issue
Open
@Semphriss Semphriss mentioned this issue Jan 14, 2022
Open
tobbi added a commit that referenced this issue Mar 7, 2022
@tobbi
Check for file in filesystem for editor
c35912b 
Addresses #1819
tobbi added a commit that referenced this issue Aug 30, 2023
@tobbi
Check for file in filesystem for editor
81ade30 
Addresses #1819
tobbi added a commit that referenced this issue Mar 29, 2024
@tobbi
Check for file in filesystem for editor
6e9cff5 
Addresses #1819
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Editor: --edit-level now also takes an absolute path SuperTux/supertux
2 participants
@tobbi @Semphriss