codrsquad / codrsquad/portable-python

possible incorrectness in libffi include dir assignment, doesn't seem impactful

Offen
#61 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Python
Sterne
86
Forks
13
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

This looks like an obvious mistake but I don't think it matters because the pkgconfig files are correct and provide the path ultimately:

https://github.com/codrsquad/portable-python/blob/89a620140f6f7bb066f77774c23187d78cf96275/src/portable_python/cpython.py#L142

            if self.active_module(LibFFI):
                yield f"LIBFFI_INCLUDEDIR={self.deps_lib}"

I'll refrain from opening a PR because I'm not sure... it's possible that this is necessary due to libffi nuances and just looks wrong, but there's no comment or commit message to indicate if thats the case.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Lies src/portable_python/cpython.py in Zeile 142 und vergleiche die Zuweisung von LIBFFI_INCLUDEDIR mit den im Issue erwähnten libffi-pkg-config-Pfaden. Prüfe, ob die Zuweisung den Build trotz korrekter pkg-config-Dateien beeinflusst. Erledigt bedeutet, das Verhalten zu bestätigen und die Abweichung zu dokumentieren oder mit einem fokussierten Test zu korrigieren, falls das Repository einen solchen bereitstellt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
build-system
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.