Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

There is no path separator between the output folder and the xml file when compiling for Android. #9645

Open
JanKrivanek opened this issue Jan 2, 2025 · 1 comment
Assignees
Labels
Area: App+Library Build Issues when building Library projects or Application projects.

Comments

@JanKrivanek
Copy link
Member

Android framework version

net9.0-android

Affected platform version

Main

Description

Path separation not properly ensured for DocumentationFile in

<DocumentationFile Condition=" '$(DocumentationFile)' == '' and '$(_ComputeFilesToPublishForRuntimeIdentifiers)' != 'true' ">$(OutputPath)$(AssemblyName).xml</DocumentationFile>

The property construction should call [MSBuild]::EnsureTrailingSlash to properly ensure the separation

FYI @NonExistPlayer

Steps to Reproduce

Details in the original issue dotnet/msbuild#11057

Did you find any workaround?

No response

Relevant log output

@jonathanpeppers
Copy link
Member

Is this only an issue when setting $(OutputPath) as a global property, or using -o? Can you share an example of the command or a .binlog?

Looking here:

They mention the example, $(OutputPath)app.publish\ in dotnet/msbuild, which makes me think this problem exists for several usages of $(OutputPath) in different places.

@jonathanpeppers jonathanpeppers removed the needs-triage Issues that need to be assigned. label Jan 6, 2025
@jonathanpeppers jonathanpeppers added this to the Under Consideration milestone Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: App+Library Build Issues when building Library projects or Application projects.
Projects
None yet
Development

No branches or pull requests

3 participants