Skip to content

Conversation

@theMarix
Copy link
Contributor

The Systemd unit file should be installed relative to the install prefix, not at an absolute path.

The Systemd unit file should be installed relative to the install
prefix, not at an absolute path.
Copy link
Member

@div72 div72 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested ACK bc30bd8.

@jamescowens jamescowens added this to the Natasha milestone Mar 24, 2024
@jamescowens jamescowens merged commit 8b05385 into gridcoin-community:development Mar 24, 2024
@jamescowens jamescowens changed the title Fix Systemd unit install location build: Fix Systemd unit install location 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