threads / patch / 48225

patchmergetools: add support for guiffy

Subject: [PATCH] mergetools: add support for guiffy

## tl;dr

2 messages between Apr 5, 2018 and Apr 5, 2018. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Bill Ritcher· Apr 5, 2018, 13:55 UTC · lore
Add guiffy as difftool and mergetool
guiffy is available on Windows, Linux, and MacOS
Signed-off-by: Bill Ritcher <Bill_Ritcher@guiffy.com>
---
 mergetools/guiffy | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 mergetools/guiffy
Show changes to mergetools/guiffy +18 −0
diff --git a/mergetools/guiffy b/mergetools/guiffy
new file mode 100644
index 000000000..8b23a13c4
--- /dev/null
+++ b/mergetools/guiffy
@@ -0,0 +1,18 @@
+diff_cmd () {
+	"$merge_tool_path" "$LOCAL" "$REMOTE"
+}
+
+merge_cmd () {
+	if $base_present
+	then
+		"$merge_tool_path" -s "$LOCAL" \
+		"$REMOTE" "$BASE" "$MERGED"
+	else
+		"$merge_tool_path" -m "$LOCAL" \
+		"$REMOTE" "$MERGED"
+	fi
+}
+
+exit_code_trustable () {
+	true
+}
-- 
2.15.1.windows.2
David Aguilar· Apr 5, 2018, 17:07 UTC · re: Bill Ritcher · lore

Re: [PATCH] mergetools: add support for guiffy

On Thu, Apr 05, 2018 at 08:55:01AM -0500, Bill Ritcher wrote:
Show 36 quoted lines
> Add guiffy as difftool and mergetool
> 
> guiffy is available on Windows, Linux, and MacOS
> 
> Signed-off-by: Bill Ritcher <Bill_Ritcher@guiffy.com>
> ---
>  mergetools/guiffy | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>  create mode 100644 mergetools/guiffy
> 
> diff --git a/mergetools/guiffy b/mergetools/guiffy
> new file mode 100644
> index 000000000..8b23a13c4
> --- /dev/null
> +++ b/mergetools/guiffy
> @@ -0,0 +1,18 @@
> +diff_cmd () {
> +	"$merge_tool_path" "$LOCAL" "$REMOTE"
> +}
> +
> +merge_cmd () {
> +	if $base_present
> +	then
> +		"$merge_tool_path" -s "$LOCAL" \
> +		"$REMOTE" "$BASE" "$MERGED"
> +	else
> +		"$merge_tool_path" -m "$LOCAL" \
> +		"$REMOTE" "$MERGED"
> +	fi
> +}
> +
> +exit_code_trustable () {
> +	true
> +}
> -- 
> 2.15.1.windows.2
I tested this on Linux and it works great.  Thanks Bill.
Acked-by: David Aguilar <davvid@gmail.com>
cheers,
-- 
David

← back to recent threads