qrrk / Catapult

A cross-platform launcher for Cataclysm: DDA and BN
MIT License
389 stars 61 forks source link

Add export_presets.cfg #172

Closed goatastronaut0212 closed 1 week ago

goatastronaut0212 commented 3 months ago

I really like your project and I would love to bring it to Nixpkgs but I notice some minor issue. It doesn't have export_presets.cfg like other projects for example https://github.com/Orama-Interactive/Pixelorama. And it make me a little bit hard to package it from source code.

I would love to know your thoughts on this. If your repo have this, it will be less work for me to do when packaging this repo in the future.

AriaMoradi commented 3 months ago

Take a look at #149

Creating releases changes the file each time and having the changes committed is painful and one setup doesn't work for all the developers so, we decided to not commit it at all.

goatastronaut0212 commented 3 months ago

The one export_presets.cfg maybe annoy some people but what about the second option? I would love to have something like export_presets.cfg.linux lie somewhere in the repo though due to third option isn't that automation and packaging a binary file is not a good thing to do with non-FHS system like Guix or NixOS.

AriaMoradi commented 1 week ago

I guess a linux and win cfg file could serve as example and documentation, at the same time while we don't ship a default file, it won't break anyone's setup.