cake-build/cake

Use OS-native directory separators in Path.FullPath

Open

#2.367 aberto em 25 de nov. de 2018

Ver no GitHub
 (4 comments) (4 reactions) (0 assignees)C# (738 forks)batch import
Help wantedImprovementUp-for-grabs

Métricas do repositório

Stars
 (3.742 stars)
Métricas de merge de PR
 (Mesclagem média 1d 1h) (25 fundiu PRs em 30d)

Description

There are applications which do not understand the alternate directory separator; for example, https://github.com/cake-build/cake/issues/2337. There have also been multiple CLI tools I've used where I had to do .FullName.Replace('/', '\\').

Would you consider returning OS-native directory separators from Path.FullName? It will just work in all situations, and it will not stand out and look strange on Windows any longer even when it does work. (At times, I've manually edited MSBuild logs etc so people don't ask why I'm using weird paths.)

Guia do colaborador

Direção de pesquisa
Examine a implementação de Path.FullPath no código do Cake e verifique se ele usa o separador de diretório nativo do sistema operacional (Path.DirectorySeparatorChar) em vez de uma barra codificada. Em seguida, verifique se todos os testes passam no Windows.
Pilha de tecnologia
csharp
Domain
build system
Tipo Issue
Funcionalidade
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70