[Wishlist] Don't set known merge tools

Bug #1045849 reported by IWATA Hidetaka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
QBzr
New
Undecided
Unassigned

Bug Description

Now, known merge tools are automatically added to list, even if the tools is not installed.
And there is no way to configure them.

I want to change this behavior like below:
 * Don't add known merge tools automatically.
 * Provide a way to add known merge tools easily.
 * After added, allow to edit or remove it.

Revision history for this message
IWATA Hidetaka (hid-iwata) wrote :
Revision history for this message
Gordon Tyler (doxxx) wrote :

The current design is to allow external merge tools to work out of the box without user configuration if they're on the command-line. Only merge tools which can be found are actually shown in the qconflicts tool.

Also bear in mind that this has to work without qbzr installed. The mergetools feature is part of bzr core itself.

Revision history for this message
IWATA Hidetaka (hid-iwata) wrote : Re: [Bug 1045849] Re: [Wishlist] Don't set known merge tools

I think current design has problems.

At least, tools which are not found in system must be hidden.
This is qbzr's task.

and if possible, known tools should be configurable or overridable.
I don't want to add paths of all tools to environment variable, and I want
to specify extra args to some tools.

2012/09/05 5:25 "Gordon Tyler" <email address hidden>:
>
> The current design is to allow external merge tools to work out of the
> box without user configuration if they're on the command-line. Only
> merge tools which can be found are actually shown in the qconflicts
> tool.
>
> Also bear in mind that this has to work without qbzr installed. The
> mergetools feature is part of bzr core itself.
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1045849
>
> Title:
> [Wishlist] Don't set known merge tools
>
> Status in Qt frontend for Bazaar:
> New
>
> Bug description:
> Now, known merge tools are automatically added to list, even if the
tools is not installed.
> And there is no way to configure them.
>
> I want to change this behavior like below:
> * Don't add known merge tools automatically.
> * Provide a way to add known merge tools easily.
> * After added, allow to edit or remove it.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/qbzr/+bug/1045849/+subscriptions
 2012/09/05 5:25 "Gordon Tyler" <email address hidden>:

> The current design is to allow external merge tools to work out of the
> box without user configuration if they're on the command-line. Only
> merge tools which can be found are actually shown in the qconflicts
> tool.
>
> Also bear in mind that this has to work without qbzr installed. The
> mergetools feature is part of bzr core itself.
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1045849
>
> Title:
> [Wishlist] Don't set known merge tools
>
> Status in Qt frontend for Bazaar:
> New
>
> Bug description:
> Now, known merge tools are automatically added to list, even if the
> tools is not installed.
> And there is no way to configure them.
>
> I want to change this behavior like below:
> * Don't add known merge tools automatically.
> * Provide a way to add known merge tools easily.
> * After added, allow to edit or remove it.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/qbzr/+bug/1045849/+subscriptions
>

Revision history for this message
Gordon Tyler (doxxx) wrote :

The intended method for using an alternate command-line for a known merge tool is to register a new merge tool name ("winmerge-funkyoptions") with the modified command-line. To make the known merge tool commandlines editable would require also adding the ability to reset it back to defaults and allowing a merge tool definition in the config to override a known merge tool with the same name.

I am uncomfortable not showing the known merge tools in qconfig since they are actually defined and will be shown in the qconflicts dropdown if found on the PATH. What might be useful is to add another column to the table which shows a checkmark if the merge tool can be found by bzr.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.