luxtorpeda icon indicating copy to clipboard operation
luxtorpeda copied to clipboard

Remove obsolete sourceport files after an update

Open Shockwave508 opened this issue 1 year ago • 7 comments

Feature Request

Multiple versions of a sourceport could easily stack whenever a new update is installed, within the .cache folder, gradually draining onboard disk space over time.

Solution

If a new version of a sourceport is downloaded through Luxtorpeda, delete the older, obsolete version after successful install.

Additional Context

No response

Shockwave508 avatar May 23 '24 04:05 Shockwave508

Currently the client doesn't have a concept of versions, in that it does not see the older content in there.

I could do some other things like either provide a mode to delete the cache files as soon as they are downloaded and used, which would save on disk but add more network traffic. Or have some logic to remove inside the folders files that don't match.

What are your thoughts @Shockwave508 ?

d10sfan avatar Jul 27 '24 17:07 d10sfan

Currently the client doesn't have a concept of versions, in that it does not see the older content in there.

I could do some other things like either provide a mode to delete the cache files as soon as they are downloaded and used, which would save on disk but add more network traffic. Or have some logic to remove inside the folders files that don't match.

What are your thoughts @Shockwave508 ?

I think the latter is a good stopgap measure.

SkylarLK avatar Sep 06 '24 02:09 SkylarLK

Latter is good. Will prevent obsolete copies piling up.

Shockwave508 avatar Sep 07 '24 15:09 Shockwave508

This has been open for nearly a year.

SladeWilson52 avatar Aug 05 '25 13:08 SladeWilson52

Yes that's correct, did you have something constructive to add here?

I'm still interested in doing this at some point but it's fairly complex since the current logic does not know about files that aren't in the json file and I've been pretty busy.

So I'll plan on getting to this when I can.

d10sfan avatar Aug 05 '25 14:08 d10sfan

Also if someone is interested in looking into this in the meantime pull requests are always welcome

d10sfan avatar Aug 05 '25 14:08 d10sfan

Yes that's correct, did you have something constructive to add here?

My apologies, I got the impression that nothing was happening. I assume a rewrite of the logic will take a lot of time.

SladeWilson52 avatar Aug 05 '25 15:08 SladeWilson52