Extend PKG_CONFIG_PATH rather than overwriting it
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 20/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- github-actions, typescript
Piste de recherche
L’issue identifie la gestion de PKG_CONFIG_PATH par setup-python comme point d’entrée ; examine le code de l’action qui concerne l’environnement et l’écriture des chemins, ainsi que tous les tests pertinents. Vérifie le comportement avec un PKG_CONFIG_PATH existant et confirme que les répertoires précédents et générés restent tous deux disponibles. Le travail est déjà représenté par PR #1242.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- TypeScript
- Étoiles
- 2.2k
- Forks
- 739
- Merge moyen
- 6 j 18 h
- PR mergées (30 j)
- 1
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de actions/setup-python
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
actions/setup-python#1348 · 5 commentaires ·
-
feature request
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
actions/setup-python#1346 · 12 commentaires ·
-
feature request
Difficulté 3/5 1-2 jours Accessibilité débutants 58/100
actions/setup-python#1308 · 8 commentaires ·
-
feature request
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
actions/setup-python#1301 · 5 commentaires ·
-
feature request
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
actions/setup-python#1294 · 3 commentaires · 2 réactions ·
Toutes les issues de actions/setup-python
Issues similaires
-
blocklist removal
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
MetaMask/eth-phishing-detect#296544 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
pastelsky/bundlephobia#1122 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
category/development priority/P2 scope/file-operations scope/testing type/enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Enatega Customer and Rider app: Add-ons price is not visible to customer after order is placed. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100