#59749 move TrimEnd after path normalization

[dlosch]

Invocation of dotnet-getdocument.dll tool from Microsoft.Extensions.ApiDescription.Server.targets may result in trailing backslash and broken build on Windows

• You've read the Contributor Guide and Code of Conduct.
• You've included unit or integration tests for your change, where applicable.
• You've included inline docs for your change, where applicable.
• There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars)

Description

Value of OpenApiDocumentsDirectory must be trimmed of trailing backslash on Windows after path normalization.
OpenApiDocumentsDirectory = ../foo/ -> Path.GetFullPath (on Windows) -> <rooted Path>\foo\

Move TrimEnd('\') to after path normalization, as a Linux style directory separator would not be trimmed, translated to windows style backslash by the path normalization done by GetFullPath if build runs on Windows. Resulting value in _DotNetGetDocumentOutputPath would still have trailing backslash and break the command line (on Windows).

There seem to be some changes to this in past commits, moving the TrimEnd around. Looks like a regression.

Fixes #59749 (in this specific format)

[dotnet-policy-service]

Thanks for your PR, @dlosch. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.