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. _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
