Paths

Table of Contentst

Use jdiff instead of jsondiff
ClosedPublic
Actions

Authored by dereckson on Feb 26 2026, 00:17.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Jul 27, 01:36

	Unknown Object (File)
	Sun, Jul 26, 23:09

	Unknown Object (File)
	Thu, Jul 23, 23:50

	Unknown Object (File)
	Tue, Jul 21, 11:04

	Unknown Object (File)
	Mon, Jul 20, 16:59

	Unknown Object (File)
	Sun, Jul 19, 15:24

	Unknown Object (File)
	Sun, Jul 19, 12:29

	Unknown Object (File)
	Sun, Jul 19, 12:26

View All Files

Subscribers

None

Details

Reviewers

yousra

Commits

rOPS48e86fa47a4d: Use jdiff instead of jsondiff

Summary

jdiff seems the new name of jsondiff executable.

This fixes the following issue:

$ make test
[...]
run`'s command `jsondiff /tmp/tmp.CXo1sQNPui data/acmedns-merged.json`
exited with code 127, indicating 'Command not found'.

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Feb 26 2026, 00:17

dereckson created this revision.

Harbormaster completed remote builds in B6371: Diff 10320.Feb 26 2026, 00:17

Caught this issue during dependencies test.

yousra accepted this revision.Mar 2 2026, 11:45

This revision is now accepted and ready to land.Mar 2 2026, 11:45

Closed by commit rOPS48e86fa47a4d: Use jdiff instead of jsondiff (authored by dereckson, committed by yousra). · Explain WhyMar 2 2026, 11:46

This revision was automatically updated to reflect the committed changes.

yousra added a commit: rOPS48e86fa47a4d: Use jdiff instead of jsondiff.

Revision Contents
Changeset List

Path

Size

_tests/

scripts/

bats/

test_edit_acme_dns_accounts.sh

2 lines

Diff 10336

View Options

_tests/scripts/bats/test_edit_acme_dns_accounts.sh

Use jdiff instead of jsondiffClosedPublicActions