Extend PKG_CONFIG_PATH rather than overwriting it
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 20/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Estagnada
- Stack de tecnologia
- github-actions, typescript
Direção de pesquisa
A issue identifica o tratamento de PKG_CONFIG_PATH pelo setup-python como ponto de entrada; inspecione o código da action relacionado ao ambiente e à escrita de caminhos, além de quaisquer testes relevantes. Verifique o comportamento com um PKG_CONFIG_PATH existente e confirme que tanto os diretórios anteriores quanto os gerados continuam disponíveis. O trabalho já está representado pelo PR #1242.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Description:
As of now, the setup-python action overwrites the PKG_CONFIG_PATH variable during its execution, which can lead to errors if this variable has already been set to a specific value (which is then no longer available). PKG_CONFIG_PATH works as the normal PATH does (from the man page of pkg-config)
PKG_CONFIG_PATH
A colon-separated list of high-priority directories where pc(5) files are looked up.
It would therefore be a good idea to instead extend PKG_CONFIG_PATH by adding the new directory to the existing value if one is present in the environment. This will make both existing and new pc files available (We would not want to overwrite PATH either, would we?)
Justification:
I realized this issue after debugging session of a CI pipeline which had a global
env:
PKG_CONFIG_PATH: '/special/path'
attached to a given workflow. After installing python using setup-python the python dependencies (which included a native library with a pc file) could not be installed any more, since that variable had been overwritten. This is a rather insidious error which is really hard to find.
Are you willing to submit a PR?
Working on a PR.
- Linguagem predominante
- TypeScript
- Estrelas
- 2.2k
- Forks
- 739
- Merge médio
- 6d 18h
- PRs com merge (30d)
- 1
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de actions/setup-python
-
bug
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 55/100
actions/setup-python#1348 · 5 comentários ·
-
feature request
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
actions/setup-python#1346 · 12 comentários ·
-
feature request
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 58/100
actions/setup-python#1308 · 8 comentários ·
-
feature request
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 35/100
actions/setup-python#1301 · 5 comentários ·
-
feature request
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 55/100
actions/setup-python#1294 · 3 comentários · 2 reações ·
Todas as issues de actions/setup-python
Issues semelhantes
-
blocklist removal
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
MetaMask/eth-phishing-detect#296544 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
pastelsky/bundlephobia#1122 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
category/development priority/P2 scope/file-operations scope/testing type/enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
-
Enatega Customer and Rider app: Add-ons price is not visible to customer after order is placed. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100