Re: [GNC-dev] Windows Build problem
Yes, Thanks. Committed on your behalf and pushed. Regards, John Ralls > On Sep 13, 2019, at 10:37 AM, Matthew Forbis via gnucash-devel > wrote: > > I might have fixed it myself. Does the following change make sense? > > > > CMakeLists.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/CMakeLists.txt b/CMakeLists.txt > index 0654dfb5a..b054f9cc2 100644 > --- a/CMakeLists.txt > +++ b/CMakeLists.txt > @@ -217,9 +217,9 @@ if (WITH_GNUCASH) >pkg_check_modules (WEBKIT REQUIRED IMPORTED_TARGET webkitgtk-3.0) > else() >pkg_check_modules (WEBKIT REQUIRED webkitgtk-3.0) > + _pkg_create_imp_target(WEBKIT) > endif() > set(WEBKIT1 1 CACHE INTERNAL "WebKitGtk") > -_pkg_create_imp_target(WEBKIT) >else (WIN32 OR APPLE) > If (NOT CMAKE_VERSION VERSION_LESS 3.6) >pkg_check_modules (WEBKIT IMPORTED_TARGET webkit2gtk-4.0) > > > Thanks, > Matt > > > > On Friday, September 13, 2019, 12:26:22 PM CDT, Matthew Forbis via > gnucash-devel wrote: > > > > > > Hi, > I'm attempting to build on windows to try to still solve the report bug I'm > interested in and it seems that the patch below might have broken the windows > build (or I don't know what I'm doing is also a possibilty). > With the commit John pushed yesterday, I get the following error message when > cmake is running: > > CMake Error at CMakeLists.txt:222 (_pkg_create_imp_target): > _pkg_create_imp_target Function invoked with incorrect arguments for > function named: _pkg_create_imp_target > > Can anyone help? > Thanks,Matt > > > On Thursday, September 12, 2019, 11:22:42 PM CDT, John Ralls > wrote: > > Updatedvia https://github.com/Gnucash/gnucash/commit/41c58ec0 (commit) > via https://github.com/Gnucash/gnucash/commit/c23f3f05 (commit) > from https://github.com/Gnucash/gnucash/commit/70eb7172 (commit) > > > > commit 41c58ec00ab2276ca9b326570d07689fc47bc7e9 > Author: John Ralls > Date: Thu Sep 12 14:45:23 2019 -0700 > > Convert Gtk3 and Webkit to imported targets. > > This causes CMake to include their headers with -isystem instead of -I, > so the compiler doesn't generate warnings on them. ArchLinux was failing > to build because of such warnings. > > Includes a function from cmake 3.6 to allow this to work on systems like > Ubuntu 14.04 that still provide only cmake 3.5. > > commit c23f3f05e311d81ce0fbb7bbc6b850e5acd5776d > Author: John Ralls > Date: Tue Sep 10 16:33:33 2019 -0700 > > Fix deprecated G_TYPE_INSTANCE_GET_PRIVATE and G_INLINE_FUNC > > with the function/declaration that they substituted. > Note that this doesn't use the recommended new GObject creation macros > because the class names in libgnucash/engine don't follow the gnome > naming convention. > > > > Summary of changes: > CMakeLists.txt| 61 ++ > common/cmake_modules/GncPkgConfig.cmake| 52 ++ > gnucash/CMakeLists.txt| 6 +-- > gnucash/gnome-search/CMakeLists.txt| 2 +- > gnucash/gnome-search/gnc-general-search.c | 2 +- > gnucash/gnome-search/search-account.c | 2 +- > gnucash/gnome-search/search-boolean.c | 2 +- > gnucash/gnome-search/search-core-type.c| 2 +- > gnucash/gnome-search/search-date.c| 2 +- > gnucash/gnome-search/search-double.c | 2 +- > gnucash/gnome-search/search-int64.c| 2 +- > gnucash/gnome-search/search-numeric.c | 2 +- > gnucash/gnome-search/search-reconciled.c | 2 +- > gnucash/gnome-search/search-string.c | 2 +- > gnucash/gnome-utils/CMakeLists.txt| 4 +- > gnucash/gnome-utils/gnc-combott.c | 3 +- > gnucash/gnome-utils/gnc-currency-edit.c| 2 +- > gnucash/gnome-utils/gnc-date-format.c | 2 +- > gnucash/gnome-utils/gnc-embedded-window.c | 2 +- > gnucash/gnome-utils/gnc-main-window.c | 2 +- > gnucash/gnome-utils/gnc-period-select.c| 2 +- > gnucash/gnome-utils/gnc-plugin-file-history.c | 2 +- > gnucash/gnome-utils/gnc-plugin-manager.c | 2 +- > gnucash/gnome-utils/gnc-plugin-menu-additions.c| 2 +- > gnucash/gnome-utils/gnc-plugin-page.c | 2 +- > gnucash/gnome-utils/gnc-plugin.c | 2 +- > gnucash/gnome-utils/gnc-query-view.c | 2 +- > gnucash/gnome-utils/gnc-tree-model-account-types.c | 3 +- > gnucash/gnome-utils/gnc-tree-model-account.c | 2 +- > gnucash/gnome-utils/gnc-tree-model-commodity.c| 2 +- > gnucash/gnome-utils/gnc-tree-model-owner.c| 2 +- > gnucash/gnome-utils/gnc-tree-model-price.c| 2 +- > gnucash/gnome-utils/gnc-tree-model-selection.c| 2 +- > gnucash/gnome-utils/gnc-tree-model-split-reg.c| 2 +- > gnucash/gnome-util
Re: [GNC-dev] Windows Build problem
I might have fixed it myself. Does the following change make sense? CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 0654dfb5a..b054f9cc2 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -217,9 +217,9 @@ if (WITH_GNUCASH) pkg_check_modules (WEBKIT REQUIRED IMPORTED_TARGET webkitgtk-3.0) else() pkg_check_modules (WEBKIT REQUIRED webkitgtk-3.0) + _pkg_create_imp_target(WEBKIT) endif() set(WEBKIT1 1 CACHE INTERNAL "WebKitGtk") - _pkg_create_imp_target(WEBKIT) else (WIN32 OR APPLE) If (NOT CMAKE_VERSION VERSION_LESS 3.6) pkg_check_modules (WEBKIT IMPORTED_TARGET webkit2gtk-4.0) Thanks, Matt On Friday, September 13, 2019, 12:26:22 PM CDT, Matthew Forbis via gnucash-devel wrote: Hi, I'm attempting to build on windows to try to still solve the report bug I'm interested in and it seems that the patch below might have broken the windows build (or I don't know what I'm doing is also a possibilty). With the commit John pushed yesterday, I get the following error message when cmake is running: CMake Error at CMakeLists.txt:222 (_pkg_create_imp_target): _pkg_create_imp_target Function invoked with incorrect arguments for function named: _pkg_create_imp_target Can anyone help? Thanks,Matt On Thursday, September 12, 2019, 11:22:42 PM CDT, John Ralls wrote: Updated via https://github.com/Gnucash/gnucash/commit/41c58ec0 (commit) via https://github.com/Gnucash/gnucash/commit/c23f3f05 (commit) from https://github.com/Gnucash/gnucash/commit/70eb7172 (commit) commit 41c58ec00ab2276ca9b326570d07689fc47bc7e9 Author: John Ralls Date: Thu Sep 12 14:45:23 2019 -0700 Convert Gtk3 and Webkit to imported targets. This causes CMake to include their headers with -isystem instead of -I, so the compiler doesn't generate warnings on them. ArchLinux was failing to build because of such warnings. Includes a function from cmake 3.6 to allow this to work on systems like Ubuntu 14.04 that still provide only cmake 3.5. commit c23f3f05e311d81ce0fbb7bbc6b850e5acd5776d Author: John Ralls Date: Tue Sep 10 16:33:33 2019 -0700 Fix deprecated G_TYPE_INSTANCE_GET_PRIVATE and G_INLINE_FUNC with the function/declaration that they substituted. Note that this doesn't use the recommended new GObject creation macros because the class names in libgnucash/engine don't follow the gnome naming convention. Summary of changes: CMakeLists.txt | 61 ++ common/cmake_modules/GncPkgConfig.cmake | 52 ++ gnucash/CMakeLists.txt | 6 +-- gnucash/gnome-search/CMakeLists.txt | 2 +- gnucash/gnome-search/gnc-general-search.c | 2 +- gnucash/gnome-search/search-account.c | 2 +- gnucash/gnome-search/search-boolean.c | 2 +- gnucash/gnome-search/search-core-type.c | 2 +- gnucash/gnome-search/search-date.c | 2 +- gnucash/gnome-search/search-double.c | 2 +- gnucash/gnome-search/search-int64.c | 2 +- gnucash/gnome-search/search-numeric.c | 2 +- gnucash/gnome-search/search-reconciled.c | 2 +- gnucash/gnome-search/search-string.c | 2 +- gnucash/gnome-utils/CMakeLists.txt | 4 +- gnucash/gnome-utils/gnc-combott.c | 3 +- gnucash/gnome-utils/gnc-currency-edit.c | 2 +- gnucash/gnome-utils/gnc-date-format.c | 2 +- gnucash/gnome-utils/gnc-embedded-window.c | 2 +- gnucash/gnome-utils/gnc-main-window.c | 2 +- gnucash/gnome-utils/gnc-period-select.c | 2 +- gnucash/gnome-utils/gnc-plugin-file-history.c | 2 +- gnucash/gnome-utils/gnc-plugin-manager.c | 2 +- gnucash/gnome-utils/gnc-plugin-menu-additions.c | 2 +- gnucash/gnome-utils/gnc-plugin-page.c | 2 +- gnucash/gnome-utils/gnc-plugin.c | 2 +- gnucash/gnome-utils/gnc-query-view.c | 2 +- gnucash/gnome-utils/gnc-tree-model-account-types.c | 3 +- gnucash/gnome-utils/gnc-tree-model-account.c | 2 +- gnucash/gnome-utils/gnc-tree-model-commodity.c | 2 +- gnucash/gnome-utils/gnc-tree-model-owner.c | 2 +- gnucash/gnome-utils/gnc-tree-model-price.c | 2 +- gnucash/gnome-utils/gnc-tree-model-selection.c | 2 +- gnucash/gnome-utils/gnc-tree-model-split-reg.c | 2 +- gnucash/gnome-utils/gnc-tree-model.c | 2 +- gnucash/gnome-utils/gnc-tree-view-account.c | 2 +- gnucash/gnome-utils/gnc-tree-view-commodity.c | 2 +- gnucash/gnome-utils/gnc-tree-view-owner.c | 2 +- gnucash/gnome-utils/gnc-tree-view-price.c | 2 +- gnucash/gnome-utils/gnc-tree-view-split-reg.c | 2 +- gnucash/gnome-utils/gnc-tree-view-sx-list.c | 2 +- g