[pve-devel] [PATCH v1 pve-common 10/18] pbsclient: use `File::Spec->catfile` to concatenate file paths

[Max Carrara]

On Mon Nov 11, 2024 at 8:16 PM CET, Thomas Lamprecht wrote:
> a commit message explaining why/benefits would be good – especially as IIRC
> we currently do not use this module at all?

Sure, I'll add one!

I've opted to using `File::Spec` here because I've been using it a lot
for other things (that haven't landed on the mailing list yet). The
reasoning is mostly something like "oh there's a core module that
handles concatenating paths and file names, I will use that instead of
doing it by hand".

... granted, this is maybe a little pedantic here; I mainly just thought
it wouldn't hurt. But yes, I could've specified that in the commit
message.

I can add a message or just drop the commit in the next revision, if
you'd like.