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

False positive warning on Windows due to upper/lower case drive letter: WARN Using manifest ... PIXI_PROJECT_MANIFEST rather than local ... #2288

Open
2 tasks done
Huite opened this issue Oct 15, 2024 · 2 comments
Labels
🐞 bug Something isn't working windows Related to MS Windows

Comments

@Huite
Copy link

Huite commented Oct 15, 2024

Checks

  • I have checked that this issue has not already been reported.

  • I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example

mkdir c:/tmp/case
cd c:/tmp/case
pixi init
pixi shell
pixi task list

Note the lower case on c:/.

Issue description

The example generates the following output:

C:\tmp via 🐍
❯ mkdir c:/tmp/case

C:\tmp via 🐍
❯ cd c:/tmp/case

c:\tmp\case
❯ pixi init
Created C:\tmp\case\pixi.toml

c:\tmp\case
❯ pixi shell
(case)
c:\tmp\case via 🅒 case
❯ pixi task list
 WARN Using manifest C:\tmp\case\pixi.toml from `PIXI_PROJECT_MANIFEST` rather than local c:\tmp\case\pixi.toml
No tasks found

I think it's due to pixi shell. This one doesn't trigger the warning:

cd c:/tmp/case
pixi init
pixi task list
cd C:/tmp/case
pixi task list
cd c:/tmp/case
pixi task list

Expected behavior

Ideally, on Windows, pixi doesn't care about upper or lower case for the drive letter.

@Hofer-Julian
Copy link
Contributor

Hey Huite 👋🏻

I am curious. How do you get a lower case drive letter in the first place?

But yes, this sounds like something we should fix.

@ruben-arts ruben-arts added windows Related to MS Windows 🐞 bug Something isn't working labels Oct 16, 2024
@Huite
Copy link
Author

Huite commented Oct 16, 2024

Good morning Julian!

You get a lower case drive letter by explicitly cding as such: cd c:/tmp/case. As far as I can tell, this is just Windows being weird and inconsistent: if I do cd c:/TMP it normalizes the tmp part; but it never changes the drive letter (probably this has a bunch of reasons...).

You can see in my printed output that I initially start with an upper case letter, but after the change directory, it's showing a lower case:

C:\tmp via 🐍
❯ cd c:/tmp/case

c:\tmp\case

Windows preserves the case of the drive letter as you type it when changing directories.

Funnily enough, this isn't the first time I've had issues with this Windows "feature". A while ago, I had a different software application that failed completely because it could not find the directory -- I can't remember which, I should've made a note!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working windows Related to MS Windows
Projects
None yet
Development

No branches or pull requests

3 participants