facebook/zstd

zstd fails to process some filenames on Windows [we need a hero]

Open

#3.882 aberto em 24 de jan. de 2024

Ver no GitHub
 (19 comments) (0 reactions) (0 assignees)C (2.481 forks)batch import
help wanted

Métricas do repositório

Stars
 (27.100 stars)
Métricas de merge de PR
 (Mesclagem média 50m) (2 fundiu PRs em 30d)

Description

Windows 7 x64, zstd 1.5.5

$ dir /b
birds.txt
pájaros.txt
vögel.txt
птицы.txt

$ zstd *.*
zstd: can't stat pajaros.txt : No such file or directory -- ignored
zstd: can't stat vogel.txt : No such file or directory -- ignored
  2 files compressed : 85.71% (  3.00 KiB =>   2.57 KiB)
 
$ dir /b *.zst
birds.txt.zst
птицы.txt.zst

@Cyan4973 said zstd doesn't do anything special, it uses the C standard fopen() function. This should be naturally compatible with any utf-8 character set. But I suspect Windows does something different with the character set, so there might be a need to employ other non-portable Windows variants of > fopen for these scenarios.

Guia do colaborador

Direção de pesquisa
O problema é que o zstd falha ao processar nomes de arquivo com caracteres não ASCII no Windows porque ele usa a função fopen() padrão do C, que não lida com nomes de arquivo UTF 8 nessa plataforma. A correção provavelmente envolve o uso de funções de arquivo específicas do Windows para caracteres largos, como wfopen() ou a API mais recente CreateFileW(). Examine o código fonte em programs/fileio.c, onde a abertura de arquivos é tratada, e investigue como usar condicionalmente a função apropriada com base na macro WIN32. Consulte o comentário do mantenedor na issue para obter orientação.
Pilha de tecnologia
c
Domain
clitooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
CWindows API basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45