-
-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Latest Conda-forge version of ncvis-climate not linking on MacOS (Mac CPU) #15
Comments
@mahf708, is this something you have the bandwidth to look into? It seems like maybe the run_exports are just too loose on this package: |
@czender install anything before wxwidgets 3.2.6 resolves the issues (so try @xylar that's a good suggestion; it's clearly the issue at play, though I would need to defer to the wxwidgets feestock maintainers @hmaarrfk: any thoughts on this appreciated; if not objections, I will be happy to issue a PR and get this sorted. |
To be verbose, I think the solution would entail making wxwidgets a conda-forge-globally tracked pin on the patch version; this may have annoying consequence and may be undesirable, especially this could be a small bug that went undetected. Either way, we will need to issue some patch in admin-requests to get things to fix things for current builds. I will take care of this second point once we get clarity about what we want to do. |
happy to get help with wxwidgets However, they claim it is ABI compatible... https://www.wxwidgets.org/news/2024/09/wxwidgets-3.2.6-released/ maybe it is something I broke? |
Interesting they claim that. If you look at wxWidgets/wxWidgets@v3.2.5...v3.2.6 and find the Makefile.in file, do you think that keeps it ABI compatible? Quoted below for brevity: From d7f6a7e8d2522e740af0899418c22af19fdb4aeb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Va=CC=81clav=20Slavi=CC=81k?= <vaclav@slavik.io>
Date: Sat, 10 Feb 2024 11:46:13 +0100
Subject: [PATCH 38/73] Fix AddAvailableCatalog() API
Knowing msgIdLanguage is necessary for correct evaluation of preferred
languages order; without it, it would be possible to load undesirable
translation file.
See 2af36e9560c694a750b6aa835d32d12a0393fb83 for details.
---
Makefile.in | 6 +++---
build/bakefiles/version.bkl | 6 +++---
include/wx/translation.h | 12 ++++++++++++
interface/wx/translation.h | 15 +++++++++++++--
src/common/translation.cpp | 31 ++++++++++++++++---------------
version-script.in | 7 +++++++
6 files changed, 54 insertions(+), 23 deletions(-)
diff --git a/Makefile.in b/Makefile.in
index d6c26128855e..86948bf57673 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -2215,7 +2215,7 @@ LOCALE_LINGUAS = ca cs da de el es fi fr hu id it ja nl pl ru sl sv tr uk zh zh_
@COND_DEPS_TRACKING_1@CXXC = $(BK_DEPS) $(CXX)
@COND_USE_PLUGINS_0@PLUGIN_ADV_EXTRALIBS = $(EXTRALIBS_SDL)
@COND_PLATFORM_MACOSX_1@WXMACVERSION_CMD = \
-@COND_PLATFORM_MACOSX_1@ -compatibility_version 3.0 -current_version 3.4
+@COND_PLATFORM_MACOSX_1@ -compatibility_version 4.0 -current_version 4.0
@COND_USE_GUI_0@PORTNAME = base
@COND_USE_GUI_1@PORTNAME = $(TOOLKIT_LOWERCASE)$(TOOLKIT_VERSION)
@COND_TOOLKIT_MAC@WXBASEPORT = _carbon
@@ -13977,9 +13977,9 @@ COND_MONOLITHIC_0_SHARED_1_USE_GUI_1_USE_HTML_1___htmldll_library_link_LIBR_0 \
@COND_USE_SOVERSION_0@dll___targetsuf2 = .$(SO_SUFFIX)
@COND_PLATFORM_MACOSX_0_USE_SOVERCYGWIN_0_USE_SOVERSION_1@dll___targetsuf3 \
@COND_PLATFORM_MACOSX_0_USE_SOVERCYGWIN_0_USE_SOVERSION_1@ = \
-@COND_PLATFORM_MACOSX_0_USE_SOVERCYGWIN_0_USE_SOVERSION_1@ .$(SO_SUFFIX).0.2.4
+@COND_PLATFORM_MACOSX_0_USE_SOVERCYGWIN_0_USE_SOVERSION_1@ .$(SO_SUFFIX).0.3.0
@COND_PLATFORM_MACOSX_1_USE_SOVERSION_1@dll___targetsuf3 \
-@COND_PLATFORM_MACOSX_1_USE_SOVERSION_1@ = .0.2.4.$(SO_SUFFIX)
+@COND_PLATFORM_MACOSX_1_USE_SOVERSION_1@ = .0.3.0.$(SO_SUFFIX)
@COND_USE_SOVERCYGWIN_1_USE_SOVERSION_1@dll___targetsuf3 = -0.$(SO_SUFFIX)
@COND_USE_SOVERSION_0@dll___targetsuf3 = .$(SO_SUFFIX)
@COND_USE_SOVERSION_1_USE_SOVERSOLARIS_1@dll___targetsuf3 = .$(SO_SUFFIX).0 diff --git a/Makefile.in b/Makefile.in
index 93fbb2328d86..d6c26128855e 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -82,7 +82,7 @@ wxCFLAGS_C99 = @wxCFLAGS_C99@
DESTDIR =
WX_RELEASE = 3.2
WX_RELEASE_NODOT = 32
-WX_VERSION = $(WX_RELEASE).5
+WX_VERSION = $(WX_RELEASE).6
LIBDIRNAME = $(wx_top_builddir)/lib
WXREGEX_CFLAGS = $(WX_CPPFLAGS) -DNDEBUG \
-I$(wx_top_builddir)/3rdparty/pcre/src -D__WX__ -DHAVE_CONFIG_H \
@@ -2215,7 +2215,7 @@ LOCALE_LINGUAS = ca cs da de el es fi fr hu id it ja nl pl ru sl sv tr uk zh zh_
@COND_DEPS_TRACKING_1@CXXC = $(BK_DEPS) $(CXX)
@COND_USE_PLUGINS_0@PLUGIN_ADV_EXTRALIBS = $(EXTRALIBS_SDL)
@COND_PLATFORM_MACOSX_1@WXMACVERSION_CMD = \
-@COND_PLATFORM_MACOSX_1@ -compatibility_version 3.0 -current_version 3.3
+@COND_PLATFORM_MACOSX_1@ -compatibility_version 3.0 -current_version 3.4
@COND_USE_GUI_0@PORTNAME = base
@COND_USE_GUI_1@PORTNAME = $(TOOLKIT_LOWERCASE)$(TOOLKIT_VERSION)
@COND_TOOLKIT_MAC@WXBASEPORT = _carbon
@@ -13977,9 +13977,9 @@ COND_MONOLITHIC_0_SHARED_1_USE_GUI_1_USE_HTML_1___htmldll_library_link_LIBR_0 \
@COND_USE_SOVERSION_0@dll___targetsuf2 = .$(SO_SUFFIX)
@COND_PLATFORM_MACOSX_0_USE_SOVERCYGWIN_0_USE_SOVERSION_1@dll___targetsuf3 \
@COND_PLATFORM_MACOSX_0_USE_SOVERCYGWIN_0_USE_SOVERSION_1@ = \
-@COND_PLATFORM_MACOSX_0_USE_SOVERCYGWIN_0_USE_SOVERSION_1@ .$(SO_SUFFIX).0.2.3
+@COND_PLATFORM_MACOSX_0_USE_SOVERCYGWIN_0_USE_SOVERSION_1@ .$(SO_SUFFIX).0.2.4
@COND_PLATFORM_MACOSX_1_USE_SOVERSION_1@dll___targetsuf3 \
-@COND_PLATFORM_MACOSX_1_USE_SOVERSION_1@ = .0.2.3.$(SO_SUFFIX)
+@COND_PLATFORM_MACOSX_1_USE_SOVERSION_1@ = .0.2.4.$(SO_SUFFIX)
@COND_USE_SOVERCYGWIN_1_USE_SOVERSION_1@dll___targetsuf3 = -0.$(SO_SUFFIX)
@COND_USE_SOVERSION_0@dll___targetsuf3 = .$(SO_SUFFIX)
@COND_USE_SOVERSION_1_USE_SOVERSOLARIS_1@dll___targetsuf3 = .$(SO_SUFFIX).0 |
This is enough research for me to know the pr you would make is well thought out. |
happy to help, @czender; we will clean this up more systematically once we figure out a clean solution for the overall conda-forge ecosystem.
Let me investigate a little more; file an issue upstream; and then I will issue a PR with a reasonable solution. Thanks for your help and patience :) |
Thanks for jumping on this @mahf708, especially because I know you're not feeling great. |
This is now resolved. |
"now resolved", rather than "not resolved"? |
Yes sorry. It is RESOLVED now. Resolution involved a few steps:
now hopefully everything should be fixed and no need for any workarounds. |
Solution to issue cannot be found in the documentation.
Issue
Hi @xylar and @mahf708. This issue appears to be due to a slight mismatch between the desired and actual versions (
3.2.0.2.3
vs3.2.0.2.0
) oflibwx_osx_cocoau_xrc
that are installed. My understanding is that Conda-Forge feedstock are supposed to take care of library synchronization across any given Conda installation.NB: The same behavior occurs from an environment that only includes
ncvis-climate
, so I'm in a pickle. My reproducer script isthe results of which follow:
Any help appreciated,
Charlie
Installed packages
Environment info
The text was updated successfully, but these errors were encountered: