Paths

Table of Contentst

Use jdiff instead of jsondiff
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F25302329: D3978.id10336.diff
	Thu, Apr 9, 05:26

	Unknown Object (File)
	Mon, Mar 30, 09:09

	Unknown Object (File)
	Sat, Mar 28, 09:28

	Unknown Object (File)
	Thu, Mar 26, 10:49

	Unknown Object (File)
	Tue, Mar 24, 18:04

	Unknown Object (File)
	Tue, Mar 24, 16:22

	Unknown Object (File)
	Tue, Mar 24, 13:16

	Unknown Object (File)
	Tue, Mar 24, 13:16

View All 58 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