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

Completely stop GGEZ from creating an application config in ~/.config #356

Closed
ghost opened this issue Apr 19, 2018 · 3 comments

Comments

@ghost
Copy link

commented Apr 19, 2018

I would love to use GGEZ over piston as using one extern crate over the amount required in piston would be great, however, I have one problem.

My problem with GGEZ is that I can't find any easy way to stop it from creating an application config folder in ~/.config. For most people this is not a problem, but I find my self trying to keep all of my files clean, but this is just a waste of space in my case.

I believe it is a great feature to implement by default for large projects, however, I would like to know if there's a way I can stop this completely without external scripts.

I'm using Linux by the way, in-case it doesn't happen on other systems.

Sorry for the long issue for such a small... issue; but thanks in advance.

@icefoxen

This comment has been minimized.

Copy link
Contributor

commented Apr 19, 2018

This is pretty reasonable, and shouldn't at all be difficult to do. The directory creation happens here on the app_root. There's functions to check for the directories without creating them so we just need to use those instead and create the directories lazily when someone writes to them.... that second part is probably the tricky bit.

icefoxen added a commit that referenced this issue Apr 23, 2018

Made the PhysicalFS VFS lazily create its root directories.
Basically, instead of ggez creating all its directories when the
Filesystem is created, it creates them when something tries to access
them (if they don't already exist).

This does make me a little frumple since it involves an extra system
call per file operation, but oh well.

Resolves #356.
@icefoxen

This comment has been minimized.

Copy link
Contributor

commented Apr 23, 2018

Resolved in e0dba15 .

@icefoxen icefoxen closed this Apr 23, 2018

@ghost

This comment has been minimized.

Copy link
Author

commented Apr 24, 2018

Thank you so much! Sorry for nit replying earlier, I hadn't received any notifications and I'm not very well organised with e-mail.

Thank you again, @icefoxen !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.