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

(inkscape) package name error due to "Dotted and dotless I letter" #1805

Closed
hkdemiralp opened this issue Feb 21, 2022 · 4 comments
Closed

(inkscape) package name error due to "Dotted and dotless I letter" #1805

hkdemiralp opened this issue Feb 21, 2022 · 4 comments

Comments

@hkdemiralp
Copy link

hkdemiralp commented Feb 21, 2022

Chocolatey Version

0.12.1

Chocolatey License

No response

Package Version

1.1.2

Current Behaviour

Doing a choco upgrade all -y gives this failure message for inkscape in Windows OS using Turkic languages:
`Failures

Expected Behaviour

"choco upgrade all -y" command should not give "ınkscape" error. The latest inkscape is already installed with chocolatey.

Steps To Reproduce

  1. choco install inkscape -y
  2. choco upgrade all -y
  3. `Failures
  1. Notice: in the chocolatey error message "ınkscape" word is also written with a dotless "I", not "i")

Environment

- Operating System: Microsoft Windows NT 10.0.22000.0
- PowerShell Version: 5.1.22000.282
- Shell:

Chocolatey Log

https://gist.github.com/hkdemiralp/8943e8bb26d4cd3418de92795f60adf1

Anything else?

I believe the problem is the package naming which exhibits the "Dotted and dotless I" problem. In Turkic languages, Dotted "İ i" and dotless "I ı" are distinct letters. If the name of the package would have been written in lovercase letters like "inkscape", instead of "InkScape", it would work just fine. Installing inkscape works but upgrading all packages gives the error message.

@hkdemiralp hkdemiralp added the Bug label Feb 21, 2022
@AdmiringWorm
Copy link
Member

This sounds more like a problem in the choco codebase at the moment than a problem with the package itself.

If the name of the package would have been written in lovercase letters like "inkscape", instead of "InkScape", it would work just fine. Installing inkscape works but upgrading all packages gives the error message.

While that would possibly fix this issue, it, unfortunately, wouldn't be right to do that right now either.
The identifier InkScape is used on purpose now due to historical reasons.
The first time the package was introduced to CCR that was the casing that was in use, and because of a bug that is believed to be in the underlying framework used by choco, we need to continue to use the same casing that the first version submitted used.
See chocolatey/choco#1040

@AdmiringWorm
Copy link
Member

As it is a problem with the conversion between upper case and lower case characters done by Chocolatey, can you please open up an issue about that over at https://github.com/chocolatey/choco/issues so it can be looked into?

@hkdemiralp
Copy link
Author

hkdemiralp commented Feb 22, 2022

As it is a problem with the conversion between upper case and lower case characters done by Chocolatey, can you please open up an issue about that over at https://github.com/chocolatey/choco/issues so it can be looked into?

Thanks. I opened up an issue as you said at chocolatey/choco#2601
I don't know if I should close the issue here. Please feel free to close, if it is the way it should be.

@pauby pauby removed the Bug label Feb 23, 2022
@pauby
Copy link
Member

pauby commented Feb 23, 2022

Thanks @hkdemiralp. I'll go ahead and close this now.

@pauby pauby closed this as completed Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants