Bug 227447

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: New ---    
Severity: Affects Only Me CC: avg, julian
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Proposed patch none

Description Raphael Kubo da Costa freebsd_committer 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 freebsd_committer 2018-04-11 17:19:44 UTC
That's fine by me.
Thank you!
Comment 2 Raphael Kubo da Costa freebsd_committer 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.