util: Use ArgsManager::GetPathArg more widely #24675

In commit "util, refactor: Use GetPathArg to read "-conf" value" (88f48f39f84ae52f85819321dddf6e6bd9381be2)

I think it would be better to use fs::PathFromString not fs::NormalizedPathFromString here

• So this is a pure refactoring change that doesn't affect behavior
• To avoid changing user provided paths unnecessarily in a way that could make issues harder to debug
• To avoid interpreting paths unnecessarily, and let the OS handle it

Updated.

In commit "util, refactor: Add fs::NormalizedPathFromString function" (bc9ea0d3506610c0613bc27ee0c5c9d56e91092a)

I think it would probably be better not to add this function because:

• We already have other functions for converting strings to paths (PathFromString and u8path), and introducing a third function will probably add confusion. Especially when there is no explanation about when this function is supposed to be favored or avoided compared to the other ones.
• I don't think as a general rule that it is a good thing to normalize paths. I think debugging and understanding log messages if easier if paths are passed around verbatim and manipulated as little as possible, and the operating system is allowed to interpret them.
• If it's useful to have a normalization function that removes trailing slashes, I don't think there's a reason for it to do string decoding at the same. This function would be simpler and more accessible as:

static inline path NormalizedPath(std::path path)
{
    fs::path result = path.lexically_normal();
    // Remove trailing slash, if present.
    return result.has_filename() ? result : result.parent_path();
}

The commit has been dropped.