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

Add export_presets.cfg #172

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

goatastronaut0212
Copy link

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
Copy link
Collaborator

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
Copy link
Author

goatastronaut0212 commented Aug 18, 2024

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.

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.

2 participants