swiftlang/swift

[SR-7320] GYB is run with python, not python2.7

Aberta

#49.868 aberto em 31 de mar. de 2018

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)Swift (10.719 forks)batch import
bugcmakegood first issue

Métricas do repositório

Stars
 (69.989 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description
Previous ID SR-7320
Radar None
Original Reporter sanxiyn (JIRA User)
Type Bug
Votes 0
Component/s
Labels Bug, CMake, StarterBug
Assignee None
Priority Medium

md5: a456e83bcda5542ee8cf2382f8fa9b21

Issue Description:

https://github.com/apple/swift/pull/1535 made GYB run with python2.7, but https://github.com/apple/swift/pull/6463 effectively reverted the change because GYB is run with PYTHON_EXECUTABLE instead of using shebang line. This causes the problem 1535 fixed to reappear.

Guia do colaborador

Direção de pesquisa
Investigue os arquivos de configuração CMake no repositório que invocam o GYB. Examine os PRs #1535 e #6463 referenciados para entender a correção anterior e sua reversão. O problema é que o GYB é executado usando PYTHON EXECUTABLE em vez de usar explicitamente python2.7. Modifique o CMake para garantir que python2.7 seja usado. Verifique se há outras ocorrências onde a versão do Python possa ser ambígua.
Pilha de tecnologia
swiftpython
Domain
build system
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.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
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
CMake
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80