You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "David Goodenough (JIRA)" <ji...@apache.org> on 2006/12/31 23:31:22 UTC

[jira] Commented: (OFBIZ-581) Tidy up imports in Java source

    [ http://issues.apache.org/jira/browse/OFBIZ-581?page=comments#action_12461590 ] 
            
David Goodenough commented on OFBIZ-581:
----------------------------------------

I will try to find out how to configure Eclipse to use relative paths when it generates patches.  Alternatively I will edit the patches manually.  Thanks for fixing it.

> Tidy up imports in Java source
> ------------------------------
>
>                 Key: OFBIZ-581
>                 URL: http://issues.apache.org/jira/browse/OFBIZ-581
>             Project: Apache OFBiz (The Open for Business Project)
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>            Reporter: David Goodenough
>            Priority: Trivial
>         Attachments: clean-imports.patch, clean-imports.patch
>
>
> The current ofbiz source code contains a number of unnecessary imports.  This patch tidy them all up.  While the effect of this is minor (it will speed compilation a little and reduces the possibility of incorrect source in the future) it makes it easier to see those errors/warning that really matter.  

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Re: [jira] Commented: (OFBIZ-581) Tidy up imports in Java source

Posted by Jacques Le Roux <ja...@les7arts.com>.
Thanks Scott,

I guess that's what we all do in this case

BTW Happy new year to all :o)

Jacques

From: "Scott Gray" <le...@gmail.com>
> I find it easy enough to just open the patch, highlight the first
> instance of the path that i don't want, then ctrl + f and click replace all
>
> Regards
> Scott
>
> David Goodenough (JIRA) wrote:
> >     [ http://issues.apache.org/jira/browse/OFBIZ-581?page=comments#action_12461590 ]
> >
> > David Goodenough commented on OFBIZ-581:
> > ----------------------------------------
> >
> > I will try to find out how to configure Eclipse to use relative paths when it generates patches.  Alternatively I will edit the
patches manually.  Thanks for fixing it.
> >
> >
> >> Tidy up imports in Java source
> >> ------------------------------
> >>
> >>                 Key: OFBIZ-581
> >>                 URL: http://issues.apache.org/jira/browse/OFBIZ-581
> >>             Project: Apache OFBiz (The Open for Business Project)
> >>          Issue Type: Improvement
> >>    Affects Versions: SVN trunk
> >>            Reporter: David Goodenough
> >>            Priority: Trivial
> >>         Attachments: clean-imports.patch, clean-imports.patch
> >>
> >>
> >> The current ofbiz source code contains a number of unnecessary imports.  This patch tidy them all up.  While the effect of this
is minor (it will speed compilation a little and reduces the possibility of incorrect source in the future) it makes it easier to
see those errors/warning that really matter.
> >>
> >
> >


Re: [jira] Commented: (OFBIZ-581) Tidy up imports in Java source

Posted by Scott Gray <le...@gmail.com>.
I find it easy enough to just open the patch, highlight the first 
instance of the path that i don't want, then ctrl + f and click replace all

Regards
Scott

David Goodenough (JIRA) wrote:
>     [ http://issues.apache.org/jira/browse/OFBIZ-581?page=comments#action_12461590 ] 
>             
> David Goodenough commented on OFBIZ-581:
> ----------------------------------------
>
> I will try to find out how to configure Eclipse to use relative paths when it generates patches.  Alternatively I will edit the patches manually.  Thanks for fixing it.
>
>   
>> Tidy up imports in Java source
>> ------------------------------
>>
>>                 Key: OFBIZ-581
>>                 URL: http://issues.apache.org/jira/browse/OFBIZ-581
>>             Project: Apache OFBiz (The Open for Business Project)
>>          Issue Type: Improvement
>>    Affects Versions: SVN trunk
>>            Reporter: David Goodenough
>>            Priority: Trivial
>>         Attachments: clean-imports.patch, clean-imports.patch
>>
>>
>> The current ofbiz source code contains a number of unnecessary imports.  This patch tidy them all up.  While the effect of this is minor (it will speed compilation a little and reduces the possibility of incorrect source in the future) it makes it easier to see those errors/warning that really matter.  
>>     
>
>   


Re: [jira] Commented: (OFBIZ-581) Tidy up imports in Java source

Posted by David E Jones <jo...@undersunconsulting.com>.
This most likely won't help you, but I just do it from the command  
line with an "svn diff > mychanges.patch".

-David


On Dec 31, 2006, at 4:17 PM, Chris Howe wrote:

> If you find out, could you please post?  I've been
> manually editing mine and would like to stop doing
> that :).  Thanks!
> --- "David Goodenough (JIRA)" <ji...@apache.org> wrote:
>
>>     [
>>
> http://issues.apache.org/jira/browse/OFBIZ-581? 
> page=comments#action_12461590
>> ]
>>
>> David Goodenough commented on OFBIZ-581:
>> ----------------------------------------
>>
>> I will try to find out how to configure Eclipse to
>> use relative paths when it generates patches.
>> Alternatively I will edit the patches manually.
>> Thanks for fixing it.
>>
>>> Tidy up imports in Java source
>>> ------------------------------
>>>
>>>                 Key: OFBIZ-581
>>>                 URL:
>> http://issues.apache.org/jira/browse/OFBIZ-581
>>>             Project: Apache OFBiz (The Open for
>> Business Project)
>>>          Issue Type: Improvement
>>>    Affects Versions: SVN trunk
>>>            Reporter: David Goodenough
>>>            Priority: Trivial
>>>         Attachments: clean-imports.patch,
>> clean-imports.patch
>>>
>>>
>>> The current ofbiz source code contains a number of
>> unnecessary imports.  This patch tidy them all up.
>> While the effect of this is minor (it will speed
>> compilation a little and reduces the possibility of
>> incorrect source in the future) it makes it easier
>> to see those errors/warning that really matter.
>>
>> -- 
>> This message is automatically generated by JIRA.
>> -
>> If you think it was sent incorrectly contact one of
>> the administrators:
>>
> http://issues.apache.org/jira/secure/Administrators.jspa
>> -
>> For more information on JIRA, see:
>> http://www.atlassian.com/software/jira
>>
>>
>>
>


Re: [jira] Commented: (OFBIZ-581) Tidy up imports in Java source

Posted by Chris Howe <cj...@yahoo.com>.
If you find out, could you please post?  I've been
manually editing mine and would like to stop doing
that :).  Thanks!
--- "David Goodenough (JIRA)" <ji...@apache.org> wrote:

>     [
>
http://issues.apache.org/jira/browse/OFBIZ-581?page=comments#action_12461590
> ] 
>             
> David Goodenough commented on OFBIZ-581:
> ----------------------------------------
> 
> I will try to find out how to configure Eclipse to
> use relative paths when it generates patches. 
> Alternatively I will edit the patches manually. 
> Thanks for fixing it.
> 
> > Tidy up imports in Java source
> > ------------------------------
> >
> >                 Key: OFBIZ-581
> >                 URL:
> http://issues.apache.org/jira/browse/OFBIZ-581
> >             Project: Apache OFBiz (The Open for
> Business Project)
> >          Issue Type: Improvement
> >    Affects Versions: SVN trunk
> >            Reporter: David Goodenough
> >            Priority: Trivial
> >         Attachments: clean-imports.patch,
> clean-imports.patch
> >
> >
> > The current ofbiz source code contains a number of
> unnecessary imports.  This patch tidy them all up. 
> While the effect of this is minor (it will speed
> compilation a little and reduces the possibility of
> incorrect source in the future) it makes it easier
> to see those errors/warning that really matter.  
> 
> -- 
> This message is automatically generated by JIRA.
> -
> If you think it was sent incorrectly contact one of
> the administrators:
>
http://issues.apache.org/jira/secure/Administrators.jspa
> -
> For more information on JIRA, see:
> http://www.atlassian.com/software/jira
> 
>         
>