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 markphip
Full name Mark Phippard
Date 2016-04-22 08:17:57 PDT
Message 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 | 4 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: