Extend PKG_CONFIG_PATH rather than overwriting it
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 20/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- github-actions, typescript
Rechercherichtung
Das Issue identifiziert die Handhabung von PKG_CONFIG_PATH durch setup-python als Einstiegspunkt; untersuche den Umgebungs- und Pfadschreibcode der Action sowie alle relevanten Tests. Überprüfe das Verhalten mit einem vorhandenen PKG_CONFIG_PATH und bestätige, dass sowohl die bisherigen als auch die generierten Verzeichnisse weiterhin verfügbar bleiben. Die Arbeit ist bereits durch PR #1242 repräsentiert.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 2.2k
- Forks
- 739
- Ø Merge
- 6 T. 18 Std.
- Gemergte PRs (30 T.)
- 1
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus actions/setup-python
-
bug
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
actions/setup-python#1348 · 5 Kommentare ·
-
feature request
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
actions/setup-python#1346 · 12 Kommentare ·
-
feature request
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 58/100
actions/setup-python#1308 · 8 Kommentare ·
-
feature request
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 35/100
actions/setup-python#1301 · 5 Kommentare ·
-
feature request
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
actions/setup-python#1294 · 3 Kommentare · 2 Reaktionen ·
Alle Issues in actions/setup-python
Ähnliche Issues
-
blocklist removal
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
MetaMask/eth-phishing-detect#296544 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
pastelsky/bundlephobia#1122 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
category/development priority/P2 scope/file-operations scope/testing type/enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Enatega Customer and Rider app: Add-ons price is not visible to customer after order is placed. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100