Skip to content

Conversation

@CyberTailor
Copy link
Contributor

@CyberTailor CyberTailor commented Jan 27, 2024

This PR is a follow-up to #2676. The following changes make it possible to build Gridcoin with CMake in Windows.

It also adds the Hunter dependency manager for systems without package management abilities.

TODO for next pull requests:

  • Add NSIS integration to CMake
  • Make the code compile with MSVC

@jamescowens
Copy link
Member

I just merged #2734. You probably need to rebase this on the development branch again.

@div72
Copy link
Member

div72 commented Jan 30, 2024

@CyberTailor Why hunter, instead of (for example) conan?

P.S. MacOS CI failure is unrelated, seems like a GH issue.

@CyberTailor
Copy link
Contributor Author

Why hunter, instead of (for example) conan?

I'm not familiar with Conan and don't know any projects that use it with CMake.

@CyberTailor CyberTailor reopened this Jan 30, 2024
@CyberTailor
Copy link
Contributor Author

CyberTailor commented Jan 30, 2024

(sorry for confusion, i've mistook browser tabs)

@jamescowens jamescowens requested a review from div72 January 30, 2024 17:06
@jamescowens jamescowens added this to the Miss Piggy milestone Jan 30, 2024
@jamescowens
Copy link
Member

@CyberTailor can we get this into Miss Piggy or should I push to Natasha?

@CyberTailor
Copy link
Contributor Author

I think yes. It'd be hard to continue without feedback

@jamescowens
Copy link
Member

@div72?

@jamescowens jamescowens modified the milestones: Miss Piggy, Natasha Feb 25, 2024
@div72
Copy link
Member

div72 commented Feb 26, 2024

I did take a peek at this PR, nothing popped out to me but I haven't used CMake before and I'm not a Windows user so I don't really think I'm qualified to give a proper review. There does seem to be something weird on how defines are processed (also on MacOS) but it seems to compile and run ok.


Since automake is currently our sanctioned build system, it shouldn't matter if this lands in Miss Piggy or Natasha imo as long as there are no code changes.

@jamescowens
Copy link
Member

I am not qualified to review it either. If it seems to compile and run ok, lets go ahead and merge it.

@jamescowens jamescowens merged commit e33e41d into gridcoin-community:development Mar 24, 2024
jamescowens added a commit to jamescowens/Gridcoin-Research that referenced this pull request Apr 10, 2024
Added
 - build: add option for sanitizers gridcoin-community#2553 (@div72)
 - build: CMake: Initial Windows support (MSYS2) gridcoin-community#2733 (@CyberTailor)

Changed
 - build: enforce SSE2 on x86 targets gridcoin-community#2746 (@div72)
 - consensus: Update checkpoint data for mainnet and testnet gridcoin-community#2756 (@jamescowens)
 - gui, util: Enhance verify checkpoints fail handling; use RegistryBookmarks for DB passivation gridcoin-community#2758 (@jamescowens)

Removed

Fixed
 - build, depends: fix compilation with XCode 15 gridcoin-community#2747 (@div72)
 - Fix man page installation path for cmake builds gridcoin-community#2749 (@theMarix)
 - consensus, mrc, sidestake: add mrc fees to staker to rewards to be allocated via sidestaking gridcoin-community#2753 (@jamescowens)
 - Fix Systemd unit install location gridcoin-community#2754 (@theMarix)
 - scraper: Corrections to scraper_net after removal of cntPartsRcvd decrement and increment gridcoin-community#2755 (@jamescowens)
 - rpc: fix setban segfault gridcoin-community#2757 (@div72)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants