|Summary:||[PATCH] devel/rbtools: Make the port work with BSD diff|
|Product:||Ports & Packages||Reporter:||Raphael Kubo da Costa <rakuco>|
|Component:||Individual Port(s)||Assignee:||Raphael Kubo da Costa <rakuco>|
|Status:||Closed Overcome By Events|
|Severity:||Affects Only Me||CC:||avg, julian, rene|
Description Raphael Kubo da Costa 2018-04-11 17:18:01 UTC
I've received individual emails over the past few weeks concerning devel/rbtools, and how it does not work on HEAD because diff(1) there is BSD diff instead of GNU diff (per base r317209). BSD diff does not support --version, which causes rbtools' verification code to fail. While it's possible to make the verification code use gdiff instead, the rest of the code base does not seem to respect that and either hardcode diff as the command name or not seem to use diff directly at all. I'm tempted to just disable the verification code altogether, as BSD diff should be compatible with GNU diff in most aspects as far as I know.
Comment 1 Andriy Gapon 2018-04-11 17:19:44 UTC
That's fine by me. Thank you!
Comment 2 Raphael Kubo da Costa 2018-04-11 17:22:16 UTC
Created attachment 192436 [details] Proposed patch I'd appreciate it if someone could see if the attached patch works in production too. I've run rbtools' test suite and everything worked as expected with BSD diff and this patch.
Comment 3 Rene Ladan 2020-12-31 17:47:16 UTC
Stale PR, and this port is py27-only, so closing.