oeed / CraftOS-Standards

Community standard file formats, communication systems, etc. for ComputerCraft and CraftOS 2.0
Other
20 stars 16 forks source link

Make Tempdir Standard use settings API for location #67

Open MagnificentPako opened 6 years ago

MagnificentPako commented 6 years ago

I suggest that the Tempdir standard moves from a hardcoded location to a path defined by a setting, which can be changed to the user's preferences.

SquidDev commented 6 years ago

I think this probably needs a little bit more rewording than just replacing every instance of /tmp with settings.get("tempdir", "/tmp").

I'm also curious of your use-case here? It seems to add a lot of complications without much tangible benefit.

oeed commented 6 years ago

Could you please elaborate on the benefits/use cases of this?

I can see this causing a large number of difficulties and complications. With this you would basically have to include a settings API with every program that wants to save temporary files. We don't have anything else in this API you mention either, so I doubt it would be used that much. The way I see it the more simple this is the more likely it is going to be used.

So unfortunately it's probably a no from me, but if you do have solutions to these issues by all means point them out.