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 TOML support #39

Open
deestt opened this issue Jan 6, 2023 · 4 comments
Open

Add TOML support #39

deestt opened this issue Jan 6, 2023 · 4 comments
Labels
enhancement New feature or request wontfix This will not be worked on

Comments

@deestt
Copy link

deestt commented Jan 6, 2023

No description provided.

@dasavick dasavick added enhancement New feature or request wontfix This will not be worked on labels Jan 7, 2023
@dasavick
Copy link
Member

dasavick commented Jan 7, 2023

I already looked into this format before. I'm afraid there may be no suitable and supported TOML library for java. Okaeri-configs needs both read and write for any abstract structure with support of at least text values, collection, and map.

Ability to insert comments programmatically would also be greatly appreciated. The libraries I have seen before support mainly read and require config skeleton to be already present, thus preventing okaeri-configs from generating a config file with the default values from the POJO.

@dasavick dasavick changed the title add support toml Add TOML support Jan 7, 2023
@deestt
Copy link
Author

deestt commented Jan 7, 2023

ja widzialem np night config

@dasavick
Copy link
Member

dasavick commented Jan 7, 2023

Edit1: Mózg zepsuty i pomyliłem AGPL z LGPL, w trakcie analizy.

Edit2: Jednak nic nie rozumiem. Proszę dać libkę na MIT albo Apache License 2.0. 💀

Edit3: Chyba jednak akceptowalne i nie tworzy za dużo obowiązków dla osób, które wyłącznie na "użytek własny" by stosowały (we wtyczce na własnym serwerze etc.), a jedynie dla dystrybucji aplikacji lub wtyczek (plików jar) używających tego modułu. Może kiedyś zobaczę, czy da się to realnie zaimplementować.


Wygląda na to, że night-config używa własnej implementacji TOML. Kod jest jednak na licencji LGPL. Jest to dość restrykcyjna licencja, która moim zdaniem słabo nadaje się do tego typu bibliotek.

Zgodnie z moim stanem wiedzy, użycie okaeri-configs z potencjalnym modułem TOML opartym o tę bibliotekę, tworzyłoby obowiązek poinformowania o użyciu night-config oraz udostępnienia każdemu użytkownikowi dostępu do plików źródłowych obecnie używanej wersji night-config w twojej aplikacji, czy też wtyczce.

@deestt
Copy link
Author

deestt commented Jan 16, 2023

night config w GNU jest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants