Skip to content

Conversation

@bdovaz
Copy link

@bdovaz bdovaz commented Dec 1, 2025

In this commit 5a23948#diff-f8f453a6bb79d1bba0a5f48cfd1d9a73373d4e68008591548d15bb0ef30abaa4R9 I see that the path has been changed, but it has been changed to a Windows path and not a normalized one that any OS can interpret.

This is starting to cause problems when you try to read the package metadata (license in this case) through dotnet tooling or NuGet packages with their API when doing so from a Linux machine.

@pavel-mikula-sonarsource
Copy link
Contributor

Hi,
can you clarify the problem a bit?

I understand that Unix systems are generally not capable of understanding paths written by Windows, as it only work the other way around. We should be able to update that.

What is not clear to me is where the change happened. When I look at the nuspec file from the commit, I see all old paths (like icon in windows format, same as in the new nuspec.

At the same time, while the documentation seems to refer it to it as "path" everywhere, without explicit separator rules, all the examples are using windows \ path separator in the .nuspec reference.

Can you share what are the commands or tools that you're using to reproduce the issue?

Copy link
Contributor

@pavel-mikula-sonarsource pavel-mikula-sonarsource left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a consistency check for now

@bdovaz
Copy link
Author

bdovaz commented Dec 5, 2025

@pavel-mikula-sonarsource Ready to review.

And regarding your questions, everything comes from here: sensslen/nuget-license#290

But it can potentially be affected by other tools, so it's better to use / instead of \.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants