Skip to content

create new folder per dump#75

Open
schicho wants to merge 1 commit intoemukidid:masterfrom
schicho:createFolders
Open

create new folder per dump#75
schicho wants to merge 1 commit intoemukidid:masterfrom
schicho:createFolders

Conversation

@schicho
Copy link
Contributor

@schicho schicho commented Aug 21, 2020

Reduces the clutter in the root directory and also makes it easier to find the corresponding files.

@schicho

This comment has been minimized.

@emukidid
Copy link
Owner

I like this but can we make it an option "Create directory per dump [Yes/No]"? I'm happy for it to always come under cleanrip-dumps or whatever though, dumping to the root was never a good idea 😂

use tabs for indent.

use better(?) folder naming and fix directory creation after inserting a new storage device.

also check if game dump directory has been previously created, in case the user reuses a storage device when dumping in smaller chunks.
@schicho
Copy link
Contributor Author

schicho commented Oct 26, 2020

If I implement an option for it, it would require a lot of checks in the several dumping/writing functions (dump_bca, dump_game, dump_info) to control wether to write to /cleanrip-dumps/ or /cleanrip-dumps/gamename/ .

That will result in a lot of duplication and probably does not make the code very readable.

You decide. I can implement it, but I don't think it's great that way.

As the middle way, I can also just change it to use /cleanrip-dumps/ , which is still an improvment over dumping straight into the root directory.

@MasterLuma
Copy link

Can this still be implemented??

@vic1707
Copy link

vic1707 commented Jul 15, 2025

How are we doing on this feature?

@vic1707
Copy link

vic1707 commented Jan 27, 2026

Hi @emukidid I see a lot of updates recently, would you mind giving this feature some of your precious time?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants