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

why was slf4j-api added as a dependency? #1094

Closed
zoff99 opened this issue Apr 14, 2024 · 2 comments
Closed

why was slf4j-api added as a dependency? #1094

zoff99 opened this issue Apr 14, 2024 · 2 comments
Labels
question Further information is requested

Comments

@zoff99
Copy link

zoff99 commented Apr 14, 2024

i read: #997 (comment)
and: 32082c0
and: 89dbda1

but i can not see any reason this was done.

here it says "Read the Changelog", but there is nothing about slf4j-api in the changelog.
why was this added? can you link to documentation about the decision or some code in sqlite that requires it?

why was it not possible to make this optional?

@zoff99 zoff99 added the triage label Apr 14, 2024
@gotson
Copy link
Collaborator

gotson commented Apr 15, 2024

it was added because it's used in almost any library, and it lets the application control logging. You can check #802

@gotson gotson added question Further information is requested and removed triage labels Apr 15, 2024
@jinsihou19
Copy link

yes,it is trouble for me to add external dependencies. look it...

@gotson gotson closed this as completed May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants