ocaml/dune

[dune-configurator] PKG_CONFIG_PATH is ignored on macos

Offen

#10.805 geöffnet am 06.08.2024

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)OCaml (478 Forks)github user discovery
help wantedmacos

Repository-Metriken

Stars
 (1.882 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 20h) (345 gemergte PRs in 30 T)

Beschreibung

Expected Behavior

When providing custom PKG_CONFIG_PATH variable dependencies

Actual Behavior

PKG_CONFIG_PATH is ignored in favor of brew https://github.com/ocaml/dune/blob/main/otherlibs/configurator/src/v1.ml

Reproduction

  1. Get a mac
  2. Add a folder with custom pkg-config, e.g. vcpkg install libpng
  3. Run PKG_CONFIG= dune build
  4. Observe error

Specifications

  • Version of dune (output of dune --version): 3.16.0
  • Version of ocaml (output of ocamlc --version): 5.2.0
  • Operating system (distribution and version): MacOS 14.5

Additional information

Despite the code looks correct it is for some reason still not running, would be also good to log the modified PKG_CONFIG_PATH

Contributor Guide

Research-Richtung
Untersuchen Sie den Code in otherlibs/configurator/src/v1.ml, um zu verstehen, warum PKG CONFIG PATH auf macOS durch Homebrew Pfade überschrieben wird. Prüfen Sie, ob es eine Bedingung gibt, die benutzerdefinierte Pfade überspringt.
Tech Stack
Keine
Domain
build system
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitOCamlmacOS
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60