Skip to content

cleanup: remove imediff2 legacy; add imediff invocation tests - #58

Merged
rich0 merged 2 commits into
developfrom
cleanup/imediff-legacy
Jun 20, 2026
Merged

cleanup: remove imediff2 legacy; add imediff invocation tests#58
rich0 merged 2 commits into
developfrom
cleanup/imediff-legacy

Conversation

@rich0

@rich0 rich0 commented Jun 20, 2026

Copy link
Copy Markdown
Owner

Fixes #57

Summary

Verifies and locks in modern imediff command invocations for stages 3 and 4. Upstream osamuaoki/imediff confirms the existing launch_tool() strings are correct (-a -o with live/ancestor/new file order). No invocation changes.

Removes legacy imediff2 support and aligns docs to dev-util/imediff 3.4.x+.

Changes

  • Add install_mock_imediff() and Tier D tests for stage 3 (3-way) and stage 4 (2-way)
  • Remove imediff2 from check_tool regexes, --help, and launch_tool 2-way match
  • Update cfg-update.conf, DEPENDENCIES.md, ARCHITECTURE.md, INVENTORY.md, ebuild

Testing

perl -c cfg-update
./test/run-tests.sh --full

196 passed, 0 failed.

Drop obsolete imediff2 detection and documentation. Modern dev-util/imediff
supports both 2-way and 3-way merges; existing -a -o launch_tool invocations
are unchanged.

Add Tier D mock imediff tests for stage 3 (3-way) and stage 4 (2-way) to
lock in command-line arguments and merge outcomes.

Fixes #57
@rich0
rich0 force-pushed the cleanup/imediff-legacy branch from 22e60a5 to be26cfd Compare June 20, 2026 14:14
Resolve conflicts with gtkdiff removal (PR #59): keep imediff-only
support and no gtkdiff in tool_saves_mergefile_when_aborted or --help.
@rich0
rich0 merged commit aa393e3 into develop Jun 20, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Refactor/cleanup cfg-update Jun 20, 2026
@rich0
rich0 deleted the cleanup/imediff-legacy branch August 1, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant