Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

cjmayo
Copy link
Contributor

@cjmayo cjmayo commented Sep 11, 2025

See linked issue.

@MaartenBent
Copy link
Contributor

I do not understand this change, libdir is already assigned this exact value in wx_write_config here:

set(libdir "\${exec_prefix}/${library_dir}")

And then wx_configure_script is called from this function.

And for wx_write_config_inplace this is not needed, because it doesn't contain libdir.

@cjmayo
Copy link
Contributor Author

cjmayo commented Sep 12, 2025

Adding this patch:

--- a/build/cmake/config.cmake
+++ b/build/cmake/config.cmake
@@ -12,9 +12,11 @@ file(MAKE_DIRECTORY ${wxCONFIG_DIR})
 set(TOOLCHAIN_FULLNAME ${wxBUILD_FILE_ID})
 
 macro(wx_configure_script input output)
+    message("1 " ${output} " " ${libdir})
     # variable used in wx-config.in
     wx_get_install_dir(library "lib")
     set(libdir "\${exec_prefix}/${library_dir}")
+    message("2 "  ${output} " " ${libdir})
     # variables used in wx-config-inplace.in
     set(abs_top_srcdir ${wxSOURCE_DIR})
     set(abs_top_builddir ${wxBINARY_DIR})

I get:

1 inplace-gtk3-unicode-3.3 
2 inplace-gtk3-unicode-3.3 ${exec_prefix}/lib64
1 gtk3-unicode-3.3 ${exec_prefix}/lib
2 gtk3-unicode-3.3 ${exec_prefix}/lib64

@cjmayo
Copy link
Contributor Author

cjmayo commented Sep 13, 2025

Yes, not needed since 771d4df.

@cjmayo cjmayo closed this Sep 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants