Skip to content

dhcpv4: server4: Add a simple logging implementation - #375

Open
purpleidea wants to merge 1 commit into
insomniacslk:masterfrom
purpleidea:feat/embedded-logging-take2
Open

dhcpv4: server4: Add a simple logging implementation#375
purpleidea wants to merge 1 commit into
insomniacslk:masterfrom
purpleidea:feat/embedded-logging-take2

Conversation

@purpleidea

Copy link
Copy Markdown

This adds a simple implementation of the logging struct so that you
don't need to implement the complex interface if you don't need it.

This adds a simple implementation of the logging struct so that you
don't need to implement the complex interface if you don't need it.
@purpleidea

Copy link
Copy Markdown
Author

I think that the solution in https://github.com/insomniacslk/dhcp/pull/371/files was over-engineered, but in any case, I expect most people like myself just want simple logging. This patch adds a simple implementation so that you can easily "get logging" by passing in your own printf function straight away.

This is meant as an alternative to #373 which wasn't accepted.

Cheers!

@pmazzini

Copy link
Copy Markdown
Collaborator

Can you sign the commit?

@purpleidea

Copy link
Copy Markdown
Author

@pmazzini You want to merge this now?

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