python / python/cpython

Confusing syntax error for missing comma after import-as target

Aberta
#146,492 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

3.14 3.15 interpreter-core topic-parser type-bug
Linguagem predominante
Python
Estrelas
77.2k
Forks
35.9k
Métricas de merge de PRs
Métricas de PR pendentes

Descrição

Bug report

Bug description:

Right now we have:

>>> import a as a  b
  File "<python-input-0>", line 1
    import a as a  b
                ^
SyntaxError: cannot use name as import target

>>> from x import a as a  b
  File "<python-input-1>", line 1
    from x import a as a  b
                       ^
SyntaxError: cannot use name as import target

This happens since these cases match these two rules.

One possible fix would be to add NAME to the list of not-matched tokens so that the current rules don't match this case.

However, I have an idea for a slightly nicer error. PR forthcoming.

cc @pablogsal @lysnikolaou @sobolevn

CPython versions tested on:

3.14, 3.15, CPython main branch

Operating systems tested on:

Linux

Linked PRs
  • gh-146493

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Reproduza os dois exemplos de import-as no Python REPL e, em seguida, inspecione as duas regras correspondentes em Grammar/python.gram próximas às linhas 1435 e 1439. Compare o diagnóstico resultante com o comportamento esperado descrito na issue; considera-se concluído quando o caso de vírgula ausente recebe um erro de sintaxe mais claro. Um PR vinculado já está indicado.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
python
Domínio
compilers
Tipo de issue
Bug
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
25/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.