-
Notifications
You must be signed in to change notification settings - Fork 701
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
cabal sdist
corrupted when using Unicode
#2558
Comments
Once this is fixed, we should also add a regression test. |
@hvr By the way, what operating system is this on, and what is your default encoding set to? (It should work correctly regardless of these facts, but knowing these may help to track it down.) |
@ttuegel Ubuntu 15.04 w/ |
Real-world example:
$ ls -d test/resources/acc*
test/resources/accent test/resources/accenté
$ runhaskell Setup.lhs sdist
Building source dist for yaml-0.8.18.4...
Preprocessing library yaml-0.8.18.4...
Preprocessing executable 'yaml2json' for yaml-0.8.18.4...
Preprocessing executable 'json2yaml' for yaml-0.8.18.4...
Preprocessing test suite 'spec' for yaml-0.8.18.4...
Source tarball created: dist/yaml-0.8.18.4.tar.gz
$ cabal unpack dist/yaml-0.8.18.4.tar.gz
Unpacking to yaml-0.8.18.4/
$ ls -d yaml-0.8.18.4/test/resources/acc*
yaml-0.8.18.4/test/resources/accenté
$ cabal --version
cabal-install version 1.24.0.0
compiled using version 1.24.0.0 of the Cabal library
$ locale
LANG=ru_RU.UTF-8 Note how |
Edward pointed out it's even more complicated: #3758 |
underlying/related issue in tar haskell/tar#6 |
Thanks. And there is even some traffic in the tar issue, so let me downgrade priority on our side. Unless a workaround on cabal side is urgent for any users? |
The tar issue was closed a year ago. I wonder if benefit from that and can close the issue... |
I just confirmed that the reproducer in the OP is working now. (God bless reproducers!) Closing optimistically... |
That works as expected; however, when using
cabal-install
:The resulting tarball has filenames with
無
replaced by!
(fwiw,無
is UTF8-encoded as0xe7 0x84 0xa1
)See also #2557
The text was updated successfully, but these errors were encountered: