Paths

Table of Contentst

Use jdiff instead of jsondiff
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F24713288: D3978.id.diff
	Fri, Mar 6, 21:50

	F24712151: D3978.id10336.diff
	Fri, Mar 6, 19:59

	Unknown Object (File)
	Fri, Mar 6, 03:40

	Unknown Object (File)
	Fri, Mar 6, 01:50

	Unknown Object (File)
	Thu, Mar 5, 19:20

	Unknown Object (File)
	Tue, Mar 3, 20:37

	Unknown Object (File)
	Tue, Mar 3, 10:35

	Unknown Object (File)
	Tue, Mar 3, 10:35

View All 23 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.Thu, Feb 26, 00:17

dereckson created this revision.

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

Caught this issue during dependencies test.

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

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

Closed by commit rOPS48e86fa47a4d: Use jdiff instead of jsondiff (authored by dereckson, committed by yousra). · Explain WhyMon, Mar 2, 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