I'm sorry if my first point and the use of capitals for emphasis offended you but I think you forget to actually respond to the main point, the configurability of a configuration file...
Regards Scott On 22/04/2012, at 12:32 AM, Pierre Smits wrote: > 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"/> >>>>> >>>>> >>>>> >> >>
