Minor dependency formatting glitch

[SoapGentoo]

On Meson 0.47.2 I get

Dependency GMock found: YES (prebuilt)

On HEAD Meson I get

Dependency GMock found: YES , prebuilt

due to the system GMock missing a version specifier.

[jon-turney]

This is probably the result of changes in #3657.

What do you think the correct output should be?

[SoapGentoo]

For simplicity, I would keep the formatting the old way, if only to remove the context-sensitive nature of the comma. I would suggest

Dependency GMock found: YES (prebuilt)

and

Dependency GMock found: YES 1.9.0 (pkgconfig)

as all log components are space-separated and don't require extra context-aware information for handling the comma. Alternatively

Dependency GMock found: YES (1.9.0, pkgconfig)

which only requires appending the comma to the version string, if present. I don't have any hard opinions on this, even keeping it how it is modulo the trailing comma after the YES is just as fine, just from a programmatic point-of-view this last option is the hardest to implement.

Your ideas?

[jon-turney]

Your ideas?

I was just trying to keep things the same, unfortunately the logging wasn't consistent.

after this PR, I get

Dependency GTest found: NO (tried pkgconfig and system)

and

Dependency GTest found: YES (building self)