Installed iceberg-config.cmake calls find_dependency(Avro) instead of avro-cpp
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 88/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- cmake, cpp
- Bereich
- build-system
Rechercherichtung
Beginne in cmake_modules/IcebergThirdpartyToolchain.cmake bei resolve_avro_dependency() und untersuche anschließend iceberg_find_dependencies() in der installierten Konfiguration. Reproduziere das Problem mit einer systemweiten avro-cpp-Abhängigkeit und einem Consumer, der find_package(iceberg CONFIG REQUIRED) aufruft. Erledigt ist die Aufgabe, wenn die installierte Konfiguration avro-cpp anfordert und mit dem Systempaket erfolgreich konfiguriert wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Summary
When iceberg-cpp is built against a system Avro (avro-cpp found through FIND_PACKAGE_ARGS), the installed
iceberg-config.cmake can't be consumed: find_package(iceberg CONFIG) fails at find_dependency(Avro).
Cause
In resolve_avro_dependency() (cmake_modules/IcebergThirdpartyToolchain.cmake), the non-vendored branch does:
else()
set(AVRO_VENDORED FALSE)
list(APPEND ICEBERG_SYSTEM_DEPENDENCIES Avro)
endif()
However, the package that was actually found is avro-cpp. It is declared with fetchcontent_declare(avro-cpp ...
FIND_PACKAGE_ARGS ...), and its config exports the avro-cpp::avrocpp_* targets. Avro 1.12 installs
avro-cpp-config.cmake and no Avro package. So iceberg_find_dependencies() in the installed config calls
find_dependency(Avro), which can't be satisfied.
Reproduction
Seen with v0.4.0 (the code is unchanged on main) using vcpkg's avro-cpp 1.12.2 on x64-windows:
- Build and install iceberg-cpp with the bundle enabled against the system
avro-cpp. - In a consumer project, call
find_package(iceberg CONFIG REQUIRED). - Configuring fails because the
Avropackage can't be found.
Suggested fix
Record the real package name:
list(APPEND ICEBERG_SYSTEM_DEPENDENCIES avro-cpp)
The same one-line change is carried as a patch in the proposed vcpkg port; I'm happy to open a PR here as well.
- Vorherrschende Sprache
- C++
- Sterne
- 223
- Forks
- 127
- Ø Merge
- 1 T. 13 Std.
- Gemergte PRs (30 T.)
- 28
Entwicklungsumgebung
Die Einrichtungsdateien dieses Projekts haben wir noch nicht geprüft. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
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 apache/iceberg-cpp
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 20/100
apache/iceberg-cpp#959 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
apache/iceberg-cpp#955 · 3 Kommentare · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
apache/iceberg-cpp#946 · 5 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
apache/iceberg-cpp#944 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
apache/iceberg-cpp#943 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in apache/iceberg-cpp
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Icinga/icinga2#11058 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
component: split-view platform: windows
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
zen-browser/desktop#15616 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
-
area/ysql kind/bug priority/medium status/awaiting-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
yugabyte/yugabyte-db#34415 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
WayfireWM/wayfire#3148 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag