Skip to content

Hotfix for #66273#66287

Merged
bombasticSlacks merged 2 commits intoCleverRaven:masterfrom
alef:fix-66273
Jun 18, 2023
Merged

Hotfix for #66273#66287
bombasticSlacks merged 2 commits intoCleverRaven:masterfrom
alef:fix-66273

Conversation

@alef
Copy link
Copy Markdown
Contributor

@alef alef commented Jun 18, 2023

Summary

Bugfixes "All directories starting with dot"

Purpose of change

Fixes #66273

Describe the solution

as_norm_dir returns ./ if the directory normalizes to ..

Describe alternatives you've considered

Rewrite all code using paths as strings to use fs::path and its append operator.

Testing

Run the executable, check for .dir/s, Play Now!, save and exit, restart, load again.

Additional context

Waiting for Actions work out linting issues

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) labels Jun 18, 2023
Comment thread src/filesystem.cpp Outdated
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Jun 18, 2023
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Jun 18, 2023
@NetSysFire NetSysFire added the (P2 - High) High priority (for ex. important bugfixes) label Jun 18, 2023
@bombasticSlacks bombasticSlacks merged commit c3e2de8 into CleverRaven:master Jun 18, 2023
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Jun 18, 2023
@alef alef deleted the fix-66273 branch June 22, 2023 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions (P2 - High) High priority (for ex. important bugfixes)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Project has become unworkable after path changes

3 participants