Sponsored Content
Skip to content

refactor: Rename system name from 'ripple' to 'xrpld' - #6347

Merged
bthomee merged 10 commits into
developfrom
bthomee/xrpld
Mar 16, 2026
Merged

refactor: Rename system name from 'ripple' to 'xrpld'#6347
bthomee merged 10 commits into
developfrom
bthomee/xrpld

Conversation

@bthomee

@bthomee bthomee commented Feb 10, 2026

Copy link
Copy Markdown
Collaborator

High Level Overview of Change

This change modifies the system name from rippled to xrpld.

Context of Change

Per XLS-0095, we are taking steps to rename ripple(d) to xrpl(d).

This change renames the system name, which is used in limited places:

  • When no explicit config file is passed via the --config flag, then the system name is used to construct the path where the config file and database may be stored, via the $XDG_CONFIG_HOME and $XDG_DATA_HOME directories, respectively.
  • It is used in the metadata and user-agent as part of RPC calls.
  • It is newly used in the full version string.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Performance (increase or change in throughput and/or latency)
  • Tests (you added tests for code that already exists, or your new feature included in this PR)
  • Documentation update
  • Chore (no impact to binary, e.g. .gitignore, formatting, dropping support for older tooling)
  • Release

Copilot AI review requested due to automatic review settings February 10, 2026 19:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the core “system name” constant used across xrpld for default config/data directory selection (XDG + /etc/opt//var/opt) and for RPC metadata/user-agent strings, aligning with the ongoing rename effort in XLS-0095.

Changes:

  • Update systemName() from "ripple" to "xrpld".

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread include/xrpl/protocol/SystemParameters.h

@godexsoft godexsoft left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not an issue for Clio 👍
May be an issue for someone else but i very much doubt.

@codecov

codecov Bot commented Feb 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@47a235b). Learn more about missing BASE report.
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
src/xrpld/app/main/Main.cpp 0.0% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##             develop   #6347   +/-   ##
=========================================
  Coverage           ?   79.8%           
=========================================
  Files              ?     878           
  Lines              ?   67894           
  Branches           ?    7555           
=========================================
  Hits               ?   54201           
  Misses             ?   13693           
  Partials           ?       0           
Files with missing lines Coverage Δ
include/xrpl/protocol/SystemParameters.h 100.0% <100.0%> (ø)
src/libxrpl/protocol/BuildInfo.cpp 98.4% <100.0%> (ø)
src/xrpld/rpc/detail/ServerHandler.cpp 87.8% <100.0%> (ø)
src/xrpld/app/main/Main.cpp 77.8% <0.0%> (ø)

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bthomee
bthomee requested a review from ximinez February 11, 2026 00:07

@legleux legleux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just the Copilot suggestion so we don't get xrpldd in certain places.

Comment thread include/xrpl/protocol/SystemParameters.h
@bthomee
bthomee requested a review from legleux March 6, 2026 08:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/xrpld/rpc/detail/ServerHandler.cpp Outdated
@bthomee
bthomee enabled auto-merge (squash) March 16, 2026 21:21
@bthomee
bthomee merged commit ffea397 into develop Mar 16, 2026
1 check passed
@bthomee
bthomee deleted the bthomee/xrpld branch March 16, 2026 21:51
@mvadari mvadari added this to the 3.2.0 milestone May 20, 2026
beartec-jpg pushed a commit to beartec-jpg/FalconLedger that referenced this pull request Jun 1, 2026
Per [XLS-0095](https://xls.xrpl.org/xls/XLS-0095-rename-rippled-to-xrpld.html), we are taking steps to rename ripple(d) to xrpl(d). This change modifies the system name from `rippled` to `xrpld`.

The system name is used in limited places:
* When no explicit config file is passed via the `--config` flag, then the system name is used to construct the path where the config file and database may be stored, via the `$XDG_CONFIG_HOME` and `$XDG_DATA_HOME` directories, respectively.
* It is used in the metadata and user-agent as part of RPC calls.
* It is newly used in the full version string.
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.

5 participants