Scott,

Please see my comments inline.


Op 21 april 2012 11:35 schreef Scott Gray <[email protected]> het
volgende:

> I haven't been following all these strange jenkins threads but I have two
> points:
>

Maybe you should, so you know what the issue is about.


> 1. Don't be so aggressive, it'll get you nowhere and is annoying to read.
>

I wasn't. Just defending the case. Let me apologize for everybody who makes
you feel annoyed.


> 2. This is a CONFIGURATION file, they exist so that if the provided setup
> doesn't suit your needs YOU CAN CHANGE IT.
>

Using capitals is in terms of nettique is like shouting. It could be
perceived as agressive. One could feel annoyed!


>
> You simply don't change default settings within a release branch, it
> removes the stability that one should reasonably expect from a release.
>  But for god's sake, these are configuration files, if it doesn't suit then
> change them to to your heart's content in your


Profanity isn't called for. Be civil!


> downloaded copy.
>
> Regards
> Scott
>
> On 21/04/2012, at 8:46 PM, Pierre Smits wrote:
>
> > Maybe loss of common sense and growth of rigidness of convictions gets in
> > your way. I don't know....
> >
> > As one wants to have the test ports stay the same as production ports, so
> > that a specific group of end-users (namely developers and testers) can
> NOT
> > do testing in a CI way. Merely because it's a release. Because the new
> port
> > numbers might be in use.... And the old port numbers (read production
> port
> > numbers) are expected to be not?
> >
> > And saying: I wont deliver to JIRA, because that is NOT my way. So the
> > community can't test before code gets into the stream. Even when it is
> > asked for by the community....
> >
> > I could call that being stupid... It's surely not helping the product and
> > this community along.
> >
> > As is stated in ASF regulations and guidelines AND established in other
> > threads on this ML: End users should not have access development code or
> > test functionality in releases. For that there are other means (read
> SVN).
> > But hey, developers and testers are end users too. So treating all in the
> > same manner might be acceptible, when you're not thinking things through.
> >
> > Or as another stated: having a fix for missing help screens is not bug
> > fixing, but improving the release (to paraphrase) and is therefore not
> > going to be implemented in a release.
> >
> > Maybe you should ask yourself why you got involved in this project, this
> > community and why you were asked to join either the PMC or the group of
> > committers and whether you are still adhering to those principles or
> > guidelines. From the outside (meaning not being a PMC or a committer, but
> > being part of this community) it often seems like your not.
> >
> > If you still feel you are then stay in touch with the community. If not,
> > please state so and .....
> >
> > Regards,
> >
> > Pierre
> >
> >
> >
> >
> > Op 21 april 2012 09:35 schreef Jacques Le Roux <
> [email protected]
> >> het volgende:
> >
> >> Please Erwan revert: this is not a bug and should not be backported,
> >> unless there is a consensus in the community.
> >>
> >> Thanks
> >>
> >> Jacques
> >>
> >>
> >> Author: erwan
> >>> Date: Fri Apr 20 21:35:38 2012
> >>> New Revision: 1328518
> >>>
> >>> URL: http://svn.apache.org/viewvc?**rev=1328518&view=rev<
> http://svn.apache.org/viewvc?rev=1328518&view=rev>
> >>> Log:
> >>> OFBIZ-4827 Release branch doesn't test in Jenkins
> >>>
> >>> Modified:
> >>>  ofbiz/branches/release11.04/**framework/base/config/test-**
> >>> containers.xml
> >>>
> >>> Modified: ofbiz/branches/release11.04/**framework/base/config/test-**
> >>> containers.xml
> >>> URL: http://svn.apache.org/viewvc/**ofbiz/branches/release11.04/**
> >>> framework/base/config/test-**containers.xml?rev=1328518&r1=**
> >>> 1328517&r2=1328518&view=diff<
> http://svn.apache.org/viewvc/ofbiz/branches/release11.04/framework/base/config/test-containers.xml?rev=1328518&r1=1328517&r2=1328518&view=diff
> >
> >>> ==============================**==============================**
> >>> ==================
> >>> ---
> ofbiz/branches/release11.04/**framework/base/config/test-**containers.xml
> >>> (original)
> >>> +++
> ofbiz/branches/release11.04/**framework/base/config/test-**containers.xml
> >>> Fri Apr 20 21:35:38 2012
> >>> @@ -31,14 +31,14 @@ under the License.
> >>>
> >>>   <!-- load the naming (JNDI) server -->
> >>>   <container name="naming-container"
> class="org.ofbiz.base.**container.*
> >>> *NamingServiceContainer">
> >>> -        <property name="port" value="1099"/>
> >>> +        <property name="port" value="1100"/>
> >>>   </container>
> >>>
> >>>   <!-- RMI Service Dispatcher -->
> >>>   <container name="rmi-dispatcher" class="org.ofbiz.service.rmi.**
> >>> RmiServiceContainer">
> >>>       <property name="bound-name" value="RMIDispatcher"/>
> >>>       <property name="bound-host" value="127.0.0.1"/>
> >>> -        <property name="bound-port" value="1099"/>
> >>> +        <property name="bound-port" value="1100"/>
> >>>       <property name="delegator-name" value="test"/>
> >>>       <property name="client-factory" value="org.ofbiz.service.rmi.**
> >>> socket.ssl.**SSLClientSocketFactory"/>
> >>>       <property name="server-factory" value="org.ofbiz.service.rmi.**
> >>> socket.ssl.**SSLServerSocketFactory"/>
> >>> @@ -127,7 +127,7 @@ under the License.
> >>>           <property name="maxSpareThreads" value="50"/>
> >>>           <property name="maxThreads" value="200"/>
> >>>           <property name="minSpareThreads" value="4"/>
> >>> -            <property name="port" value="8009"/>
> >>> +            <property name="port" value="8010"/>
> >>>           <property name="tcpNoDelay" value="true"/>
> >>>           <property name="soTimeout" value="60000"/>
> >>>           <property name="tomcatAuthentication" value="true"/>
> >>> @@ -162,7 +162,7 @@ under the License.
> >>>           <property name="maxThreads" value="100"/>
> >>>           <property name="minSpareThreads" value="4"/>
> >>>           <property name="noCompressionUserAgents" value=""/>
> >>> -            <property name="port" value="8080"/>
> >>> +            <property name="port" value="8081"/>
> >>>           <property name="restrictedUserAgents" value=""/>
> >>>           <property name="server" value=""/>
> >>>           <property name="socketBuffer" value="9000"/>
> >>> @@ -200,7 +200,7 @@ under the License.
> >>>           <property name="maxThreads" value="100"/>
> >>>           <property name="minSpareThreads" value="4"/>
> >>>           <property name="noCompressionUserAgents" value=""/>
> >>> -            <property name="port" value="8443"/>
> >>> +            <property name="port" value="8444"/>
> >>>           <property name="restrictedUserAgents" value=""/>
> >>>           <property name="server" value=""/>
> >>>           <property name="socketBuffer" value="9000"/>
> >>>
> >>>
> >>>
>
>

Reply via email to