dsv-0.7.0 breaks tests when using guix build about guile-dsv

[artyom-poptsov]

Hello, Fredrik!

If I understand it correctly guile-smc (which is a new dependency in 0.7.0) is trying to write to the syslog and failing. Which is expected as it is not setup when running via guix build. I checked the source code and dsv should only setup the logger if key argument #:log-driver for dsv->scm is not #f (making this assumption from this line in dsv.scm). And it is #f by default.

Not quite as if logger is set to #f then the default logger is used which is syslog. Please try to run dsv->scm as follows:

(dsv->scm port #:format 'rfc4180 #:log-driver "null")

And let me know what will happen.

Is this a bug or am I missing something? Expected behavior is to not write anything to syslog or any logger for that matter.

The aforementioned nuance that "syslog" is used by default is mentioned also in the Guile-DSV documentation.

But maybe it is a bug in the sense that Guile-DSV breaks user's workflow. Do you think that the logging should be disabled by default for dsv->scm?

Thanks,
avp

[plattfot]

Hi,

Not quite as if logger is set to #f then the default logger is used which is syslog. Please try to run dsv->scm as follows:

(dsv->scm port #:format 'rfc4180 #:log-driver "null")

And let me know what will happen.

That solved it! Thanks.

The aforementioned nuance that "syslog" is used by default is mentioned also in the Guile-DSV documentation.

Ah, my bad I completely forgot to check the info manual. 🤦

But maybe it is a bug in the sense that Guile-DSV breaks user's workflow. Do you think that the logging should be disabled by default for dsv->scm?

It is a breaking change between 0.6.0 and 0.7.0 only if you are using guix to build (or anything else that isolates the environment). Apart from that it will not break user's workflow. Maybe just add a section in the README to highlight the behavior and then reference to the manual if users want to change the default behavior.

Thank you for an excellent library!

[artyom-poptsov]

I've added a note (see 636aaec) to the README.org about the logging configuration.