Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
11 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Adam Heath-2
[hidden email] wrote:
> Author: lektran
> Date: Wed Mar 17 17:28:01 2010
> New Revision: 924378
>
> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
> Log:
> Tabs to spaces

> Modified: ofbiz/trunk/ant
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
> ==============================================================================
> --- ofbiz/trunk/ant (original)
> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
> @@ -26,14 +26,14 @@ else
>  fi
>  
>  find_jar() {
> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
> - if [ $# = 1 ] && [ -e "$1" ]; then
> - echo "$1"
> - else
> - echo "Couldn't find ant-launcher.jar" 1>&2
> - exit 1
> - fi
> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
> +    if [ $# = 1 ] && [ -e "$1" ]; then
> +        echo "$1"
> +    else
> +        echo "Couldn't find ant-launcher.jar" 1>&2
> +        exit 1
> +    fi
>  }
>  "$JAVA" -jar "$(find_jar)" "$@"

I disagree.  shell scripts need to be tab-based.
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Scott Gray-2
On 17/03/2010, at 11:53 AM, Adam Heath wrote:

> [hidden email] wrote:
>> Author: lektran
>> Date: Wed Mar 17 17:28:01 2010
>> New Revision: 924378
>>
>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>> Log:
>> Tabs to spaces
>
>> Modified: ofbiz/trunk/ant
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/ant (original)
>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>> @@ -26,14 +26,14 @@ else
>> fi
>>
>> find_jar() {
>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>> - if [ $# = 1 ] && [ -e "$1" ]; then
>> - echo "$1"
>> - else
>> - echo "Couldn't find ant-launcher.jar" 1>&2
>> - exit 1
>> - fi
>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>> +        echo "$1"
>> +    else
>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>> +        exit 1
>> +    fi
>> }
>> "$JAVA" -jar "$(find_jar)" "$@"
>
> I disagree.  shell scripts need to be tab-based.
Why?

smime.p7s (3K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Adam Heath-2
Scott Gray wrote:

> On 17/03/2010, at 11:53 AM, Adam Heath wrote:
>
>> [hidden email] wrote:
>>> Author: lektran
>>> Date: Wed Mar 17 17:28:01 2010
>>> New Revision: 924378
>>>
>>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>>> Log:
>>> Tabs to spaces
>>> Modified: ofbiz/trunk/ant
>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>>> ==============================================================================
>>> --- ofbiz/trunk/ant (original)
>>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>>> @@ -26,14 +26,14 @@ else
>>> fi
>>>
>>> find_jar() {
>>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>> - if [ $# = 1 ] && [ -e "$1" ]; then
>>> - echo "$1"
>>> - else
>>> - echo "Couldn't find ant-launcher.jar" 1>&2
>>> - exit 1
>>> - fi
>>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>>> +        echo "$1"
>>> +    else
>>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>>> +        exit 1
>>> +    fi
>>> }
>>> "$JAVA" -jar "$(find_jar)" "$@"
>> I disagree.  shell scripts need to be tab-based.
>
> Why?

What there a discussion as to how shell scripts should be formatted?
I've used tabs for  years, tons of scripts I have worked in have used
tabs.  Is there a well known standard for formatting in shell scripts?

Changing it to match how it is done in other files is not reason enough.
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Scott Gray-2
On 17/03/2010, at 12:16 PM, Adam Heath wrote:

> Scott Gray wrote:
>> On 17/03/2010, at 11:53 AM, Adam Heath wrote:
>>
>>> [hidden email] wrote:
>>>> Author: lektran
>>>> Date: Wed Mar 17 17:28:01 2010
>>>> New Revision: 924378
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>>>> Log:
>>>> Tabs to spaces
>>>> Modified: ofbiz/trunk/ant
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>>>> ==============================================================================
>>>> --- ofbiz/trunk/ant (original)
>>>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>>>> @@ -26,14 +26,14 @@ else
>>>> fi
>>>>
>>>> find_jar() {
>>>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>> - if [ $# = 1 ] && [ -e "$1" ]; then
>>>> - echo "$1"
>>>> - else
>>>> - echo "Couldn't find ant-launcher.jar" 1>&2
>>>> - exit 1
>>>> - fi
>>>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>>>> +        echo "$1"
>>>> +    else
>>>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>>>> +        exit 1
>>>> +    fi
>>>> }
>>>> "$JAVA" -jar "$(find_jar)" "$@"
>>> I disagree.  shell scripts need to be tab-based.
>>
>> Why?
>
> What there a discussion as to how shell scripts should be formatted?
> I've used tabs for  years, tons of scripts I have worked in have used
> tabs.  Is there a well known standard for formatting in shell scripts?
>
> Changing it to match how it is done in other files is not reason enough.
We've never discussed file type specific formatting guidelines, so I guess the assumption has always been that we use consistent formatting everywhere.
You having used tabs for years is just as equally not a good reason to use tabs when everything else uses spaces.



smime.p7s (3K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Adam Heath-2
Scott Gray wrote:

> On 17/03/2010, at 12:16 PM, Adam Heath wrote:
>
>> Scott Gray wrote:
>>> On 17/03/2010, at 11:53 AM, Adam Heath wrote:
>>>
>>>> [hidden email] wrote:
>>>>> Author: lektran
>>>>> Date: Wed Mar 17 17:28:01 2010
>>>>> New Revision: 924378
>>>>>
>>>>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>>>>> Log:
>>>>> Tabs to spaces
>>>>> Modified: ofbiz/trunk/ant
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>>>>> ==============================================================================
>>>>> --- ofbiz/trunk/ant (original)
>>>>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>>>>> @@ -26,14 +26,14 @@ else
>>>>> fi
>>>>>
>>>>> find_jar() {
>>>>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>> - if [ $# = 1 ] && [ -e "$1" ]; then
>>>>> - echo "$1"
>>>>> - else
>>>>> - echo "Couldn't find ant-launcher.jar" 1>&2
>>>>> - exit 1
>>>>> - fi
>>>>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>>>>> +        echo "$1"
>>>>> +    else
>>>>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>>>>> +        exit 1
>>>>> +    fi
>>>>> }
>>>>> "$JAVA" -jar "$(find_jar)" "$@"
>>>> I disagree.  shell scripts need to be tab-based.
>>> Why?
>> What there a discussion as to how shell scripts should be formatted?
>> I've used tabs for  years, tons of scripts I have worked in have used
>> tabs.  Is there a well known standard for formatting in shell scripts?
>>
>> Changing it to match how it is done in other files is not reason enough.
>
> We've never discussed file type specific formatting guidelines, so I guess the assumption has always been that we use consistent formatting everywhere.
> You having used tabs for years is just as equally not a good reason to use tabs when everything else uses spaces.

but why change it?  that introduces churn just for the sake of it.  If
there is no consensus, then leaving things along seems the better
approach.

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Scott Gray-2
On 17/03/2010, at 12:49 PM, Adam Heath wrote:

> Scott Gray wrote:
>> On 17/03/2010, at 12:16 PM, Adam Heath wrote:
>>
>>> Scott Gray wrote:
>>>> On 17/03/2010, at 11:53 AM, Adam Heath wrote:
>>>>
>>>>> [hidden email] wrote:
>>>>>> Author: lektran
>>>>>> Date: Wed Mar 17 17:28:01 2010
>>>>>> New Revision: 924378
>>>>>>
>>>>>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>>>>>> Log:
>>>>>> Tabs to spaces
>>>>>> Modified: ofbiz/trunk/ant
>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>>>>>> ==============================================================================
>>>>>> --- ofbiz/trunk/ant (original)
>>>>>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>>>>>> @@ -26,14 +26,14 @@ else
>>>>>> fi
>>>>>>
>>>>>> find_jar() {
>>>>>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>> - if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>> - echo "$1"
>>>>>> - else
>>>>>> - echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>> - exit 1
>>>>>> - fi
>>>>>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>> +        echo "$1"
>>>>>> +    else
>>>>>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>> +        exit 1
>>>>>> +    fi
>>>>>> }
>>>>>> "$JAVA" -jar "$(find_jar)" "$@"
>>>>> I disagree.  shell scripts need to be tab-based.
>>>> Why?
>>> What there a discussion as to how shell scripts should be formatted?
>>> I've used tabs for  years, tons of scripts I have worked in have used
>>> tabs.  Is there a well known standard for formatting in shell scripts?
>>>
>>> Changing it to match how it is done in other files is not reason enough.
>>
>> We've never discussed file type specific formatting guidelines, so I guess the assumption has always been that we use consistent formatting everywhere.
>> You having used tabs for years is just as equally not a good reason to use tabs when everything else uses spaces.
>
> but why change it?  that introduces churn just for the sake of it.  If
> there is no consensus, then leaving things along seems the better
> approach.
I changed it because we've always used spaces for indentation everywhere, as it I see it you are seeking an exception to that for shell scripts and a consensus has not yet formed around that.
If you're looking for precedent then note that this is the only shell script which contained tabs for indenting and those tabs were introduced by you less than 3 months ago.

Regards
Scott


smime.p7s (3K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Adam Heath-2
Scott Gray wrote:

> On 17/03/2010, at 12:49 PM, Adam Heath wrote:
>
>> Scott Gray wrote:
>>> On 17/03/2010, at 12:16 PM, Adam Heath wrote:
>>>
>>>> Scott Gray wrote:
>>>>> On 17/03/2010, at 11:53 AM, Adam Heath wrote:
>>>>>
>>>>>> [hidden email] wrote:
>>>>>>> Author: lektran
>>>>>>> Date: Wed Mar 17 17:28:01 2010
>>>>>>> New Revision: 924378
>>>>>>>
>>>>>>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>>>>>>> Log:
>>>>>>> Tabs to spaces
>>>>>>> Modified: ofbiz/trunk/ant
>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>>>>>>> ==============================================================================
>>>>>>> --- ofbiz/trunk/ant (original)
>>>>>>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>>>>>>> @@ -26,14 +26,14 @@ else
>>>>>>> fi
>>>>>>>
>>>>>>> find_jar() {
>>>>>>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>> - if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>> - echo "$1"
>>>>>>> - else
>>>>>>> - echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>> - exit 1
>>>>>>> - fi
>>>>>>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>> +        echo "$1"
>>>>>>> +    else
>>>>>>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>> +        exit 1
>>>>>>> +    fi
>>>>>>> }
>>>>>>> "$JAVA" -jar "$(find_jar)" "$@"
>>>>>> I disagree.  shell scripts need to be tab-based.
>>>>> Why?
>>>> What there a discussion as to how shell scripts should be formatted?
>>>> I've used tabs for  years, tons of scripts I have worked in have used
>>>> tabs.  Is there a well known standard for formatting in shell scripts?
>>>>
>>>> Changing it to match how it is done in other files is not reason enough.
>>> We've never discussed file type specific formatting guidelines, so I guess the assumption has always been that we use consistent formatting everywhere.
>>> You having used tabs for years is just as equally not a good reason to use tabs when everything else uses spaces.
>> but why change it?  that introduces churn just for the sake of it.  If
>> there is no consensus, then leaving things along seems the better
>> approach.
>
> I changed it because we've always used spaces for indentation everywhere, as it I see it you are seeking an exception to that for shell scripts and a consensus has not yet formed around that.
> If you're looking for precedent then note that this is the only shell script which contained tabs for indenting and those tabs were introduced by you less than 3 months ago.

debian/ofbiz-applications.postinst
debian/ofbiz-framework.config
debian/ofbiz-framework.postrm
debian/ofbiz-framework.ofbiz.init
debian/ofbiz-specialpurpose.postinst
debian/ofbiz-applications.prerm
debian/ofbiz-framework.postinst
debian/ofbiz-framework.prerm
debian/ofbiz-specialpurpose.prerm

The above are shell scripts, added/maintained by me, of course, that
all contain tabs.

debian/move-files.pl contains tabs.

debian/rules does as well, but that is a Makefile, so it kind of has to.

debian/ofbiz-framework.logrotate has tabs, but that's the pattern in
debian.

debian/extract-entity-engine.xslt and debian/override-data.xslt are
xml files with a single-space indent.

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Scott Gray-2
On 18/03/2010, at 11:40 AM, Adam Heath wrote:

> Scott Gray wrote:
>> On 17/03/2010, at 12:49 PM, Adam Heath wrote:
>>
>>> Scott Gray wrote:
>>>> On 17/03/2010, at 12:16 PM, Adam Heath wrote:
>>>>
>>>>> Scott Gray wrote:
>>>>>> On 17/03/2010, at 11:53 AM, Adam Heath wrote:
>>>>>>
>>>>>>> [hidden email] wrote:
>>>>>>>> Author: lektran
>>>>>>>> Date: Wed Mar 17 17:28:01 2010
>>>>>>>> New Revision: 924378
>>>>>>>>
>>>>>>>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>>>>>>>> Log:
>>>>>>>> Tabs to spaces
>>>>>>>> Modified: ofbiz/trunk/ant
>>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>>>>>>>> ==============================================================================
>>>>>>>> --- ofbiz/trunk/ant (original)
>>>>>>>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>>>>>>>> @@ -26,14 +26,14 @@ else
>>>>>>>> fi
>>>>>>>>
>>>>>>>> find_jar() {
>>>>>>>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>>> - if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>>> - echo "$1"
>>>>>>>> - else
>>>>>>>> - echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>>> - exit 1
>>>>>>>> - fi
>>>>>>>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>>> +        echo "$1"
>>>>>>>> +    else
>>>>>>>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>>> +        exit 1
>>>>>>>> +    fi
>>>>>>>> }
>>>>>>>> "$JAVA" -jar "$(find_jar)" "$@"
>>>>>>> I disagree.  shell scripts need to be tab-based.
>>>>>> Why?
>>>>> What there a discussion as to how shell scripts should be formatted?
>>>>> I've used tabs for  years, tons of scripts I have worked in have used
>>>>> tabs.  Is there a well known standard for formatting in shell scripts?
>>>>>
>>>>> Changing it to match how it is done in other files is not reason enough.
>>>> We've never discussed file type specific formatting guidelines, so I guess the assumption has always been that we use consistent formatting everywhere.
>>>> You having used tabs for years is just as equally not a good reason to use tabs when everything else uses spaces.
>>> but why change it?  that introduces churn just for the sake of it.  If
>>> there is no consensus, then leaving things along seems the better
>>> approach.
>>
>> I changed it because we've always used spaces for indentation everywhere, as it I see it you are seeking an exception to that for shell scripts and a consensus has not yet formed around that.
>> If you're looking for precedent then note that this is the only shell script which contained tabs for indenting and those tabs were introduced by you less than 3 months ago.
>
> debian/ofbiz-applications.postinst
> debian/ofbiz-framework.config
> debian/ofbiz-framework.postrm
> debian/ofbiz-framework.ofbiz.init
> debian/ofbiz-specialpurpose.postinst
> debian/ofbiz-applications.prerm
> debian/ofbiz-framework.postinst
> debian/ofbiz-framework.prerm
> debian/ofbiz-specialpurpose.prerm
>
> The above are shell scripts, added/maintained by me, of course, that
> all contain tabs.
>
> debian/move-files.pl contains tabs.
>
> debian/rules does as well, but that is a Makefile, so it kind of has to.
>
> debian/ofbiz-framework.logrotate has tabs, but that's the pattern in
> debian.
>
> debian/extract-entity-engine.xslt and debian/override-data.xslt are
> xml files with a single-space indent.
Yes I purposely didn't touch those because I know nothing about debian and how tabs to spaces would affect it.

Arguing that these recently introduced files set some sort of precedent when we've had shell scripts using spaces and not tabs for quite a number of years doesn't really cut it IMO.  I'm sorry that spaces don't jibe with how you like to work but you are the one trying to introduce change here, not me.

Regards
Scott

smime.p7s (3K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Adam Heath-2
Scott Gray wrote:

> On 18/03/2010, at 11:40 AM, Adam Heath wrote:
>
>> Scott Gray wrote:
>>> On 17/03/2010, at 12:49 PM, Adam Heath wrote:
>>>
>>>> Scott Gray wrote:
>>>>> On 17/03/2010, at 12:16 PM, Adam Heath wrote:
>>>>>
>>>>>> Scott Gray wrote:
>>>>>>> On 17/03/2010, at 11:53 AM, Adam Heath wrote:
>>>>>>>
>>>>>>>> [hidden email] wrote:
>>>>>>>>> Author: lektran
>>>>>>>>> Date: Wed Mar 17 17:28:01 2010
>>>>>>>>> New Revision: 924378
>>>>>>>>>
>>>>>>>>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>>>>>>>>> Log:
>>>>>>>>> Tabs to spaces
>>>>>>>>> Modified: ofbiz/trunk/ant
>>>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>>>>>>>>> ==============================================================================
>>>>>>>>> --- ofbiz/trunk/ant (original)
>>>>>>>>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>>>>>>>>> @@ -26,14 +26,14 @@ else
>>>>>>>>> fi
>>>>>>>>>
>>>>>>>>> find_jar() {
>>>>>>>>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>>>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>>>> - if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>>>> - echo "$1"
>>>>>>>>> - else
>>>>>>>>> - echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>>>> - exit 1
>>>>>>>>> - fi
>>>>>>>>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>>>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>>>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>>>> +        echo "$1"
>>>>>>>>> +    else
>>>>>>>>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>>>> +        exit 1
>>>>>>>>> +    fi
>>>>>>>>> }
>>>>>>>>> "$JAVA" -jar "$(find_jar)" "$@"
>>>>>>>> I disagree.  shell scripts need to be tab-based.
>>>>>>> Why?
>>>>>> What there a discussion as to how shell scripts should be formatted?
>>>>>> I've used tabs for  years, tons of scripts I have worked in have used
>>>>>> tabs.  Is there a well known standard for formatting in shell scripts?
>>>>>>
>>>>>> Changing it to match how it is done in other files is not reason enough.
>>>>> We've never discussed file type specific formatting guidelines, so I guess the assumption has always been that we use consistent formatting everywhere.
>>>>> You having used tabs for years is just as equally not a good reason to use tabs when everything else uses spaces.
>>>> but why change it?  that introduces churn just for the sake of it.  If
>>>> there is no consensus, then leaving things along seems the better
>>>> approach.
>>> I changed it because we've always used spaces for indentation everywhere, as it I see it you are seeking an exception to that for shell scripts and a consensus has not yet formed around that.
>>> If you're looking for precedent then note that this is the only shell script which contained tabs for indenting and those tabs were introduced by you less than 3 months ago.
>> debian/ofbiz-applications.postinst
>> debian/ofbiz-framework.config
>> debian/ofbiz-framework.postrm
>> debian/ofbiz-framework.ofbiz.init
>> debian/ofbiz-specialpurpose.postinst
>> debian/ofbiz-applications.prerm
>> debian/ofbiz-framework.postinst
>> debian/ofbiz-framework.prerm
>> debian/ofbiz-specialpurpose.prerm
>>
>> The above are shell scripts, added/maintained by me, of course, that
>> all contain tabs.
>>
>> debian/move-files.pl contains tabs.
>>
>> debian/rules does as well, but that is a Makefile, so it kind of has to.
>>
>> debian/ofbiz-framework.logrotate has tabs, but that's the pattern in
>> debian.
>>
>> debian/extract-entity-engine.xslt and debian/override-data.xslt are
>> xml files with a single-space indent.
>
> Yes I purposely didn't touch those because I know nothing about debian and how tabs to spaces would affect it.
>
> Arguing that these recently introduced files set some sort of precedent when we've had shell scripts using spaces and not tabs for quite a number of years doesn't really cut it IMO.  I'm sorry that spaces don't jibe with how you like to work but you are the one trying to introduce change here, not me.

Yeah, I know that I'm the one who introduced these files.

I didn't change any of the other shell scripts, you are the one who
did so.  I'm just trying to discuss if that is the proper approach for
ofbiz.  And I don't think we have reached a consensus.   You and I
obviously don't agree, so that proves their isn't consensus.  The
safest thing to do in that case is not change a thing with the file(s).

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

BJ Freeman
In reply to this post by Adam Heath-2
Yes I remember many discussion and a major effort to make all the file
to indent based on 4 spaces.

=========================
BJ Freeman
http://bjfreeman.elance.com
Strategic Power Office with Supplier Automation <http://www.businessesnetwork.com/automation/viewforum.php?f=93>
Specialtymarket.com <http://www.specialtymarket.com/>

Systems Integrator-- Glad to Assist

Chat  Y! messenger: bjfr33man
Linkedin
<http://www.linkedin.com/profile?viewProfile=&key=1237480&locale=en_US&trk=tab_pro>


Scott Gray sent the following on 3/18/2010 10:49 AM:

> On 18/03/2010, at 11:40 AM, Adam Heath wrote:
>
>> Scott Gray wrote:
>>> On 17/03/2010, at 12:49 PM, Adam Heath wrote:
>>>
>>>> Scott Gray wrote:
>>>>> On 17/03/2010, at 12:16 PM, Adam Heath wrote:
>>>>>
>>>>>> Scott Gray wrote:
>>>>>>> On 17/03/2010, at 11:53 AM, Adam Heath wrote:
>>>>>>>
>>>>>>>> [hidden email] wrote:
>>>>>>>>> Author: lektran
>>>>>>>>> Date: Wed Mar 17 17:28:01 2010
>>>>>>>>> New Revision: 924378
>>>>>>>>>
>>>>>>>>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>>>>>>>>> Log:
>>>>>>>>> Tabs to spaces
>>>>>>>>> Modified: ofbiz/trunk/ant
>>>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>>>>>>>>> ==============================================================================
>>>>>>>>> --- ofbiz/trunk/ant (original)
>>>>>>>>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>>>>>>>>> @@ -26,14 +26,14 @@ else
>>>>>>>>> fi
>>>>>>>>>
>>>>>>>>> find_jar() {
>>>>>>>>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>>>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>>>> - if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>>>> - echo "$1"
>>>>>>>>> - else
>>>>>>>>> - echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>>>> - exit 1
>>>>>>>>> - fi
>>>>>>>>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>>>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>>>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>>>> +        echo "$1"
>>>>>>>>> +    else
>>>>>>>>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>>>> +        exit 1
>>>>>>>>> +    fi
>>>>>>>>> }
>>>>>>>>> "$JAVA" -jar "$(find_jar)" "$@"
>>>>>>>> I disagree.  shell scripts need to be tab-based.
>>>>>>> Why?
>>>>>> What there a discussion as to how shell scripts should be formatted?
>>>>>> I've used tabs for  years, tons of scripts I have worked in have used
>>>>>> tabs.  Is there a well known standard for formatting in shell scripts?
>>>>>>
>>>>>> Changing it to match how it is done in other files is not reason enough.
>>>>> We've never discussed file type specific formatting guidelines, so I guess the assumption has always been that we use consistent formatting everywhere.
>>>>> You having used tabs for years is just as equally not a good reason to use tabs when everything else uses spaces.
>>>> but why change it?  that introduces churn just for the sake of it.  If
>>>> there is no consensus, then leaving things along seems the better
>>>> approach.
>>> I changed it because we've always used spaces for indentation everywhere, as it I see it you are seeking an exception to that for shell scripts and a consensus has not yet formed around that.
>>> If you're looking for precedent then note that this is the only shell script which contained tabs for indenting and those tabs were introduced by you less than 3 months ago.
>> debian/ofbiz-applications.postinst
>> debian/ofbiz-framework.config
>> debian/ofbiz-framework.postrm
>> debian/ofbiz-framework.ofbiz.init
>> debian/ofbiz-specialpurpose.postinst
>> debian/ofbiz-applications.prerm
>> debian/ofbiz-framework.postinst
>> debian/ofbiz-framework.prerm
>> debian/ofbiz-specialpurpose.prerm
>>
>> The above are shell scripts, added/maintained by me, of course, that
>> all contain tabs.
>>
>> debian/move-files.pl contains tabs.
>>
>> debian/rules does as well, but that is a Makefile, so it kind of has to.
>>
>> debian/ofbiz-framework.logrotate has tabs, but that's the pattern in
>> debian.
>>
>> debian/extract-entity-engine.xslt and debian/override-data.xslt are
>> xml files with a single-space indent.
>
> Yes I purposely didn't touch those because I know nothing about debian and how tabs to spaces would affect it.
>
> Arguing that these recently introduced files set some sort of precedent when we've had shell scripts using spaces and not tabs for quite a number of years doesn't really cut it IMO.  I'm sorry that spaces don't jibe with how you like to work but you are the one trying to introduce change here, not me.
>
> Regards
> Scott


Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r924378 [1/8] - in /ofbiz/trunk: ./ applications/accounting/config/ applications/accounting/webapp/accounting/WEB-INF/ applications/accounting/webapp/accounting/payment/report/ applications/accounting/widget/ applications/commonext/webapp/o...

Scott Gray-2
In reply to this post by Adam Heath-2
On 18/03/2010, at 12:00 PM, Adam Heath wrote:

> Scott Gray wrote:
>> On 18/03/2010, at 11:40 AM, Adam Heath wrote:
>>
>>> Scott Gray wrote:
>>>> On 17/03/2010, at 12:49 PM, Adam Heath wrote:
>>>>
>>>>> Scott Gray wrote:
>>>>>> On 17/03/2010, at 12:16 PM, Adam Heath wrote:
>>>>>>
>>>>>>> Scott Gray wrote:
>>>>>>>> On 17/03/2010, at 11:53 AM, Adam Heath wrote:
>>>>>>>>
>>>>>>>>> [hidden email] wrote:
>>>>>>>>>> Author: lektran
>>>>>>>>>> Date: Wed Mar 17 17:28:01 2010
>>>>>>>>>> New Revision: 924378
>>>>>>>>>>
>>>>>>>>>> URL: http://svn.apache.org/viewvc?rev=924378&view=rev
>>>>>>>>>> Log:
>>>>>>>>>> Tabs to spaces
>>>>>>>>>> Modified: ofbiz/trunk/ant
>>>>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/ant?rev=924378&r1=924377&r2=924378&view=diff
>>>>>>>>>> ==============================================================================
>>>>>>>>>> --- ofbiz/trunk/ant (original)
>>>>>>>>>> +++ ofbiz/trunk/ant Wed Mar 17 17:28:01 2010
>>>>>>>>>> @@ -26,14 +26,14 @@ else
>>>>>>>>>> fi
>>>>>>>>>>
>>>>>>>>>> find_jar() {
>>>>>>>>>> - top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>>>>> - set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>>>>> - if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>>>>> - echo "$1"
>>>>>>>>>> - else
>>>>>>>>>> - echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>>>>> - exit 1
>>>>>>>>>> - fi
>>>>>>>>>> +    top="$(cd "$(dirname "$0")"; echo "$PWD")"
>>>>>>>>>> +    set -- "$top"/framework/base/lib/ant-launcher-*.jar
>>>>>>>>>> +    if [ $# = 1 ] && [ -e "$1" ]; then
>>>>>>>>>> +        echo "$1"
>>>>>>>>>> +    else
>>>>>>>>>> +        echo "Couldn't find ant-launcher.jar" 1>&2
>>>>>>>>>> +        exit 1
>>>>>>>>>> +    fi
>>>>>>>>>> }
>>>>>>>>>> "$JAVA" -jar "$(find_jar)" "$@"
>>>>>>>>> I disagree.  shell scripts need to be tab-based.
>>>>>>>> Why?
>>>>>>> What there a discussion as to how shell scripts should be formatted?
>>>>>>> I've used tabs for  years, tons of scripts I have worked in have used
>>>>>>> tabs.  Is there a well known standard for formatting in shell scripts?
>>>>>>>
>>>>>>> Changing it to match how it is done in other files is not reason enough.
>>>>>> We've never discussed file type specific formatting guidelines, so I guess the assumption has always been that we use consistent formatting everywhere.
>>>>>> You having used tabs for years is just as equally not a good reason to use tabs when everything else uses spaces.
>>>>> but why change it?  that introduces churn just for the sake of it.  If
>>>>> there is no consensus, then leaving things along seems the better
>>>>> approach.
>>>> I changed it because we've always used spaces for indentation everywhere, as it I see it you are seeking an exception to that for shell scripts and a consensus has not yet formed around that.
>>>> If you're looking for precedent then note that this is the only shell script which contained tabs for indenting and those tabs were introduced by you less than 3 months ago.
>>> debian/ofbiz-applications.postinst
>>> debian/ofbiz-framework.config
>>> debian/ofbiz-framework.postrm
>>> debian/ofbiz-framework.ofbiz.init
>>> debian/ofbiz-specialpurpose.postinst
>>> debian/ofbiz-applications.prerm
>>> debian/ofbiz-framework.postinst
>>> debian/ofbiz-framework.prerm
>>> debian/ofbiz-specialpurpose.prerm
>>>
>>> The above are shell scripts, added/maintained by me, of course, that
>>> all contain tabs.
>>>
>>> debian/move-files.pl contains tabs.
>>>
>>> debian/rules does as well, but that is a Makefile, so it kind of has to.
>>>
>>> debian/ofbiz-framework.logrotate has tabs, but that's the pattern in
>>> debian.
>>>
>>> debian/extract-entity-engine.xslt and debian/override-data.xslt are
>>> xml files with a single-space indent.
>>
>> Yes I purposely didn't touch those because I know nothing about debian and how tabs to spaces would affect it.
>>
>> Arguing that these recently introduced files set some sort of precedent when we've had shell scripts using spaces and not tabs for quite a number of years doesn't really cut it IMO.  I'm sorry that spaces don't jibe with how you like to work but you are the one trying to introduce change here, not me.
>
> Yeah, I know that I'm the one who introduced these files.
>
> I didn't change any of the other shell scripts, you are the one who
> did so.  I'm just trying to discuss if that is the proper approach for
> ofbiz.  And I don't think we have reached a consensus.   You and I
> obviously don't agree, so that proves their isn't consensus.  The
> safest thing to do in that case is not change a thing with the file(s).
Uh say what?  You introduced tabs into a shell script that had only spaces for indentation which resulted in a mix of indentation styles.  You are the only person who has ever introduced tabs into shell scripts, we've always used spaces for indentation in any files that are managed by the community, all I did was restore the consistency that has existed for quite a number of years.

If you want to change what we've always done then discuss it and get your consensus.  All my commit did was restore the file in question back to its previous formatting state.



smime.p7s (3K) Download Attachment