Login | Register
My pages Projects Community openCollabNet

Discussions > users > Re: [Subclipse-users] Ignore Whitespace During *Merge*

subclipse
Discussion topic

Back to topic list

Re: [Subclipse-users] Ignore Whitespace During *Merge*

Author jamiejackson
Full name Jamie Jackson
Date 2016-06-22 08:57:15 PDT
Message Subclipse ticket: http://subclipse.tig​ris.org/issues/show_​bug.cgi?id=1655
Subversion (JavaHL) upstream ticket:
https://issues.apach​e.org/jira/browse/SV​N-4638

On Fri, Apr 22, 2016 at 12:03 PM, Jamie Jackson <jamiejaxon at gmail dot com>
wrote:

> Okay, thanks for the information, Mark. I sent an enhancement request to
> the Subversion users list (post pending, since I hadn't been subscribed).
>
> On Fri, Apr 22, 2016 at 11:17 AM, Mark Phippard <markphip at gmail dot com>
> wrote:
>
>> The main thing would be to look at ISVNClient as that exposes the methods
>> we use:
>>
>>
>> http://subversion.ap​ache.org/docs/javahl​/1.9/org/apache/subv​ersion/javahl/ISVNCl​ient.html
>>
>> So in this case, you want Merge to ignore whitespace. When you look at
>> the merge methods exposed, none of those seem to support passing flags that
>> tell merge to ignoreWhitespace.
>>
>> The diff method has been enhanced (seemingly in the 1.8 release) to
>> accept this new DiffOptions class with these flags, but I do not see that
>> merge has the same. That is what we would need to be able to expose UI
>> options for the same.
>>
>> Mark
>>
>>
>>
>> On Fri, Apr 22, 2016 at 10:56 AM, Jamie Jackson <jamiejaxon at gmail dot com>
>> wrote:
>>
>>> I can't claim to know squat about JavaHL, but is this the option?
>>> https://subversion.a​pache.org/docs/javah​l/latest/org/apache/​subversion/javahl/SV​NUtil.DiffOptions.Ig​noreSpace.html#chang​e
>>>
>>> If there's an upstream enhancement to be filed (to the Subversion/JavaHL
>>> team), I don't mind doing it, but I'll need to be able to write an
>>> intelligible ticket.
>>>
>>> Thanks,
>>> Jamie
>>>
>>> On Thu, Apr 21, 2016 at 3:03 PM, Mark Phippard <markphip at gmail dot com>
>>> wrote:
>>>
>>>> Those options are not available. I think things might have changed in
>>>> recent SVN releases, but the last time we touched this code the JavaHL API
>>>> did not expose any of these options to us in the API so we were not able to
>>>> expose them in our UI. As a C++ client, TortoiseSVN has access to the full
>>>> Subversion API.
>>>>
>>>> Mark
>>>>
>>>> On Thu, Apr 21, 2016 at 2:36 PM, Jamie Jackson <jamiejaxon at gmail dot com>
>>>> wrote:
>>>>
>>>>> This topic came up before (
>>>>> http://subclipse.tig​ris.org/ds/viewMessa​ge.do?dsForumId=1047​&dsMessageId=907​463
>>>>> ), but I'm not sure it was really resolved.
>>>>>
>>>>> In TortoiseSVN, there are a few whitespace-related options in the
>>>>> merge dialogs:
>>>>>
>>>>> - Ignore line endings
>>>>> - Compare whitespace
>>>>> - Ignore whitespace changes
>>>>> - Ignore all whitespaces
>>>>>
>>>>> (The last three are radios, BTW--mutually exclusive.)
>>>>>
>>>>> I routinely use the "ignore whitespace changes," and that affects
>>>>> behavior at *merge* time. In other words, that tends to create fewer
>>>>> conflicts when there are wholesale indentation changes going on.
>>>>>
>>>>> The option "Ignore whitespace" in Eclipse don't seem to behave the
>>>>> same way; rather, it seems to affect what is *displayed* in the
>>>>> conflict resolution diff, rather than affecting the behavior of the
>>>>> automated merge, itself.
>>>>>
>>>>> After all these years, I still need TortoiseSVN in my toolbox because
>>>>> of this feature; however, I'd love to drop it.
>>>>>
>>>>> Please let me know if I've missed some option to emulate the *merge* (not
>>>>> the diff viewing) behavior in Subclipse. Otherwise, please treat this as an
>>>>> enhancement request.
>>>>>
>>>>> Thanks,
>>>>> Jamie
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Thanks
>>>>
>>>> Mark Phippard
>>>> http://markphip.blogspot.com/
>>>>
>>>
>>>
>>
>>
>> --
>> Thanks
>>
>> Mark Phippard
>> http://markphip.blogspot.com/
>>
>
>
Attachments

« Previous message in topic | 6 of 6 | Next message in topic »

Messages

Show all messages in topic

[Subclipse-users] Ignore Whitespace During *Merge* jamiejackson Jamie Jackson 2016-04-21 11:37:00 PDT
     Re: [Subclipse-users] Ignore Whitespace During *Merge* markphip Mark Phippard 2016-04-21 12:03:26 PDT
         Re: [Subclipse-users] Ignore Whitespace During *Merge* jamiejackson Jamie Jackson 2016-04-22 07:56:10 PDT
             Re: [Subclipse-users] Ignore Whitespace During *Merge* markphip Mark Phippard 2016-04-22 08:17:57 PDT
                 Re: [Subclipse-users] Ignore Whitespace During *Merge* jamiejackson Jamie Jackson 2016-04-22 09:03:15 PDT
                     Re: [Subclipse-users] Ignore Whitespace During *Merge* jamiejackson Jamie Jackson 2016-06-22 08:57:15 PDT
Messages per page: