Groovy and semicolon at EOL

[Rishi Solanki]

Jacques,

Yes we would like to commit it as whole, but before commit for the same we
have plan to test each component after applying the changes. Like browse to
most pages and general work flows. We will post the updates on ticket
something like;

Test the party component;
Pages/Work Flow tested: Find Party, Create Party, View Party, My
Communications, Visits, Classification, Security, Invitation pages etc.

The above is an example of how we will confirm everything is working
properly, with some basic code review. We would follow the same steps for
other components.

Please let us know plan looks fine to you. Also in case you think we should
take care anything else to minimize the possibility of regression Or may be
if you think committing the changes per component will help in code review?

Thanks!



Rishi Solanki
Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: [phone]
http://www.hotwaxsystems.com

On Fri, Oct 28, 2016 at 4:10 PM, Jacques Le Roux <
[hidden email]> wrote:

> Hi Rishi,
>
> Will you commit as a whole?
>
> Jacques
>
>
>
> Le 28/10/2016 à 12:07, Rishi Solanki a écrit :
>
>> Started effort under - https://issues.apache.org/jira/browse/OFBIZ-8652
>>
>> Thanks to Rohit Kaushal for taking care of this. It will take 4-5 days for
>> testing.
>>
>>
>> Thanks!
>>
>> Rishi Solanki
>> Manager, Enterprise Software Development
>> HotWax Systems Pvt. Ltd.
>> Direct: [phone]
>> http://www.hotwaxsystems.com
>>
>> On Sat, Sep 17, 2016 at 2:11 AM, Jacques Le Roux <
>> [hidden email]> wrote:
>>
>> Personally I will go this way: I will add or changes lines without putting
>>> semicolons.
>>>
>>> I'm in favour of bulk changing files, but I'd prefer by component or
>>> webapp to ease reviews.
>>>
>>> Jacques
>>>
>>>
>>> Le 16/09/2016 à 15:36, Rishi Solanki a écrit :
>>>
>>> I was saying #2 as per the comment from Taher ....
>>>>
>>>> Quick Reference:
>>>>
>>>> One reply from Taher ... in the same thread.
>>>> ==========
>>>>
>>>> Okay, given the priorities and work we have at the moment, I suggest we
>>>> keep semicolons and use it as the standard unless someone volunteers to
>>>> make a full switch. WDYT?
>>>> ==========
>>>>
>>>>
>>>>
>>>> Rishi Solanki
>>>> Manager, Enterprise Software Development
>>>> HotWax Systems Pvt. Ltd.
>>>> Direct: [phone]
>>>> http://www.hotwaxsystems.com
>>>>
>>>> On Fri, Sep 16, 2016 at 3:14 PM, Jacques Le Roux <
>>>> [hidden email]> wrote:
>>>>
>>>> I guess you mean 2) by file, then it's OK with me. Though I'd no be
>>>>
>>>>> against having semicolon inconsistency in Groovy files, which was my
>>>>> initial question. So no strong opinion about 2 here.
>>>>>
>>>>> Jacques
>>>>>
>>>>>
>>>>>
>>>>> Le 16/09/2016 à 11:31, Rishi Solanki a écrit :
>>>>>
>>>>> To summarize the overall conversation;
>>>>>
>>>>>> 1) We have decided to bulk remove semicolons from groovy.
>>>>>> 2) Until #1 is not complete, we would keep adding semicolon for
>>>>>> consistency.
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> Rishi Solanki
>>>>>> Manager, Enterprise Software Development
>>>>>> HotWax Systems Pvt. Ltd.
>>>>>> Direct: [phone]
>>>>>> http://www.hotwaxsystems.com
>>>>>>
>>>>>> On Thu, Sep 15, 2016 at 10:00 AM, Jacques Le Roux <
>>>>>> [hidden email]> wrote:
>>>>>>
>>>>>> Actually I was wrong on this. Thanks to Jacopo I noticed that both
>>>>>>
>>>>>> Subclipse and Tortoise allow you to select a range of revisions when
>>>>>>> you
>>>>>>> look for annotations.
>>>>>>>
>>>>>>> So  it's no longer an issue for me and we can bulk remove trailing
>>>>>>> semicolons in Groovy files if we want.
>>>>>>>
>>>>>>> Sorry for the confusion
>>>>>>>
>>>>>>> Jacques
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Le 14/09/2016 à 04:42, Scott Gray a écrit :
>>>>>>>
>>>>>>> I don't particularly care one way or another if groovy files have a
>>>>>>>
>>>>>>> semi-colon at the end.  I don't even care about consistency because
>>>>>>>> it
>>>>>>>> is
>>>>>>>> such a minor thing.
>>>>>>>>
>>>>>>>> I say remove them if they're on a line you happen to be editing,
>>>>>>>> otherwise
>>>>>>>> just leave them be.
>>>>>>>>
>>>>>>>> Regarding the annotations, there's plenty of ways to search commit
>>>>>>>> logs
>>>>>>>> and
>>>>>>>> personally I've never found blame to be very useful.  I don't think
>>>>>>>> it
>>>>>>>> should be a reason to block any future bulk S/R cleanups.  We've had
>>>>>>>> plenty
>>>>>>>> in the past (Double -> BigDecimal, Delegator -> EntityQuery,
>>>>>>>> whitespace
>>>>>>>> removal, etc.) and we should continue to do it to keep things clean.
>>>>>>>>
>>>>>>>> For searching diffs, before using git-svn I used to use: svn log
>>>>>>>> -diff
>>>>>>>> <path/to.file> and then use the search in the terminal to find the
>>>>>>>> string
>>>>>>>> I'm looking for.
>>>>>>>>
>>>>>>>> Regards
>>>>>>>> Scott
>>>>>>>>
>>>>>>>> On 14 September 2016 at 07:33, Jacques Le Roux <
>>>>>>>> [hidden email]
>>>>>>>>
>>>>>>>> wrote:
>>>>>>>>
>>>>>>>> Le 13/09/2016 à 21:28, Jacques Le Roux a écrit :
>>>>>>>>>
>>>>>>>>> OK found that the same than in Subclipse also exists in TortoiseSVN
>>>>>>>>>
>>>>>>>>> But you need to use a command line (weird for a GUI), eg (from
>>>>>>>>>
>>>>>>>>>> TortoiseSVN root folder)
>>>>>>>>>>
>>>>>>>>>> Actually wrong, simply pick a file in Windows file explorer using
>>>>>>>>>>
>>>>>>>>>> TortoiseSVN  context menu, et voilà!
>>>>>>>>>>
>>>>>>>>> I confirm, totally comparable to Subclipse annotations
>>>>>>>>>
>>>>>>>>> Jacques
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> TortoiseProc.exe /command:blame /path:"C:\projectASF-Mars\ofbi
>>>>>>>>>
>>>>>>>>> z\applications\product\src\main\java\org\apache\ofbiz\
>>>>>>>>>
>>>>>>>>>> product\catalog\CatalogWorker.java"
>>>>>>>>>>
>>>>>>>>>> All is explained here https://tortoisesvn.net/docs/r
>>>>>>>>>> elease/TortoiseSVN_en/tsvn-automation.html#tsvn-automation-basics
>>>>>>>>>>
>>>>>>>>>>     From the resulting UI (comparable to Subclipse) I guess
>>>>>>>>>> changing
>>>>>>>>>> all
>>>>>>>>>> lines of a file will have the same effect.
>>>>>>>>>> Even if indeed the annotations are not lost, they are very hard to
>>>>>>>>>> use
>>>>>>>>>> if
>>>>>>>>>> you need to compare revision by revision.
>>>>>>>>>>
>>>>>>>>>> Jacques
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Le 13/09/2016 à 20:21, Jacques Le Roux a écrit :
>>>>>>>>>>
>>>>>>>>>> BTW thinking about it, don't you have something similar in
>>>>>>>>>> IntellIJ?
>>>>>>>>>>
>>>>>>>>>> I found an (old) image there https://markphip.blogspot.fr/2
>>>>>>>>>>
>>>>>>>>>>> 006/12/subclipse-live-annotations.html
>>>>>>>>>>>
>>>>>>>>>>> Jacques
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Le 13/09/2016 à 20:16, Jacques Le Roux a écrit :
>>>>>>>>>>>
>>>>>>>>>>> Thanks Jacopo,
>>>>>>>>>>>
>>>>>>>>>>> I found how to use it in TortoiseSVN (it starts from the log
>>>>>>>>>>> view)
>>>>>>>>>>>
>>>>>>>>>>>> It's complementary to what Subclipse gives and so interesting
>>>>>>>>>>>> but
>>>>>>>>>>>> not
>>>>>>>>>>>> comparable.
>>>>>>>>>>>>
>>>>>>>>>>>> You don't have this global view Subclipse offers with each
>>>>>>>>>>>> annotation
>>>>>>>>>>>> by line from start (r1) to HEAD.
>>>>>>>>>>>> Very useful with colored annotations in the same column than
>>>>>>>>>>>> lines
>>>>>>>>>>>> numbers. But it unfortunately contains only the last revision if
>>>>>>>>>>>> all
>>>>>>>>>>>> lines
>>>>>>>>>>>> have been modified together in that revision.
>>>>>>>>>>>> Note: to see it you need to use "Show Quick Diff" ("Revision"
>>>>>>>>>>>> and
>>>>>>>>>>>> "Combined Colors" are then default options, hovering is enough
>>>>>>>>>>>> for
>>>>>>>>>>>> me).
>>>>>>>>>>>> Same than you decide to show line numbers in this column... More
>>>>>>>>>>>> for
>>>>>>>>>>>> those who are still using Eclipse...
>>>>>>>>>>>>
>>>>>>>>>>>> Jacques
>>>>>>>>>>>>
>>>>>>>>>>>> Le 13/09/2016 à 17:40, Jacopo Cappellato a écrit :
>>>>>>>>>>>>
>>>>>>>>>>>> Some examples:
>>>>>>>>>>>>
>>>>>>>>>>>> svn blame README.md
>>>>>>>>>>>>
>>>>>>>>>>>>> after review you run
>>>>>>>>>>>>>
>>>>>>>>>>>>> svn blame README.md -r 1:1757044
>>>>>>>>>>>>>
>>>>>>>>>>>>> and then
>>>>>>>>>>>>>
>>>>>>>>>>>>> svn blame README.md -r 1:1757042
>>>>>>>>>>>>>
>>>>>>>>>>>>> and so on to get back in history... nothing is lost,
>>>>>>>>>>>>> annotations
>>>>>>>>>>>>> are
>>>>>>>>>>>>> always
>>>>>>>>>>>>> there.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>
>>>>>>>>>>>>> PS: I think there is some trick to do the same with TortoiseSVN
>>>>>>>>>>>>> but I
>>>>>>>>>>>>> can't
>>>>>>>>>>>>> tell you the details since I don't use it
>>>>>>>>>>>>>
>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 5:16 PM, Jacques Le Roux <
>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>
>>>>>>>>>>>>> Le 13/09/2016 à 16:45, Jacopo Cappellato a écrit :
>>>>>>>>>>>>>
>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 4:36 PM, Jacques Le Roux <
>>>>>>>>>>>>>
>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Before applying a such change, I'd really like to know if
>>>>>>>>>>>>>>> everybody
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> is
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> aware of what that means when it comes to svn annotations. I
>>>>>>>>>>>>>>>> repeat: we
>>>>>>>>>>>>>>>> will then lose all the svn annotations history in all the
>>>>>>>>>>>>>>>> Groovy
>>>>>>>>>>>>>>>> files.
>>>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Jacques, are you aware that you can pass the -r argument to
>>>>>>>>>>>>>>>> the
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> blame/annotate command?
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> I must say I never use that when looking at annotations in a
>>>>>>>>>>>>>>> file
>>>>>>>>>>>>>>> in
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Eclipse. It's maybe useful in certain circumstances, but I
>>>>>>>>>>>>>>> hardly
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> see
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>> when.
>>>>>>>>>>>>>> And once all the lines a file has been modified in one
>>>>>>>>>>>>>> commit, I
>>>>>>>>>>>>>> guess -r
>>>>>>>>>>>>>> does not help at all, anyway you get only this information. Or
>>>>>>>>>>>>>> do
>>>>>>>>>>>>>> I
>>>>>>>>>>>>>> miss
>>>>>>>>>>>>>> something? Should I know the revision I'm looking for? I
>>>>>>>>>>>>>> rather
>>>>>>>>>>>>>> try
>>>>>>>>>>>>>> to know
>>>>>>>>>>>>>> when and why a line has been changed, what are the reasons of
>>>>>>>>>>>>>> these
>>>>>>>>>>>>>> changes, maybe to find an related Jira, etc.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>

[Jacques Le Roux]

Hi Rishi,

It's not the first time we change a *simple thing* in all the source. I can live with that, you seem well organised :)

BTW after appling the patch at OFBIZ-8652 I still find 57 trailing semicolons :)

Thanks

Jacques


Le 01/11/2016 à 07:35, Rishi Solanki a écrit :

> Jacques,
>
> Yes we would like to commit it as whole, but before commit for the same we
> have plan to test each component after applying the changes. Like browse to
> most pages and general work flows. We will post the updates on ticket
> something like;
>
> Test the party component;
> Pages/Work Flow tested: Find Party, Create Party, View Party, My
> Communications, Visits, Classification, Security, Invitation pages etc.
>
> The above is an example of how we will confirm everything is working
> properly, with some basic code review. We would follow the same steps for
> other components.
>
> Please let us know plan looks fine to you. Also in case you think we should
> take care anything else to minimize the possibility of regression Or may be
> if you think committing the changes per component will help in code review?
>
> Thanks!
>
>
>
> Rishi Solanki
> Manager, Enterprise Software Development
> HotWax Systems Pvt. Ltd.
> Direct: [phone]
> http://www.hotwaxsystems.com
>
> On Fri, Oct 28, 2016 at 4:10 PM, Jacques Le Roux <
> [hidden email]> wrote:
>
>> Hi Rishi,
>>
>> Will you commit as a whole?
>>
>> Jacques
>>
>>
>>
>> Le 28/10/2016 à 12:07, Rishi Solanki a écrit :
>>
>>> Started effort under - https://issues.apache.org/jira/browse/OFBIZ-8652
>>>
>>> Thanks to Rohit Kaushal for taking care of this. It will take 4-5 days for
>>> testing.
>>>
>>>
>>> Thanks!
>>>
>>> Rishi Solanki
>>> Manager, Enterprise Software Development
>>> HotWax Systems Pvt. Ltd.
>>> Direct: [phone]
>>> http://www.hotwaxsystems.com
>>>
>>> On Sat, Sep 17, 2016 at 2:11 AM, Jacques Le Roux <
>>> [hidden email]> wrote:
>>>
>>> Personally I will go this way: I will add or changes lines without putting
>>>> semicolons.
>>>>
>>>> I'm in favour of bulk changing files, but I'd prefer by component or
>>>> webapp to ease reviews.
>>>>
>>>> Jacques
>>>>
>>>>
>>>> Le 16/09/2016 à 15:36, Rishi Solanki a écrit :
>>>>
>>>> I was saying #2 as per the comment from Taher ....
>>>>> Quick Reference:
>>>>>
>>>>> One reply from Taher ... in the same thread.
>>>>> ==========
>>>>>
>>>>> Okay, given the priorities and work we have at the moment, I suggest we
>>>>> keep semicolons and use it as the standard unless someone volunteers to
>>>>> make a full switch. WDYT?
>>>>> ==========
>>>>>
>>>>>
>>>>>
>>>>> Rishi Solanki
>>>>> Manager, Enterprise Software Development
>>>>> HotWax Systems Pvt. Ltd.
>>>>> Direct: [phone]
>>>>> http://www.hotwaxsystems.com
>>>>>
>>>>> On Fri, Sep 16, 2016 at 3:14 PM, Jacques Le Roux <
>>>>> [hidden email]> wrote:
>>>>>
>>>>> I guess you mean 2) by file, then it's OK with me. Though I'd no be
>>>>>
>>>>>> against having semicolon inconsistency in Groovy files, which was my
>>>>>> initial question. So no strong opinion about 2 here.
>>>>>>
>>>>>> Jacques
>>>>>>
>>>>>>
>>>>>>
>>>>>> Le 16/09/2016 à 11:31, Rishi Solanki a écrit :
>>>>>>
>>>>>> To summarize the overall conversation;
>>>>>>
>>>>>>> 1) We have decided to bulk remove semicolons from groovy.
>>>>>>> 2) Until #1 is not complete, we would keep adding semicolon for
>>>>>>> consistency.
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Rishi Solanki
>>>>>>> Manager, Enterprise Software Development
>>>>>>> HotWax Systems Pvt. Ltd.
>>>>>>> Direct: [phone]
>>>>>>> http://www.hotwaxsystems.com
>>>>>>>
>>>>>>> On Thu, Sep 15, 2016 at 10:00 AM, Jacques Le Roux <
>>>>>>> [hidden email]> wrote:
>>>>>>>
>>>>>>> Actually I was wrong on this. Thanks to Jacopo I noticed that both
>>>>>>>
>>>>>>> Subclipse and Tortoise allow you to select a range of revisions when
>>>>>>>> you
>>>>>>>> look for annotations.
>>>>>>>>
>>>>>>>> So  it's no longer an issue for me and we can bulk remove trailing
>>>>>>>> semicolons in Groovy files if we want.
>>>>>>>>
>>>>>>>> Sorry for the confusion
>>>>>>>>
>>>>>>>> Jacques
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> Le 14/09/2016 à 04:42, Scott Gray a écrit :
>>>>>>>>
>>>>>>>> I don't particularly care one way or another if groovy files have a
>>>>>>>>
>>>>>>>> semi-colon at the end.  I don't even care about consistency because
>>>>>>>>> it
>>>>>>>>> is
>>>>>>>>> such a minor thing.
>>>>>>>>>
>>>>>>>>> I say remove them if they're on a line you happen to be editing,
>>>>>>>>> otherwise
>>>>>>>>> just leave them be.
>>>>>>>>>
>>>>>>>>> Regarding the annotations, there's plenty of ways to search commit
>>>>>>>>> logs
>>>>>>>>> and
>>>>>>>>> personally I've never found blame to be very useful.  I don't think
>>>>>>>>> it
>>>>>>>>> should be a reason to block any future bulk S/R cleanups.  We've had
>>>>>>>>> plenty
>>>>>>>>> in the past (Double -> BigDecimal, Delegator -> EntityQuery,
>>>>>>>>> whitespace
>>>>>>>>> removal, etc.) and we should continue to do it to keep things clean.
>>>>>>>>>
>>>>>>>>> For searching diffs, before using git-svn I used to use: svn log
>>>>>>>>> -diff
>>>>>>>>> <path/to.file> and then use the search in the terminal to find the
>>>>>>>>> string
>>>>>>>>> I'm looking for.
>>>>>>>>>
>>>>>>>>> Regards
>>>>>>>>> Scott
>>>>>>>>>
>>>>>>>>> On 14 September 2016 at 07:33, Jacques Le Roux <
>>>>>>>>> [hidden email]
>>>>>>>>>
>>>>>>>>> wrote:
>>>>>>>>>
>>>>>>>>> Le 13/09/2016 à 21:28, Jacques Le Roux a écrit :
>>>>>>>>>> OK found that the same than in Subclipse also exists in TortoiseSVN
>>>>>>>>>>
>>>>>>>>>> But you need to use a command line (weird for a GUI), eg (from
>>>>>>>>>>
>>>>>>>>>>> TortoiseSVN root folder)
>>>>>>>>>>>
>>>>>>>>>>> Actually wrong, simply pick a file in Windows file explorer using
>>>>>>>>>>>
>>>>>>>>>>> TortoiseSVN  context menu, et voilà!
>>>>>>>>>>>
>>>>>>>>>> I confirm, totally comparable to Subclipse annotations
>>>>>>>>>>
>>>>>>>>>> Jacques
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> TortoiseProc.exe /command:blame /path:"C:\projectASF-Mars\ofbi
>>>>>>>>>>
>>>>>>>>>> z\applications\product\src\main\java\org\apache\ofbiz\
>>>>>>>>>>
>>>>>>>>>>> product\catalog\CatalogWorker.java"
>>>>>>>>>>>
>>>>>>>>>>> All is explained here https://tortoisesvn.net/docs/r
>>>>>>>>>>> elease/TortoiseSVN_en/tsvn-automation.html#tsvn-automation-basics
>>>>>>>>>>>
>>>>>>>>>>>      From the resulting UI (comparable to Subclipse) I guess
>>>>>>>>>>> changing
>>>>>>>>>>> all
>>>>>>>>>>> lines of a file will have the same effect.
>>>>>>>>>>> Even if indeed the annotations are not lost, they are very hard to
>>>>>>>>>>> use
>>>>>>>>>>> if
>>>>>>>>>>> you need to compare revision by revision.
>>>>>>>>>>>
>>>>>>>>>>> Jacques
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Le 13/09/2016 à 20:21, Jacques Le Roux a écrit :
>>>>>>>>>>>
>>>>>>>>>>> BTW thinking about it, don't you have something similar in
>>>>>>>>>>> IntellIJ?
>>>>>>>>>>>
>>>>>>>>>>> I found an (old) image there https://markphip.blogspot.fr/2
>>>>>>>>>>>
>>>>>>>>>>>> 006/12/subclipse-live-annotations.html
>>>>>>>>>>>>
>>>>>>>>>>>> Jacques
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> Le 13/09/2016 à 20:16, Jacques Le Roux a écrit :
>>>>>>>>>>>>
>>>>>>>>>>>> Thanks Jacopo,
>>>>>>>>>>>>
>>>>>>>>>>>> I found how to use it in TortoiseSVN (it starts from the log
>>>>>>>>>>>> view)
>>>>>>>>>>>>
>>>>>>>>>>>>> It's complementary to what Subclipse gives and so interesting
>>>>>>>>>>>>> but
>>>>>>>>>>>>> not
>>>>>>>>>>>>> comparable.
>>>>>>>>>>>>>
>>>>>>>>>>>>> You don't have this global view Subclipse offers with each
>>>>>>>>>>>>> annotation
>>>>>>>>>>>>> by line from start (r1) to HEAD.
>>>>>>>>>>>>> Very useful with colored annotations in the same column than
>>>>>>>>>>>>> lines
>>>>>>>>>>>>> numbers. But it unfortunately contains only the last revision if
>>>>>>>>>>>>> all
>>>>>>>>>>>>> lines
>>>>>>>>>>>>> have been modified together in that revision.
>>>>>>>>>>>>> Note: to see it you need to use "Show Quick Diff" ("Revision"
>>>>>>>>>>>>> and
>>>>>>>>>>>>> "Combined Colors" are then default options, hovering is enough
>>>>>>>>>>>>> for
>>>>>>>>>>>>> me).
>>>>>>>>>>>>> Same than you decide to show line numbers in this column... More
>>>>>>>>>>>>> for
>>>>>>>>>>>>> those who are still using Eclipse...
>>>>>>>>>>>>>
>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>
>>>>>>>>>>>>> Le 13/09/2016 à 17:40, Jacopo Cappellato a écrit :
>>>>>>>>>>>>>
>>>>>>>>>>>>> Some examples:
>>>>>>>>>>>>>
>>>>>>>>>>>>> svn blame README.md
>>>>>>>>>>>>>
>>>>>>>>>>>>>> after review you run
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> svn blame README.md -r 1:1757044
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> and then
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> svn blame README.md -r 1:1757042
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> and so on to get back in history... nothing is lost,
>>>>>>>>>>>>>> annotations
>>>>>>>>>>>>>> are
>>>>>>>>>>>>>> always
>>>>>>>>>>>>>> there.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> PS: I think there is some trick to do the same with TortoiseSVN
>>>>>>>>>>>>>> but I
>>>>>>>>>>>>>> can't
>>>>>>>>>>>>>> tell you the details since I don't use it
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 5:16 PM, Jacques Le Roux <
>>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Le 13/09/2016 à 16:45, Jacopo Cappellato a écrit :
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 4:36 PM, Jacques Le Roux <
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>> Before applying a such change, I'd really like to know if
>>>>>>>>>>>>>>>> everybody
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> is
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> aware of what that means when it comes to svn annotations. I
>>>>>>>>>>>>>>>>> repeat: we
>>>>>>>>>>>>>>>>> will then lose all the svn annotations history in all the
>>>>>>>>>>>>>>>>> Groovy
>>>>>>>>>>>>>>>>> files.
>>>>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> Jacques, are you aware that you can pass the -r argument to
>>>>>>>>>>>>>>>>> the
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> blame/annotate command?
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> I must say I never use that when looking at annotations in a
>>>>>>>>>>>>>>>> file
>>>>>>>>>>>>>>>> in
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Eclipse. It's maybe useful in certain circumstances, but I
>>>>>>>>>>>>>>>> hardly
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> see
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> when.
>>>>>>>>>>>>>>> And once all the lines a file has been modified in one
>>>>>>>>>>>>>>> commit, I
>>>>>>>>>>>>>>> guess -r
>>>>>>>>>>>>>>> does not help at all, anyway you get only this information. Or
>>>>>>>>>>>>>>> do
>>>>>>>>>>>>>>> I
>>>>>>>>>>>>>>> miss
>>>>>>>>>>>>>>> something? Should I know the revision I'm looking for? I
>>>>>>>>>>>>>>> rather
>>>>>>>>>>>>>>> try
>>>>>>>>>>>>>>> to know
>>>>>>>>>>>>>>> when and why a line has been changed, what are the reasons of
>>>>>>>>>>>>>>> these
>>>>>>>>>>>>>>> changes, maybe to find an related Jira, etc.
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>

[Rishi Solanki]

Yes we have noticed some semicolons, and it seems we need to replace them
manually. Because in all groovy files we have seen the semicolon in the
lincense text as well.

Thank you for your help Jacques :-)

Rishi Solanki
Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: [phone]
http://www.hotwaxsystems.com

On Wed, Nov 2, 2016 at 2:36 AM, Jacques Le Roux <
[hidden email]> wrote:

> Hi Rishi,
>
> It's not the first time we change a *simple thing* in all the source. I
> can live with that, you seem well organised :)
>
> BTW after appling the patch at OFBIZ-8652 I still find 57 trailing
> semicolons :)
>
> Thanks
>
> Jacques
>
>
>
> Le 01/11/2016 à 07:35, Rishi Solanki a écrit :
>
>> Jacques,
>>
>> Yes we would like to commit it as whole, but before commit for the same we
>> have plan to test each component after applying the changes. Like browse
>> to
>> most pages and general work flows. We will post the updates on ticket
>> something like;
>>
>> Test the party component;
>> Pages/Work Flow tested: Find Party, Create Party, View Party, My
>> Communications, Visits, Classification, Security, Invitation pages etc.
>>
>> The above is an example of how we will confirm everything is working
>> properly, with some basic code review. We would follow the same steps for
>> other components.
>>
>> Please let us know plan looks fine to you. Also in case you think we
>> should
>> take care anything else to minimize the possibility of regression Or may
>> be
>> if you think committing the changes per component will help in code
>> review?
>>
>> Thanks!
>>
>>
>>
>> Rishi Solanki
>> Manager, Enterprise Software Development
>> HotWax Systems Pvt. Ltd.
>> Direct: [phone]
>> http://www.hotwaxsystems.com
>>
>> On Fri, Oct 28, 2016 at 4:10 PM, Jacques Le Roux <
>> [hidden email]> wrote:
>>
>> Hi Rishi,
>>>
>>> Will you commit as a whole?
>>>
>>> Jacques
>>>
>>>
>>>
>>> Le 28/10/2016 à 12:07, Rishi Solanki a écrit :
>>>
>>> Started effort under - https://issues.apache.org/jira/browse/OFBIZ-8652
>>>>
>>>> Thanks to Rohit Kaushal for taking care of this. It will take 4-5 days
>>>> for
>>>> testing.
>>>>
>>>>
>>>> Thanks!
>>>>
>>>> Rishi Solanki
>>>> Manager, Enterprise Software Development
>>>> HotWax Systems Pvt. Ltd.
>>>> Direct: [phone]
>>>> http://www.hotwaxsystems.com
>>>>
>>>> On Sat, Sep 17, 2016 at 2:11 AM, Jacques Le Roux <
>>>> [hidden email]> wrote:
>>>>
>>>> Personally I will go this way: I will add or changes lines without
>>>> putting
>>>>
>>>>> semicolons.
>>>>>
>>>>> I'm in favour of bulk changing files, but I'd prefer by component or
>>>>> webapp to ease reviews.
>>>>>
>>>>> Jacques
>>>>>
>>>>>
>>>>> Le 16/09/2016 à 15:36, Rishi Solanki a écrit :
>>>>>
>>>>> I was saying #2 as per the comment from Taher ....
>>>>>
>>>>>> Quick Reference:
>>>>>>
>>>>>> One reply from Taher ... in the same thread.
>>>>>> ==========
>>>>>>
>>>>>> Okay, given the priorities and work we have at the moment, I suggest
>>>>>> we
>>>>>> keep semicolons and use it as the standard unless someone volunteers
>>>>>> to
>>>>>> make a full switch. WDYT?
>>>>>> ==========
>>>>>>
>>>>>>
>>>>>>
>>>>>> Rishi Solanki
>>>>>> Manager, Enterprise Software Development
>>>>>> HotWax Systems Pvt. Ltd.
>>>>>> Direct: [phone]
>>>>>> http://www.hotwaxsystems.com
>>>>>>
>>>>>> On Fri, Sep 16, 2016 at 3:14 PM, Jacques Le Roux <
>>>>>> [hidden email]> wrote:
>>>>>>
>>>>>> I guess you mean 2) by file, then it's OK with me. Though I'd no be
>>>>>>
>>>>>> against having semicolon inconsistency in Groovy files, which was my
>>>>>>> initial question. So no strong opinion about 2 here.
>>>>>>>
>>>>>>> Jacques
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Le 16/09/2016 à 11:31, Rishi Solanki a écrit :
>>>>>>>
>>>>>>> To summarize the overall conversation;
>>>>>>>
>>>>>>> 1) We have decided to bulk remove semicolons from groovy.
>>>>>>>> 2) Until #1 is not complete, we would keep adding semicolon for
>>>>>>>> consistency.
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> Rishi Solanki
>>>>>>>> Manager, Enterprise Software Development
>>>>>>>> HotWax Systems Pvt. Ltd.
>>>>>>>> Direct: [phone]
>>>>>>>> http://www.hotwaxsystems.com
>>>>>>>>
>>>>>>>> On Thu, Sep 15, 2016 at 10:00 AM, Jacques Le Roux <
>>>>>>>> [hidden email]> wrote:
>>>>>>>>
>>>>>>>> Actually I was wrong on this. Thanks to Jacopo I noticed that both
>>>>>>>>
>>>>>>>> Subclipse and Tortoise allow you to select a range of revisions when
>>>>>>>>
>>>>>>>>> you
>>>>>>>>> look for annotations.
>>>>>>>>>
>>>>>>>>> So  it's no longer an issue for me and we can bulk remove trailing
>>>>>>>>> semicolons in Groovy files if we want.
>>>>>>>>>
>>>>>>>>> Sorry for the confusion
>>>>>>>>>
>>>>>>>>> Jacques
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Le 14/09/2016 à 04:42, Scott Gray a écrit :
>>>>>>>>>
>>>>>>>>> I don't particularly care one way or another if groovy files have a
>>>>>>>>>
>>>>>>>>> semi-colon at the end.  I don't even care about consistency because
>>>>>>>>>
>>>>>>>>>> it
>>>>>>>>>> is
>>>>>>>>>> such a minor thing.
>>>>>>>>>>
>>>>>>>>>> I say remove them if they're on a line you happen to be editing,
>>>>>>>>>> otherwise
>>>>>>>>>> just leave them be.
>>>>>>>>>>
>>>>>>>>>> Regarding the annotations, there's plenty of ways to search commit
>>>>>>>>>> logs
>>>>>>>>>> and
>>>>>>>>>> personally I've never found blame to be very useful.  I don't
>>>>>>>>>> think
>>>>>>>>>> it
>>>>>>>>>> should be a reason to block any future bulk S/R cleanups.  We've
>>>>>>>>>> had
>>>>>>>>>> plenty
>>>>>>>>>> in the past (Double -> BigDecimal, Delegator -> EntityQuery,
>>>>>>>>>> whitespace
>>>>>>>>>> removal, etc.) and we should continue to do it to keep things
>>>>>>>>>> clean.
>>>>>>>>>>
>>>>>>>>>> For searching diffs, before using git-svn I used to use: svn log
>>>>>>>>>> -diff
>>>>>>>>>> <path/to.file> and then use the search in the terminal to find the
>>>>>>>>>> string
>>>>>>>>>> I'm looking for.
>>>>>>>>>>
>>>>>>>>>> Regards
>>>>>>>>>> Scott
>>>>>>>>>>
>>>>>>>>>> On 14 September 2016 at 07:33, Jacques Le Roux <
>>>>>>>>>> [hidden email]
>>>>>>>>>>
>>>>>>>>>> wrote:
>>>>>>>>>>
>>>>>>>>>> Le 13/09/2016 à 21:28, Jacques Le Roux a écrit :
>>>>>>>>>>
>>>>>>>>>>> OK found that the same than in Subclipse also exists in
>>>>>>>>>>> TortoiseSVN
>>>>>>>>>>>
>>>>>>>>>>> But you need to use a command line (weird for a GUI), eg (from
>>>>>>>>>>>
>>>>>>>>>>> TortoiseSVN root folder)
>>>>>>>>>>>>
>>>>>>>>>>>> Actually wrong, simply pick a file in Windows file explorer
>>>>>>>>>>>> using
>>>>>>>>>>>>
>>>>>>>>>>>> TortoiseSVN  context menu, et voilà!
>>>>>>>>>>>>
>>>>>>>>>>>> I confirm, totally comparable to Subclipse annotations
>>>>>>>>>>>
>>>>>>>>>>> Jacques
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> TortoiseProc.exe /command:blame /path:"C:\projectASF-Mars\ofbi
>>>>>>>>>>>
>>>>>>>>>>> z\applications\product\src\main\java\org\apache\ofbiz\
>>>>>>>>>>>
>>>>>>>>>>> product\catalog\CatalogWorker.java"
>>>>>>>>>>>>
>>>>>>>>>>>> All is explained here https://tortoisesvn.net/docs/r
>>>>>>>>>>>> elease/TortoiseSVN_en/tsvn-automation.html#tsvn-automation-
>>>>>>>>>>>> basics
>>>>>>>>>>>>
>>>>>>>>>>>>      From the resulting UI (comparable to Subclipse) I guess
>>>>>>>>>>>> changing
>>>>>>>>>>>> all
>>>>>>>>>>>> lines of a file will have the same effect.
>>>>>>>>>>>> Even if indeed the annotations are not lost, they are very hard
>>>>>>>>>>>> to
>>>>>>>>>>>> use
>>>>>>>>>>>> if
>>>>>>>>>>>> you need to compare revision by revision.
>>>>>>>>>>>>
>>>>>>>>>>>> Jacques
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> Le 13/09/2016 à 20:21, Jacques Le Roux a écrit :
>>>>>>>>>>>>
>>>>>>>>>>>> BTW thinking about it, don't you have something similar in
>>>>>>>>>>>> IntellIJ?
>>>>>>>>>>>>
>>>>>>>>>>>> I found an (old) image there https://markphip.blogspot.fr/2
>>>>>>>>>>>>
>>>>>>>>>>>> 006/12/subclipse-live-annotations.html
>>>>>>>>>>>>>
>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> Le 13/09/2016 à 20:16, Jacques Le Roux a écrit :
>>>>>>>>>>>>>
>>>>>>>>>>>>> Thanks Jacopo,
>>>>>>>>>>>>>
>>>>>>>>>>>>> I found how to use it in TortoiseSVN (it starts from the log
>>>>>>>>>>>>> view)
>>>>>>>>>>>>>
>>>>>>>>>>>>> It's complementary to what Subclipse gives and so interesting
>>>>>>>>>>>>>> but
>>>>>>>>>>>>>> not
>>>>>>>>>>>>>> comparable.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> You don't have this global view Subclipse offers with each
>>>>>>>>>>>>>> annotation
>>>>>>>>>>>>>> by line from start (r1) to HEAD.
>>>>>>>>>>>>>> Very useful with colored annotations in the same column than
>>>>>>>>>>>>>> lines
>>>>>>>>>>>>>> numbers. But it unfortunately contains only the last revision
>>>>>>>>>>>>>> if
>>>>>>>>>>>>>> all
>>>>>>>>>>>>>> lines
>>>>>>>>>>>>>> have been modified together in that revision.
>>>>>>>>>>>>>> Note: to see it you need to use "Show Quick Diff" ("Revision"
>>>>>>>>>>>>>> and
>>>>>>>>>>>>>> "Combined Colors" are then default options, hovering is enough
>>>>>>>>>>>>>> for
>>>>>>>>>>>>>> me).
>>>>>>>>>>>>>> Same than you decide to show line numbers in this column...
>>>>>>>>>>>>>> More
>>>>>>>>>>>>>> for
>>>>>>>>>>>>>> those who are still using Eclipse...
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Le 13/09/2016 à 17:40, Jacopo Cappellato a écrit :
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Some examples:
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> svn blame README.md
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> after review you run
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> svn blame README.md -r 1:1757044
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> and then
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> svn blame README.md -r 1:1757042
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> and so on to get back in history... nothing is lost,
>>>>>>>>>>>>>>> annotations
>>>>>>>>>>>>>>> are
>>>>>>>>>>>>>>> always
>>>>>>>>>>>>>>> there.
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> PS: I think there is some trick to do the same with
>>>>>>>>>>>>>>> TortoiseSVN
>>>>>>>>>>>>>>> but I
>>>>>>>>>>>>>>> can't
>>>>>>>>>>>>>>> tell you the details since I don't use it
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 5:16 PM, Jacques Le Roux <
>>>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Le 13/09/2016 à 16:45, Jacopo Cappellato a écrit :
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 4:36 PM, Jacques Le Roux <
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> Before applying a such change, I'd really like to know if
>>>>>>>>>>>>>>>>> everybody
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> is
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> aware of what that means when it comes to svn annotations.
>>>>>>>>>>>>>>>>>> I
>>>>>>>>>>>>>>>>>> repeat: we
>>>>>>>>>>>>>>>>>> will then lose all the svn annotations history in all the
>>>>>>>>>>>>>>>>>> Groovy
>>>>>>>>>>>>>>>>>> files.
>>>>>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> Jacques, are you aware that you can pass the -r argument
>>>>>>>>>>>>>>>>>> to
>>>>>>>>>>>>>>>>>> the
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> blame/annotate command?
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> I must say I never use that when looking at annotations
>>>>>>>>>>>>>>>>> in a
>>>>>>>>>>>>>>>>> file
>>>>>>>>>>>>>>>>> in
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> Eclipse. It's maybe useful in certain circumstances, but I
>>>>>>>>>>>>>>>>> hardly
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> see
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> when.
>>>>>>>>>>>>>>>> And once all the lines a file has been modified in one
>>>>>>>>>>>>>>>> commit, I
>>>>>>>>>>>>>>>> guess -r
>>>>>>>>>>>>>>>> does not help at all, anyway you get only this information.
>>>>>>>>>>>>>>>> Or
>>>>>>>>>>>>>>>> do
>>>>>>>>>>>>>>>> I
>>>>>>>>>>>>>>>> miss
>>>>>>>>>>>>>>>> something? Should I know the revision I'm looking for? I
>>>>>>>>>>>>>>>> rather
>>>>>>>>>>>>>>>> try
>>>>>>>>>>>>>>>> to know
>>>>>>>>>>>>>>>> when and why a line has been changed, what are the reasons
>>>>>>>>>>>>>>>> of
>>>>>>>>>>>>>>>> these
>>>>>>>>>>>>>>>> changes, maybe to find an related Jira, etc.
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>

[Jacques Le Roux]

Hi Rishi,

No need to do it by hand. I did it with regexp S/R in Eclipse and have updated the patch at OFBIZ-8652, please check

Thanks

Jacques


Le 02/11/2016 à 09:18, Rishi Solanki a écrit :

> Yes we have noticed some semicolons, and it seems we need to replace them
> manually. Because in all groovy files we have seen the semicolon in the
> lincense text as well.
>
> Thank you for your help Jacques :-)
>
> Rishi Solanki
> Manager, Enterprise Software Development
> HotWax Systems Pvt. Ltd.
> Direct: [phone]
> http://www.hotwaxsystems.com
>
> On Wed, Nov 2, 2016 at 2:36 AM, Jacques Le Roux <
> [hidden email]> wrote:
>
>> Hi Rishi,
>>
>> It's not the first time we change a *simple thing* in all the source. I
>> can live with that, you seem well organised :)
>>
>> BTW after appling the patch at OFBIZ-8652 I still find 57 trailing
>> semicolons :)
>>
>> Thanks
>>
>> Jacques
>>
>>
>>
>> Le 01/11/2016 à 07:35, Rishi Solanki a écrit :
>>
>>> Jacques,
>>>
>>> Yes we would like to commit it as whole, but before commit for the same we
>>> have plan to test each component after applying the changes. Like browse
>>> to
>>> most pages and general work flows. We will post the updates on ticket
>>> something like;
>>>
>>> Test the party component;
>>> Pages/Work Flow tested: Find Party, Create Party, View Party, My
>>> Communications, Visits, Classification, Security, Invitation pages etc.
>>>
>>> The above is an example of how we will confirm everything is working
>>> properly, with some basic code review. We would follow the same steps for
>>> other components.
>>>
>>> Please let us know plan looks fine to you. Also in case you think we
>>> should
>>> take care anything else to minimize the possibility of regression Or may
>>> be
>>> if you think committing the changes per component will help in code
>>> review?
>>>
>>> Thanks!
>>>
>>>
>>>
>>> Rishi Solanki
>>> Manager, Enterprise Software Development
>>> HotWax Systems Pvt. Ltd.
>>> Direct: [phone]
>>> http://www.hotwaxsystems.com
>>>
>>> On Fri, Oct 28, 2016 at 4:10 PM, Jacques Le Roux <
>>> [hidden email]> wrote:
>>>
>>> Hi Rishi,
>>>> Will you commit as a whole?
>>>>
>>>> Jacques
>>>>
>>>>
>>>>
>>>> Le 28/10/2016 à 12:07, Rishi Solanki a écrit :
>>>>
>>>> Started effort under - https://issues.apache.org/jira/browse/OFBIZ-8652
>>>>> Thanks to Rohit Kaushal for taking care of this. It will take 4-5 days
>>>>> for
>>>>> testing.
>>>>>
>>>>>
>>>>> Thanks!
>>>>>
>>>>> Rishi Solanki
>>>>> Manager, Enterprise Software Development
>>>>> HotWax Systems Pvt. Ltd.
>>>>> Direct: [phone]
>>>>> http://www.hotwaxsystems.com
>>>>>
>>>>> On Sat, Sep 17, 2016 at 2:11 AM, Jacques Le Roux <
>>>>> [hidden email]> wrote:
>>>>>
>>>>> Personally I will go this way: I will add or changes lines without
>>>>> putting
>>>>>
>>>>>> semicolons.
>>>>>>
>>>>>> I'm in favour of bulk changing files, but I'd prefer by component or
>>>>>> webapp to ease reviews.
>>>>>>
>>>>>> Jacques
>>>>>>
>>>>>>
>>>>>> Le 16/09/2016 à 15:36, Rishi Solanki a écrit :
>>>>>>
>>>>>> I was saying #2 as per the comment from Taher ....
>>>>>>
>>>>>>> Quick Reference:
>>>>>>>
>>>>>>> One reply from Taher ... in the same thread.
>>>>>>> ==========
>>>>>>>
>>>>>>> Okay, given the priorities and work we have at the moment, I suggest
>>>>>>> we
>>>>>>> keep semicolons and use it as the standard unless someone volunteers
>>>>>>> to
>>>>>>> make a full switch. WDYT?
>>>>>>> ==========
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Rishi Solanki
>>>>>>> Manager, Enterprise Software Development
>>>>>>> HotWax Systems Pvt. Ltd.
>>>>>>> Direct: [phone]
>>>>>>> http://www.hotwaxsystems.com
>>>>>>>
>>>>>>> On Fri, Sep 16, 2016 at 3:14 PM, Jacques Le Roux <
>>>>>>> [hidden email]> wrote:
>>>>>>>
>>>>>>> I guess you mean 2) by file, then it's OK with me. Though I'd no be
>>>>>>>
>>>>>>> against having semicolon inconsistency in Groovy files, which was my
>>>>>>>> initial question. So no strong opinion about 2 here.
>>>>>>>>
>>>>>>>> Jacques
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> Le 16/09/2016 à 11:31, Rishi Solanki a écrit :
>>>>>>>>
>>>>>>>> To summarize the overall conversation;
>>>>>>>>
>>>>>>>> 1) We have decided to bulk remove semicolons from groovy.
>>>>>>>>> 2) Until #1 is not complete, we would keep adding semicolon for
>>>>>>>>> consistency.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Rishi Solanki
>>>>>>>>> Manager, Enterprise Software Development
>>>>>>>>> HotWax Systems Pvt. Ltd.
>>>>>>>>> Direct: [phone]
>>>>>>>>> http://www.hotwaxsystems.com
>>>>>>>>>
>>>>>>>>> On Thu, Sep 15, 2016 at 10:00 AM, Jacques Le Roux <
>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>
>>>>>>>>> Actually I was wrong on this. Thanks to Jacopo I noticed that both
>>>>>>>>>
>>>>>>>>> Subclipse and Tortoise allow you to select a range of revisions when
>>>>>>>>>
>>>>>>>>>> you
>>>>>>>>>> look for annotations.
>>>>>>>>>>
>>>>>>>>>> So  it's no longer an issue for me and we can bulk remove trailing
>>>>>>>>>> semicolons in Groovy files if we want.
>>>>>>>>>>
>>>>>>>>>> Sorry for the confusion
>>>>>>>>>>
>>>>>>>>>> Jacques
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Le 14/09/2016 à 04:42, Scott Gray a écrit :
>>>>>>>>>>
>>>>>>>>>> I don't particularly care one way or another if groovy files have a
>>>>>>>>>>
>>>>>>>>>> semi-colon at the end.  I don't even care about consistency because
>>>>>>>>>>
>>>>>>>>>>> it
>>>>>>>>>>> is
>>>>>>>>>>> such a minor thing.
>>>>>>>>>>>
>>>>>>>>>>> I say remove them if they're on a line you happen to be editing,
>>>>>>>>>>> otherwise
>>>>>>>>>>> just leave them be.
>>>>>>>>>>>
>>>>>>>>>>> Regarding the annotations, there's plenty of ways to search commit
>>>>>>>>>>> logs
>>>>>>>>>>> and
>>>>>>>>>>> personally I've never found blame to be very useful.  I don't
>>>>>>>>>>> think
>>>>>>>>>>> it
>>>>>>>>>>> should be a reason to block any future bulk S/R cleanups.  We've
>>>>>>>>>>> had
>>>>>>>>>>> plenty
>>>>>>>>>>> in the past (Double -> BigDecimal, Delegator -> EntityQuery,
>>>>>>>>>>> whitespace
>>>>>>>>>>> removal, etc.) and we should continue to do it to keep things
>>>>>>>>>>> clean.
>>>>>>>>>>>
>>>>>>>>>>> For searching diffs, before using git-svn I used to use: svn log
>>>>>>>>>>> -diff
>>>>>>>>>>> <path/to.file> and then use the search in the terminal to find the
>>>>>>>>>>> string
>>>>>>>>>>> I'm looking for.
>>>>>>>>>>>
>>>>>>>>>>> Regards
>>>>>>>>>>> Scott
>>>>>>>>>>>
>>>>>>>>>>> On 14 September 2016 at 07:33, Jacques Le Roux <
>>>>>>>>>>> [hidden email]
>>>>>>>>>>>
>>>>>>>>>>> wrote:
>>>>>>>>>>>
>>>>>>>>>>> Le 13/09/2016 à 21:28, Jacques Le Roux a écrit :
>>>>>>>>>>>
>>>>>>>>>>>> OK found that the same than in Subclipse also exists in
>>>>>>>>>>>> TortoiseSVN
>>>>>>>>>>>>
>>>>>>>>>>>> But you need to use a command line (weird for a GUI), eg (from
>>>>>>>>>>>>
>>>>>>>>>>>> TortoiseSVN root folder)
>>>>>>>>>>>>> Actually wrong, simply pick a file in Windows file explorer
>>>>>>>>>>>>> using
>>>>>>>>>>>>>
>>>>>>>>>>>>> TortoiseSVN  context menu, et voilà!
>>>>>>>>>>>>>
>>>>>>>>>>>>> I confirm, totally comparable to Subclipse annotations
>>>>>>>>>>>> Jacques
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> TortoiseProc.exe /command:blame /path:"C:\projectASF-Mars\ofbi
>>>>>>>>>>>>
>>>>>>>>>>>> z\applications\product\src\main\java\org\apache\ofbiz\
>>>>>>>>>>>>
>>>>>>>>>>>> product\catalog\CatalogWorker.java"
>>>>>>>>>>>>> All is explained here https://tortoisesvn.net/docs/r
>>>>>>>>>>>>> elease/TortoiseSVN_en/tsvn-automation.html#tsvn-automation-
>>>>>>>>>>>>> basics
>>>>>>>>>>>>>
>>>>>>>>>>>>>       From the resulting UI (comparable to Subclipse) I guess
>>>>>>>>>>>>> changing
>>>>>>>>>>>>> all
>>>>>>>>>>>>> lines of a file will have the same effect.
>>>>>>>>>>>>> Even if indeed the annotations are not lost, they are very hard
>>>>>>>>>>>>> to
>>>>>>>>>>>>> use
>>>>>>>>>>>>> if
>>>>>>>>>>>>> you need to compare revision by revision.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> Le 13/09/2016 à 20:21, Jacques Le Roux a écrit :
>>>>>>>>>>>>>
>>>>>>>>>>>>> BTW thinking about it, don't you have something similar in
>>>>>>>>>>>>> IntellIJ?
>>>>>>>>>>>>>
>>>>>>>>>>>>> I found an (old) image there https://markphip.blogspot.fr/2
>>>>>>>>>>>>>
>>>>>>>>>>>>> 006/12/subclipse-live-annotations.html
>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Le 13/09/2016 à 20:16, Jacques Le Roux a écrit :
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Thanks Jacopo,
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> I found how to use it in TortoiseSVN (it starts from the log
>>>>>>>>>>>>>> view)
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> It's complementary to what Subclipse gives and so interesting
>>>>>>>>>>>>>>> but
>>>>>>>>>>>>>>> not
>>>>>>>>>>>>>>> comparable.
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> You don't have this global view Subclipse offers with each
>>>>>>>>>>>>>>> annotation
>>>>>>>>>>>>>>> by line from start (r1) to HEAD.
>>>>>>>>>>>>>>> Very useful with colored annotations in the same column than
>>>>>>>>>>>>>>> lines
>>>>>>>>>>>>>>> numbers. But it unfortunately contains only the last revision
>>>>>>>>>>>>>>> if
>>>>>>>>>>>>>>> all
>>>>>>>>>>>>>>> lines
>>>>>>>>>>>>>>> have been modified together in that revision.
>>>>>>>>>>>>>>> Note: to see it you need to use "Show Quick Diff" ("Revision"
>>>>>>>>>>>>>>> and
>>>>>>>>>>>>>>> "Combined Colors" are then default options, hovering is enough
>>>>>>>>>>>>>>> for
>>>>>>>>>>>>>>> me).
>>>>>>>>>>>>>>> Same than you decide to show line numbers in this column...
>>>>>>>>>>>>>>> More
>>>>>>>>>>>>>>> for
>>>>>>>>>>>>>>> those who are still using Eclipse...
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Le 13/09/2016 à 17:40, Jacopo Cappellato a écrit :
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Some examples:
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> svn blame README.md
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> after review you run
>>>>>>>>>>>>>>>> svn blame README.md -r 1:1757044
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> and then
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> svn blame README.md -r 1:1757042
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> and so on to get back in history... nothing is lost,
>>>>>>>>>>>>>>>> annotations
>>>>>>>>>>>>>>>> are
>>>>>>>>>>>>>>>> always
>>>>>>>>>>>>>>>> there.
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> PS: I think there is some trick to do the same with
>>>>>>>>>>>>>>>> TortoiseSVN
>>>>>>>>>>>>>>>> but I
>>>>>>>>>>>>>>>> can't
>>>>>>>>>>>>>>>> tell you the details since I don't use it
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 5:16 PM, Jacques Le Roux <
>>>>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Le 13/09/2016 à 16:45, Jacopo Cappellato a écrit :
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 4:36 PM, Jacques Le Roux <
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> Before applying a such change, I'd really like to know if
>>>>>>>>>>>>>>>>>> everybody
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> is
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> aware of what that means when it comes to svn annotations.
>>>>>>>>>>>>>>>>>>> I
>>>>>>>>>>>>>>>>>>> repeat: we
>>>>>>>>>>>>>>>>>>> will then lose all the svn annotations history in all the
>>>>>>>>>>>>>>>>>>> Groovy
>>>>>>>>>>>>>>>>>>> files.
>>>>>>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> Jacques, are you aware that you can pass the -r argument
>>>>>>>>>>>>>>>>>>> to
>>>>>>>>>>>>>>>>>>> the
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> blame/annotate command?
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> I must say I never use that when looking at annotations
>>>>>>>>>>>>>>>>>> in a
>>>>>>>>>>>>>>>>>> file
>>>>>>>>>>>>>>>>>> in
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> Eclipse. It's maybe useful in certain circumstances, but I
>>>>>>>>>>>>>>>>>> hardly
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> see
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> when.
>>>>>>>>>>>>>>>>> And once all the lines a file has been modified in one
>>>>>>>>>>>>>>>>> commit, I
>>>>>>>>>>>>>>>>> guess -r
>>>>>>>>>>>>>>>>> does not help at all, anyway you get only this information.
>>>>>>>>>>>>>>>>> Or
>>>>>>>>>>>>>>>>> do
>>>>>>>>>>>>>>>>> I
>>>>>>>>>>>>>>>>> miss
>>>>>>>>>>>>>>>>> something? Should I know the revision I'm looking for? I
>>>>>>>>>>>>>>>>> rather
>>>>>>>>>>>>>>>>> try
>>>>>>>>>>>>>>>>> to know
>>>>>>>>>>>>>>>>> when and why a line has been changed, what are the reasons
>>>>>>>>>>>>>>>>> of
>>>>>>>>>>>>>>>>> these
>>>>>>>>>>>>>>>>> changes, maybe to find an related Jira, etc.
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>

[Rishi Solanki]

Thank you very much Jacques.

Rishi Solanki
Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: [phone]
http://www.hotwaxsystems.com

On Wed, Nov 2, 2016 at 4:55 PM, Jacques Le Roux <
[hidden email]> wrote:

> Hi Rishi,
>
> No need to do it by hand. I did it with regexp S/R in Eclipse and have
> updated the patch at OFBIZ-8652, please check
>
> Thanks
>
> Jacques
>
>
>
> Le 02/11/2016 à 09:18, Rishi Solanki a écrit :
>
>> Yes we have noticed some semicolons, and it seems we need to replace them
>> manually. Because in all groovy files we have seen the semicolon in the
>> lincense text as well.
>>
>> Thank you for your help Jacques :-)
>>
>> Rishi Solanki
>> Manager, Enterprise Software Development
>> HotWax Systems Pvt. Ltd.
>> Direct: [phone]
>> http://www.hotwaxsystems.com
>>
>> On Wed, Nov 2, 2016 at 2:36 AM, Jacques Le Roux <
>> [hidden email]> wrote:
>>
>> Hi Rishi,
>>>
>>> It's not the first time we change a *simple thing* in all the source. I
>>> can live with that, you seem well organised :)
>>>
>>> BTW after appling the patch at OFBIZ-8652 I still find 57 trailing
>>> semicolons :)
>>>
>>> Thanks
>>>
>>> Jacques
>>>
>>>
>>>
>>> Le 01/11/2016 à 07:35, Rishi Solanki a écrit :
>>>
>>> Jacques,
>>>>
>>>> Yes we would like to commit it as whole, but before commit for the same
>>>> we
>>>> have plan to test each component after applying the changes. Like browse
>>>> to
>>>> most pages and general work flows. We will post the updates on ticket
>>>> something like;
>>>>
>>>> Test the party component;
>>>> Pages/Work Flow tested: Find Party, Create Party, View Party, My
>>>> Communications, Visits, Classification, Security, Invitation pages etc.
>>>>
>>>> The above is an example of how we will confirm everything is working
>>>> properly, with some basic code review. We would follow the same steps
>>>> for
>>>> other components.
>>>>
>>>> Please let us know plan looks fine to you. Also in case you think we
>>>> should
>>>> take care anything else to minimize the possibility of regression Or may
>>>> be
>>>> if you think committing the changes per component will help in code
>>>> review?
>>>>
>>>> Thanks!
>>>>
>>>>
>>>>
>>>> Rishi Solanki
>>>> Manager, Enterprise Software Development
>>>> HotWax Systems Pvt. Ltd.
>>>> Direct: [phone]
>>>> http://www.hotwaxsystems.com
>>>>
>>>> On Fri, Oct 28, 2016 at 4:10 PM, Jacques Le Roux <
>>>> [hidden email]> wrote:
>>>>
>>>> Hi Rishi,
>>>>
>>>>> Will you commit as a whole?
>>>>>
>>>>> Jacques
>>>>>
>>>>>
>>>>>
>>>>> Le 28/10/2016 à 12:07, Rishi Solanki a écrit :
>>>>>
>>>>> Started effort under - https://issues.apache.org/jira
>>>>> /browse/OFBIZ-8652
>>>>>
>>>>>> Thanks to Rohit Kaushal for taking care of this. It will take 4-5 days
>>>>>> for
>>>>>> testing.
>>>>>>
>>>>>>
>>>>>> Thanks!
>>>>>>
>>>>>> Rishi Solanki
>>>>>> Manager, Enterprise Software Development
>>>>>> HotWax Systems Pvt. Ltd.
>>>>>> Direct: [phone]
>>>>>> http://www.hotwaxsystems.com
>>>>>>
>>>>>> On Sat, Sep 17, 2016 at 2:11 AM, Jacques Le Roux <
>>>>>> [hidden email]> wrote:
>>>>>>
>>>>>> Personally I will go this way: I will add or changes lines without
>>>>>> putting
>>>>>>
>>>>>> semicolons.
>>>>>>>
>>>>>>> I'm in favour of bulk changing files, but I'd prefer by component or
>>>>>>> webapp to ease reviews.
>>>>>>>
>>>>>>> Jacques
>>>>>>>
>>>>>>>
>>>>>>> Le 16/09/2016 à 15:36, Rishi Solanki a écrit :
>>>>>>>
>>>>>>> I was saying #2 as per the comment from Taher ....
>>>>>>>
>>>>>>> Quick Reference:
>>>>>>>>
>>>>>>>> One reply from Taher ... in the same thread.
>>>>>>>> ==========
>>>>>>>>
>>>>>>>> Okay, given the priorities and work we have at the moment, I suggest
>>>>>>>> we
>>>>>>>> keep semicolons and use it as the standard unless someone volunteers
>>>>>>>> to
>>>>>>>> make a full switch. WDYT?
>>>>>>>> ==========
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> Rishi Solanki
>>>>>>>> Manager, Enterprise Software Development
>>>>>>>> HotWax Systems Pvt. Ltd.
>>>>>>>> Direct: [phone]
>>>>>>>> http://www.hotwaxsystems.com
>>>>>>>>
>>>>>>>> On Fri, Sep 16, 2016 at 3:14 PM, Jacques Le Roux <
>>>>>>>> [hidden email]> wrote:
>>>>>>>>
>>>>>>>> I guess you mean 2) by file, then it's OK with me. Though I'd no be
>>>>>>>>
>>>>>>>> against having semicolon inconsistency in Groovy files, which was my
>>>>>>>>
>>>>>>>>> initial question. So no strong opinion about 2 here.
>>>>>>>>>
>>>>>>>>> Jacques
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Le 16/09/2016 à 11:31, Rishi Solanki a écrit :
>>>>>>>>>
>>>>>>>>> To summarize the overall conversation;
>>>>>>>>>
>>>>>>>>> 1) We have decided to bulk remove semicolons from groovy.
>>>>>>>>>
>>>>>>>>>> 2) Until #1 is not complete, we would keep adding semicolon for
>>>>>>>>>> consistency.
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Rishi Solanki
>>>>>>>>>> Manager, Enterprise Software Development
>>>>>>>>>> HotWax Systems Pvt. Ltd.
>>>>>>>>>> Direct: [phone]
>>>>>>>>>> http://www.hotwaxsystems.com
>>>>>>>>>>
>>>>>>>>>> On Thu, Sep 15, 2016 at 10:00 AM, Jacques Le Roux <
>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>
>>>>>>>>>> Actually I was wrong on this. Thanks to Jacopo I noticed that both
>>>>>>>>>>
>>>>>>>>>> Subclipse and Tortoise allow you to select a range of revisions
>>>>>>>>>> when
>>>>>>>>>>
>>>>>>>>>> you
>>>>>>>>>>> look for annotations.
>>>>>>>>>>>
>>>>>>>>>>> So  it's no longer an issue for me and we can bulk remove
>>>>>>>>>>> trailing
>>>>>>>>>>> semicolons in Groovy files if we want.
>>>>>>>>>>>
>>>>>>>>>>> Sorry for the confusion
>>>>>>>>>>>
>>>>>>>>>>> Jacques
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Le 14/09/2016 à 04:42, Scott Gray a écrit :
>>>>>>>>>>>
>>>>>>>>>>> I don't particularly care one way or another if groovy files
>>>>>>>>>>> have a
>>>>>>>>>>>
>>>>>>>>>>> semi-colon at the end.  I don't even care about consistency
>>>>>>>>>>> because
>>>>>>>>>>>
>>>>>>>>>>> it
>>>>>>>>>>>> is
>>>>>>>>>>>> such a minor thing.
>>>>>>>>>>>>
>>>>>>>>>>>> I say remove them if they're on a line you happen to be editing,
>>>>>>>>>>>> otherwise
>>>>>>>>>>>> just leave them be.
>>>>>>>>>>>>
>>>>>>>>>>>> Regarding the annotations, there's plenty of ways to search
>>>>>>>>>>>> commit
>>>>>>>>>>>> logs
>>>>>>>>>>>> and
>>>>>>>>>>>> personally I've never found blame to be very useful.  I don't
>>>>>>>>>>>> think
>>>>>>>>>>>> it
>>>>>>>>>>>> should be a reason to block any future bulk S/R cleanups.  We've
>>>>>>>>>>>> had
>>>>>>>>>>>> plenty
>>>>>>>>>>>> in the past (Double -> BigDecimal, Delegator -> EntityQuery,
>>>>>>>>>>>> whitespace
>>>>>>>>>>>> removal, etc.) and we should continue to do it to keep things
>>>>>>>>>>>> clean.
>>>>>>>>>>>>
>>>>>>>>>>>> For searching diffs, before using git-svn I used to use: svn log
>>>>>>>>>>>> -diff
>>>>>>>>>>>> <path/to.file> and then use the search in the terminal to find
>>>>>>>>>>>> the
>>>>>>>>>>>> string
>>>>>>>>>>>> I'm looking for.
>>>>>>>>>>>>
>>>>>>>>>>>> Regards
>>>>>>>>>>>> Scott
>>>>>>>>>>>>
>>>>>>>>>>>> On 14 September 2016 at 07:33, Jacques Le Roux <
>>>>>>>>>>>> [hidden email]
>>>>>>>>>>>>
>>>>>>>>>>>> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>> Le 13/09/2016 à 21:28, Jacques Le Roux a écrit :
>>>>>>>>>>>>
>>>>>>>>>>>> OK found that the same than in Subclipse also exists in
>>>>>>>>>>>>> TortoiseSVN
>>>>>>>>>>>>>
>>>>>>>>>>>>> But you need to use a command line (weird for a GUI), eg (from
>>>>>>>>>>>>>
>>>>>>>>>>>>> TortoiseSVN root folder)
>>>>>>>>>>>>>
>>>>>>>>>>>>>> Actually wrong, simply pick a file in Windows file explorer
>>>>>>>>>>>>>> using
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> TortoiseSVN  context menu, et voilà!
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> I confirm, totally comparable to Subclipse annotations
>>>>>>>>>>>>>>
>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> TortoiseProc.exe /command:blame /path:"C:\projectASF-Mars\ofbi
>>>>>>>>>>>>>
>>>>>>>>>>>>> z\applications\product\src\main\java\org\apache\ofbiz\
>>>>>>>>>>>>>
>>>>>>>>>>>>> product\catalog\CatalogWorker.java"
>>>>>>>>>>>>>
>>>>>>>>>>>>>> All is explained here https://tortoisesvn.net/docs/r
>>>>>>>>>>>>>> elease/TortoiseSVN_en/tsvn-automation.html#tsvn-automation-
>>>>>>>>>>>>>> basics
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>       From the resulting UI (comparable to Subclipse) I guess
>>>>>>>>>>>>>> changing
>>>>>>>>>>>>>> all
>>>>>>>>>>>>>> lines of a file will have the same effect.
>>>>>>>>>>>>>> Even if indeed the annotations are not lost, they are very
>>>>>>>>>>>>>> hard
>>>>>>>>>>>>>> to
>>>>>>>>>>>>>> use
>>>>>>>>>>>>>> if
>>>>>>>>>>>>>> you need to compare revision by revision.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Le 13/09/2016 à 20:21, Jacques Le Roux a écrit :
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> BTW thinking about it, don't you have something similar in
>>>>>>>>>>>>>> IntellIJ?
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> I found an (old) image there https://markphip.blogspot.fr/2
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> 006/12/subclipse-live-annotations.html
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Le 13/09/2016 à 20:16, Jacques Le Roux a écrit :
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Thanks Jacopo,
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> I found how to use it in TortoiseSVN (it starts from the log
>>>>>>>>>>>>>>> view)
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> It's complementary to what Subclipse gives and so interesting
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> but
>>>>>>>>>>>>>>>> not
>>>>>>>>>>>>>>>> comparable.
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> You don't have this global view Subclipse offers with each
>>>>>>>>>>>>>>>> annotation
>>>>>>>>>>>>>>>> by line from start (r1) to HEAD.
>>>>>>>>>>>>>>>> Very useful with colored annotations in the same column than
>>>>>>>>>>>>>>>> lines
>>>>>>>>>>>>>>>> numbers. But it unfortunately contains only the last
>>>>>>>>>>>>>>>> revision
>>>>>>>>>>>>>>>> if
>>>>>>>>>>>>>>>> all
>>>>>>>>>>>>>>>> lines
>>>>>>>>>>>>>>>> have been modified together in that revision.
>>>>>>>>>>>>>>>> Note: to see it you need to use "Show Quick Diff"
>>>>>>>>>>>>>>>> ("Revision"
>>>>>>>>>>>>>>>> and
>>>>>>>>>>>>>>>> "Combined Colors" are then default options, hovering is
>>>>>>>>>>>>>>>> enough
>>>>>>>>>>>>>>>> for
>>>>>>>>>>>>>>>> me).
>>>>>>>>>>>>>>>> Same than you decide to show line numbers in this column...
>>>>>>>>>>>>>>>> More
>>>>>>>>>>>>>>>> for
>>>>>>>>>>>>>>>> those who are still using Eclipse...
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Le 13/09/2016 à 17:40, Jacopo Cappellato a écrit :
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Some examples:
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> svn blame README.md
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> after review you run
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> svn blame README.md -r 1:1757044
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> and then
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> svn blame README.md -r 1:1757042
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> and so on to get back in history... nothing is lost,
>>>>>>>>>>>>>>>>> annotations
>>>>>>>>>>>>>>>>> are
>>>>>>>>>>>>>>>>> always
>>>>>>>>>>>>>>>>> there.
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> PS: I think there is some trick to do the same with
>>>>>>>>>>>>>>>>> TortoiseSVN
>>>>>>>>>>>>>>>>> but I
>>>>>>>>>>>>>>>>> can't
>>>>>>>>>>>>>>>>> tell you the details since I don't use it
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 5:16 PM, Jacques Le Roux <
>>>>>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> Le 13/09/2016 à 16:45, Jacopo Cappellato a écrit :
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 4:36 PM, Jacques Le Roux <
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> [hidden email]> wrote:
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> Before applying a such change, I'd really like to know if
>>>>>>>>>>>>>>>>>>> everybody
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> is
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> aware of what that means when it comes to svn
>>>>>>>>>>>>>>>>>>> annotations.
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>>> I
>>>>>>>>>>>>>>>>>>>> repeat: we
>>>>>>>>>>>>>>>>>>>> will then lose all the svn annotations history in all
>>>>>>>>>>>>>>>>>>>> the
>>>>>>>>>>>>>>>>>>>> Groovy
>>>>>>>>>>>>>>>>>>>> files.
>>>>>>>>>>>>>>>>>>>> ...
>>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>>> Jacques, are you aware that you can pass the -r argument
>>>>>>>>>>>>>>>>>>>> to
>>>>>>>>>>>>>>>>>>>> the
>>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>>> blame/annotate command?
>>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>>> Jacopo
>>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>>> I must say I never use that when looking at annotations
>>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> in a
>>>>>>>>>>>>>>>>>>> file
>>>>>>>>>>>>>>>>>>> in
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> Eclipse. It's maybe useful in certain circumstances, but
>>>>>>>>>>>>>>>>>>> I
>>>>>>>>>>>>>>>>>>> hardly
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> see
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> when.
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> And once all the lines a file has been modified in one
>>>>>>>>>>>>>>>>>> commit, I
>>>>>>>>>>>>>>>>>> guess -r
>>>>>>>>>>>>>>>>>> does not help at all, anyway you get only this
>>>>>>>>>>>>>>>>>> information.
>>>>>>>>>>>>>>>>>> Or
>>>>>>>>>>>>>>>>>> do
>>>>>>>>>>>>>>>>>> I
>>>>>>>>>>>>>>>>>> miss
>>>>>>>>>>>>>>>>>> something? Should I know the revision I'm looking for? I
>>>>>>>>>>>>>>>>>> rather
>>>>>>>>>>>>>>>>>> try
>>>>>>>>>>>>>>>>>> to know
>>>>>>>>>>>>>>>>>> when and why a line has been changed, what are the reasons
>>>>>>>>>>>>>>>>>> of
>>>>>>>>>>>>>>>>>> these
>>>>>>>>>>>>>>>>>> changes, maybe to find an related Jira, etc.
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> Jacques
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>

[Arun Patidar-2]

This has been committed at rev:1767764. I have also verified application by
accessing different screens of components.

Please let me know if you find any issue regarding this change.



--
Thanks & Regards
---
Arun Patidar
Manager,Enterprise Software Development
HotWax Mediawww.hotwaxsystems.com


On Wed, Nov 2, 2016 at 5:45 PM, Rishi Solanki <[hidden email]>
wrote:

> Thank you very much Jacques.
>
> Rishi Solanki
> Manager, Enterprise Software Development
> HotWax Systems Pvt. Ltd.
> Direct: [phone]
> http://www.hotwaxsystems.com
>
> On Wed, Nov 2, 2016 at 4:55 PM, Jacques Le Roux <
> [hidden email]> wrote:
>
> > Hi Rishi,
> >
> > No need to do it by hand. I did it with regexp S/R in Eclipse and have
> > updated the patch at OFBIZ-8652, please check
> >
> > Thanks
> >
> > Jacques
> >
> >
> >
> > Le 02/11/2016 à 09:18, Rishi Solanki a écrit :
> >
> >> Yes we have noticed some semicolons, and it seems we need to replace
> them
> >> manually. Because in all groovy files we have seen the semicolon in the
> >> lincense text as well.
> >>
> >> Thank you for your help Jacques :-)
> >>
> >> Rishi Solanki
> >> Manager, Enterprise Software Development
> >> HotWax Systems Pvt. Ltd.
> >> Direct: [phone]
> >> http://www.hotwaxsystems.com
> >>
> >> On Wed, Nov 2, 2016 at 2:36 AM, Jacques Le Roux <
> >> [hidden email]> wrote:
> >>
> >> Hi Rishi,
> >>>
> >>> It's not the first time we change a *simple thing* in all the source. I
> >>> can live with that, you seem well organised :)
> >>>
> >>> BTW after appling the patch at OFBIZ-8652 I still find 57 trailing
> >>> semicolons :)
> >>>
> >>> Thanks
> >>>
> >>> Jacques
> >>>
> >>>
> >>>
> >>> Le 01/11/2016 à 07:35, Rishi Solanki a écrit :
> >>>
> >>> Jacques,
> >>>>
> >>>> Yes we would like to commit it as whole, but before commit for the
> same
> >>>> we
> >>>> have plan to test each component after applying the changes. Like
> browse
> >>>> to
> >>>> most pages and general work flows. We will post the updates on ticket
> >>>> something like;
> >>>>
> >>>> Test the party component;
> >>>> Pages/Work Flow tested: Find Party, Create Party, View Party, My
> >>>> Communications, Visits, Classification, Security, Invitation pages
> etc.
> >>>>
> >>>> The above is an example of how we will confirm everything is working
> >>>> properly, with some basic code review. We would follow the same steps
> >>>> for
> >>>> other components.
> >>>>
> >>>> Please let us know plan looks fine to you. Also in case you think we
> >>>> should
> >>>> take care anything else to minimize the possibility of regression Or
> may
> >>>> be
> >>>> if you think committing the changes per component will help in code
> >>>> review?
> >>>>
> >>>> Thanks!
> >>>>
> >>>>
> >>>>
> >>>> Rishi Solanki
> >>>> Manager, Enterprise Software Development
> >>>> HotWax Systems Pvt. Ltd.
> >>>> Direct: [phone]
> >>>> http://www.hotwaxsystems.com
> >>>>
> >>>> On Fri, Oct 28, 2016 at 4:10 PM, Jacques Le Roux <
> >>>> [hidden email]> wrote:
> >>>>
> >>>> Hi Rishi,
> >>>>
> >>>>> Will you commit as a whole?
> >>>>>
> >>>>> Jacques
> >>>>>
> >>>>>
> >>>>>
> >>>>> Le 28/10/2016 à 12:07, Rishi Solanki a écrit :
> >>>>>
> >>>>> Started effort under - https://issues.apache.org/jira
> >>>>> /browse/OFBIZ-8652
> >>>>>
> >>>>>> Thanks to Rohit Kaushal for taking care of this. It will take 4-5
> days
> >>>>>> for
> >>>>>> testing.
> >>>>>>
> >>>>>>
> >>>>>> Thanks!
> >>>>>>
> >>>>>> Rishi Solanki
> >>>>>> Manager, Enterprise Software Development
> >>>>>> HotWax Systems Pvt. Ltd.
> >>>>>> Direct: [phone]
> >>>>>> http://www.hotwaxsystems.com
> >>>>>>
> >>>>>> On Sat, Sep 17, 2016 at 2:11 AM, Jacques Le Roux <
> >>>>>> [hidden email]> wrote:
> >>>>>>
> >>>>>> Personally I will go this way: I will add or changes lines without
> >>>>>> putting
> >>>>>>
> >>>>>> semicolons.
> >>>>>>>
> >>>>>>> I'm in favour of bulk changing files, but I'd prefer by component
> or
> >>>>>>> webapp to ease reviews.
> >>>>>>>
> >>>>>>> Jacques
> >>>>>>>
> >>>>>>>
> >>>>>>> Le 16/09/2016 à 15:36, Rishi Solanki a écrit :
> >>>>>>>
> >>>>>>> I was saying #2 as per the comment from Taher ....
> >>>>>>>
> >>>>>>> Quick Reference:
> >>>>>>>>
> >>>>>>>> One reply from Taher ... in the same thread.
> >>>>>>>> ==========
> >>>>>>>>
> >>>>>>>> Okay, given the priorities and work we have at the moment, I
> suggest
> >>>>>>>> we
> >>>>>>>> keep semicolons and use it as the standard unless someone
> volunteers
> >>>>>>>> to
> >>>>>>>> make a full switch. WDYT?
> >>>>>>>> ==========
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>> Rishi Solanki
> >>>>>>>> Manager, Enterprise Software Development
> >>>>>>>> HotWax Systems Pvt. Ltd.
> >>>>>>>> Direct: [phone]
> >>>>>>>> http://www.hotwaxsystems.com
> >>>>>>>>
> >>>>>>>> On Fri, Sep 16, 2016 at 3:14 PM, Jacques Le Roux <
> >>>>>>>> [hidden email]> wrote:
> >>>>>>>>
> >>>>>>>> I guess you mean 2) by file, then it's OK with me. Though I'd no
> be
> >>>>>>>>
> >>>>>>>> against having semicolon inconsistency in Groovy files, which was
> my
> >>>>>>>>
> >>>>>>>>> initial question. So no strong opinion about 2 here.
> >>>>>>>>>
> >>>>>>>>> Jacques
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> Le 16/09/2016 à 11:31, Rishi Solanki a écrit :
> >>>>>>>>>
> >>>>>>>>> To summarize the overall conversation;
> >>>>>>>>>
> >>>>>>>>> 1) We have decided to bulk remove semicolons from groovy.
> >>>>>>>>>
> >>>>>>>>>> 2) Until #1 is not complete, we would keep adding semicolon for
> >>>>>>>>>> consistency.
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>> Rishi Solanki
> >>>>>>>>>> Manager, Enterprise Software Development
> >>>>>>>>>> HotWax Systems Pvt. Ltd.
> >>>>>>>>>> Direct: [phone]
> >>>>>>>>>> http://www.hotwaxsystems.com
> >>>>>>>>>>
> >>>>>>>>>> On Thu, Sep 15, 2016 at 10:00 AM, Jacques Le Roux <
> >>>>>>>>>> [hidden email]> wrote:
> >>>>>>>>>>
> >>>>>>>>>> Actually I was wrong on this. Thanks to Jacopo I noticed that
> both
> >>>>>>>>>>
> >>>>>>>>>> Subclipse and Tortoise allow you to select a range of revisions
> >>>>>>>>>> when
> >>>>>>>>>>
> >>>>>>>>>> you
> >>>>>>>>>>> look for annotations.
> >>>>>>>>>>>
> >>>>>>>>>>> So  it's no longer an issue for me and we can bulk remove
> >>>>>>>>>>> trailing
> >>>>>>>>>>> semicolons in Groovy files if we want.
> >>>>>>>>>>>
> >>>>>>>>>>> Sorry for the confusion
> >>>>>>>>>>>
> >>>>>>>>>>> Jacques
> >>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>> Le 14/09/2016 à 04:42, Scott Gray a écrit :
> >>>>>>>>>>>
> >>>>>>>>>>> I don't particularly care one way or another if groovy files
> >>>>>>>>>>> have a
> >>>>>>>>>>>
> >>>>>>>>>>> semi-colon at the end.  I don't even care about consistency
> >>>>>>>>>>> because
> >>>>>>>>>>>
> >>>>>>>>>>> it
> >>>>>>>>>>>> is
> >>>>>>>>>>>> such a minor thing.
> >>>>>>>>>>>>
> >>>>>>>>>>>> I say remove them if they're on a line you happen to be
> editing,
> >>>>>>>>>>>> otherwise
> >>>>>>>>>>>> just leave them be.
> >>>>>>>>>>>>
> >>>>>>>>>>>> Regarding the annotations, there's plenty of ways to search
> >>>>>>>>>>>> commit
> >>>>>>>>>>>> logs
> >>>>>>>>>>>> and
> >>>>>>>>>>>> personally I've never found blame to be very useful.  I don't
> >>>>>>>>>>>> think
> >>>>>>>>>>>> it
> >>>>>>>>>>>> should be a reason to block any future bulk S/R cleanups.
> We've
> >>>>>>>>>>>> had
> >>>>>>>>>>>> plenty
> >>>>>>>>>>>> in the past (Double -> BigDecimal, Delegator -> EntityQuery,
> >>>>>>>>>>>> whitespace
> >>>>>>>>>>>> removal, etc.) and we should continue to do it to keep things
> >>>>>>>>>>>> clean.
> >>>>>>>>>>>>
> >>>>>>>>>>>> For searching diffs, before using git-svn I used to use: svn
> log
> >>>>>>>>>>>> -diff
> >>>>>>>>>>>> <path/to.file> and then use the search in the terminal to find
> >>>>>>>>>>>> the
> >>>>>>>>>>>> string
> >>>>>>>>>>>> I'm looking for.
> >>>>>>>>>>>>
> >>>>>>>>>>>> Regards
> >>>>>>>>>>>> Scott
> >>>>>>>>>>>>
> >>>>>>>>>>>> On 14 September 2016 at 07:33, Jacques Le Roux <
> >>>>>>>>>>>> [hidden email]
> >>>>>>>>>>>>
> >>>>>>>>>>>> wrote:
> >>>>>>>>>>>>
> >>>>>>>>>>>> Le 13/09/2016 à 21:28, Jacques Le Roux a écrit :
> >>>>>>>>>>>>
> >>>>>>>>>>>> OK found that the same than in Subclipse also exists in
> >>>>>>>>>>>>> TortoiseSVN
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> But you need to use a command line (weird for a GUI), eg
> (from
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> TortoiseSVN root folder)
> >>>>>>>>>>>>>
> >>>>>>>>>>>>>> Actually wrong, simply pick a file in Windows file explorer
> >>>>>>>>>>>>>> using
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> TortoiseSVN  context menu, et voilà!
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> I confirm, totally comparable to Subclipse annotations
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>> Jacques
> >>>>>>>>>>>>>
> >>>>>>>>>>>>>
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> TortoiseProc.exe /command:blame
> /path:"C:\projectASF-Mars\ofbi
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> z\applications\product\src\main\java\org\apache\ofbiz\
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> product\catalog\CatalogWorker.java"
> >>>>>>>>>>>>>
> >>>>>>>>>>>>>> All is explained here https://tortoisesvn.net/docs/r
> >>>>>>>>>>>>>> elease/TortoiseSVN_en/tsvn-automation.html#tsvn-automation-
> >>>>>>>>>>>>>> basics
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>>       From the resulting UI (comparable to Subclipse) I
> guess
> >>>>>>>>>>>>>> changing
> >>>>>>>>>>>>>> all
> >>>>>>>>>>>>>> lines of a file will have the same effect.
> >>>>>>>>>>>>>> Even if indeed the annotations are not lost, they are very
> >>>>>>>>>>>>>> hard
> >>>>>>>>>>>>>> to
> >>>>>>>>>>>>>> use
> >>>>>>>>>>>>>> if
> >>>>>>>>>>>>>> you need to compare revision by revision.
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> Jacques
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> Le 13/09/2016 à 20:21, Jacques Le Roux a écrit :
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> BTW thinking about it, don't you have something similar in
> >>>>>>>>>>>>>> IntellIJ?
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> I found an (old) image there https://markphip.blogspot.fr/2
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> 006/12/subclipse-live-annotations.html
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>>> Jacques
> >>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>> Le 13/09/2016 à 20:16, Jacques Le Roux a écrit :
> >>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>> Thanks Jacopo,
> >>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>> I found how to use it in TortoiseSVN (it starts from the
> log
> >>>>>>>>>>>>>>> view)
> >>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>> It's complementary to what Subclipse gives and so
> interesting
> >>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>> but
> >>>>>>>>>>>>>>>> not
> >>>>>>>>>>>>>>>> comparable.
> >>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>> You don't have this global view Subclipse offers with each
> >>>>>>>>>>>>>>>> annotation
> >>>>>>>>>>>>>>>> by line from start (r1) to HEAD.
> >>>>>>>>>>>>>>>> Very useful with colored annotations in the same column
> than
> >>>>>>>>>>>>>>>> lines
> >>>>>>>>>>>>>>>> numbers. But it unfortunately contains only the last
> >>>>>>>>>>>>>>>> revision
> >>>>>>>>>>>>>>>> if
> >>>>>>>>>>>>>>>> all
> >>>>>>>>>>>>>>>> lines
> >>>>>>>>>>>>>>>> have been modified together in that revision.
> >>>>>>>>>>>>>>>> Note: to see it you need to use "Show Quick Diff"
> >>>>>>>>>>>>>>>> ("Revision"
> >>>>>>>>>>>>>>>> and
> >>>>>>>>>>>>>>>> "Combined Colors" are then default options, hovering is
> >>>>>>>>>>>>>>>> enough
> >>>>>>>>>>>>>>>> for
> >>>>>>>>>>>>>>>> me).
> >>>>>>>>>>>>>>>> Same than you decide to show line numbers in this
> column...
> >>>>>>>>>>>>>>>> More
> >>>>>>>>>>>>>>>> for
> >>>>>>>>>>>>>>>> those who are still using Eclipse...
> >>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>> Jacques
> >>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>> Le 13/09/2016 à 17:40, Jacopo Cappellato a écrit :
> >>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>> Some examples:
> >>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>> svn blame README.md
> >>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>> after review you run
> >>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> svn blame README.md -r 1:1757044
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> and then
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> svn blame README.md -r 1:1757042
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> and so on to get back in history... nothing is lost,
> >>>>>>>>>>>>>>>>> annotations
> >>>>>>>>>>>>>>>>> are
> >>>>>>>>>>>>>>>>> always
> >>>>>>>>>>>>>>>>> there.
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> Jacopo
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> PS: I think there is some trick to do the same with
> >>>>>>>>>>>>>>>>> TortoiseSVN
> >>>>>>>>>>>>>>>>> but I
> >>>>>>>>>>>>>>>>> can't
> >>>>>>>>>>>>>>>>> tell you the details since I don't use it
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 5:16 PM, Jacques Le Roux <
> >>>>>>>>>>>>>>>>> [hidden email]> wrote:
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> Le 13/09/2016 à 16:45, Jacopo Cappellato a écrit :
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> On Tue, Sep 13, 2016 at 4:36 PM, Jacques Le Roux <
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> [hidden email]> wrote:
> >>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>> ...
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>> Before applying a such change, I'd really like to know
> if
> >>>>>>>>>>>>>>>>>>> everybody
> >>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>> is
> >>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>> aware of what that means when it comes to svn
> >>>>>>>>>>>>>>>>>>> annotations.
> >>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>>> I
> >>>>>>>>>>>>>>>>>>>> repeat: we
> >>>>>>>>>>>>>>>>>>>> will then lose all the svn annotations history in all
> >>>>>>>>>>>>>>>>>>>> the
> >>>>>>>>>>>>>>>>>>>> Groovy
> >>>>>>>>>>>>>>>>>>>> files.
> >>>>>>>>>>>>>>>>>>>> ...
> >>>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>>> Jacques, are you aware that you can pass the -r
> argument
> >>>>>>>>>>>>>>>>>>>> to
> >>>>>>>>>>>>>>>>>>>> the
> >>>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>>> blame/annotate command?
> >>>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>>> Jacopo
> >>>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>>> I must say I never use that when looking at
> annotations
> >>>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>> in a
> >>>>>>>>>>>>>>>>>>> file
> >>>>>>>>>>>>>>>>>>> in
> >>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>> Eclipse. It's maybe useful in certain circumstances,
> but
> >>>>>>>>>>>>>>>>>>> I
> >>>>>>>>>>>>>>>>>>> hardly
> >>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>> see
> >>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>> when.
> >>>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>> And once all the lines a file has been modified in one
> >>>>>>>>>>>>>>>>>> commit, I
> >>>>>>>>>>>>>>>>>> guess -r
> >>>>>>>>>>>>>>>>>> does not help at all, anyway you get only this
> >>>>>>>>>>>>>>>>>> information.
> >>>>>>>>>>>>>>>>>> Or
> >>>>>>>>>>>>>>>>>> do
> >>>>>>>>>>>>>>>>>> I
> >>>>>>>>>>>>>>>>>> miss
> >>>>>>>>>>>>>>>>>> something? Should I know the revision I'm looking for? I
> >>>>>>>>>>>>>>>>>> rather
> >>>>>>>>>>>>>>>>>> try
> >>>>>>>>>>>>>>>>>> to know
> >>>>>>>>>>>>>>>>>> when and why a line has been changed, what are the
> reasons
> >>>>>>>>>>>>>>>>>> of
> >>>>>>>>>>>>>>>>>> these
> >>>>>>>>>>>>>>>>>> changes, maybe to find an related Jira, etc.
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>> Jacques
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>
> >>>>>>>>>>>>>>>>>>
> >
>