svn commit: r371866 - /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Response.java

2006-01-24 Thread remm
Author: remm
Date: Tue Jan 24 00:52:54 2006
New Revision: 371866

URL: http://svn.apache.org/viewcvs?rev=371866view=rev
Log:
- Restore useless and inefficient code.

Modified:

tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Response.java

Modified: 
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Response.java
URL: 
http://svn.apache.org/viewcvs/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Response.java?rev=371866r1=371865r2=371866view=diff
==
--- 
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Response.java
 (original)
+++ 
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Response.java
 Tue Jan 24 00:52:54 2006
@@ -599,6 +599,20 @@
 throw new IllegalStateException
 (sm.getString(coyoteResponse.getWriter.ise));
 
+/*
+ * If the response's character encoding has not been specified as
+ * described in codegetCharacterEncoding/code (i.e., the method
+ * just returns the default value codeISO-8859-1/code),
+ * codegetWriter/code updates it to codeISO-8859-1/code
+ * (with the effect that a subsequent call to getContentType() will
+ * include a charset=ISO-8859-1 component which will also be
+ * reflected in the Content-Type response header, thereby satisfying
+ * the Servlet spec requirement that containers must communicate the
+ * character encoding used for the servlet response's writer to the
+ * client).
+ */
+setCharacterEncoding(getCharacterEncoding());
+
 usingWriter = true;
 outputBuffer.checkConverter();
 if (writer == null) {



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 37356] - Tomcat does not invalidate sessions after session-timeout period has passed.

2006-01-24 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=37356.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=37356





--- Additional Comments From [EMAIL PROTECTED]  2006-01-24 12:59 ---
I have found one problem about the accessCount. 

In some times, it will not decrease to 0 after one request. So the session 
manger will never invalidate it.

The Tomcat version is 5.0.19.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 38367] New: - Executing any Catalina Ant task results in an exception

2006-01-24 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38367.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38367

   Summary: Executing any Catalina Ant task results in an exception
   Product: Tomcat 5
   Version: 5.5.14
  Platform: PC
OS/Version: Windows XP
Status: NEW
  Severity: normal
  Priority: P2
 Component: Unknown
AssignedTo: tomcat-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


which in turn breaks the build process.

Steps to Reproduce: Try executing any of the following targets where 

target name=tomcat.application.deploy description=Install Web application
deploy url=${tomcat.management.url}
username=${tomcat.management.username}
password=${tomcat.management.password}
path=${web.context.path} war=${web.war.file}/
/target


target name=tomcat.application.reload description=Reload Web 
application
reload url=${tomcat.management.url}
username=${tomcat.management.username}
password=${tomcat.management.password}
path=${web.context.path}/
/target

target name=tomcat.application.undeploy description=Remove Web 
application
undeploy url=${tomcat.management.url}
  username=${tomcat.management.username}
  password=${tomcat.management.password}
  path=${web.context.path}/
/target

target name=tomcat.application.start description=Start Tomcat 
application.
start url=${tomcat.management.url}
   username=${tomcat.management.username}
   password=${tomcat.management.password}
   path=${web.context.path}/
/target

target name=tomcat.application.stop description=Stop Tomcat 
application.
stop url=${tomcat.management.url}
  username=${tomcat.management.username}
  password=${tomcat.management.password}
  path=${web.context.path}/
/target 
target name=tomcat.applications.list description=List Tomcat applications.
list url=${tomcat.management.url}
  username=${tomcat.management.username}
  password=${tomcat.management.password}/
/target

Actual Results: After executing the task ant throws an exception. Last part of
the description is the exact ant output (the lengthy html doc was removed from
the output) 
 

Expected Results: There shouldn't be any exceptions thrown!

Build Date  Platform: 5.5.15 (Here I could just select 5.5.14!)

Additional Information: 

I imported the catalina-tasks.xml in my tomcat.xml that defines the above 
targets.

Ant - version 1.6.5
JDK - version 1.5.0_06
IDE - occurs with both netbeans (5RC1) and intellij(5.0.2). 


Likely Solution:

The ant tasks just call the urls.. which in turn yield a complete html page..
make sure that when the ant tasks call the urls the result is not an html
document but something suitable and concise.


A faulty run with ant -verbose(I snipped out the html document part of the 
output):
Detected Java version: 1.5 in: D:\opt\Java\jdk1.5.0_06\jre
Detected OS: Windows XP
parsing buildfile
jar:file:/D:/opt/netbeans/ide6/ant/nblib/org-netbeans-modules-debugger-jpda-ant.jar!/org/netbeans/modules/debugger/jpda/ant/antlib.xml
with URI =
jar:file:/D:/opt/netbeans/ide6/ant/nblib/org-netbeans-modules-debugger-jpda-ant.jar!/org/netbeans/modules/debugger/jpda/ant/antlib.xml
parsing buildfile
jar:file:/D:/opt/netbeans/ide6/ant/nblib/org-netbeans-modules-debugger-jpda-ant.jar!/org/netbeans/modules/debugger/jpda/ant/antlib.xml
with URI =
jar:file:/D:/opt/netbeans/ide6/ant/nblib/org-netbeans-modules-debugger-jpda-ant.jar!/org/netbeans/modules/debugger/jpda/ant/antlib.xml
parsing buildfile
jar:file:/D:/opt/netbeans/ide6/ant/nblib/org-netbeans-modules-ant-browsetask.jar!/org/netbeans/modules/ant/browsetask/antlib.xml
with URI =
jar:file:/D:/opt/netbeans/ide6/ant/nblib/org-netbeans-modules-ant-browsetask.jar!/org/netbeans/modules/ant/browsetask/antlib.xml
parsing buildfile
jar:file:/D:/opt/netbeans/ide6/ant/nblib/org-netbeans-modules-ant-browsetask.jar!/org/netbeans/modules/ant/browsetask/antlib.xml
with URI =
jar:file:/D:/opt/netbeans/ide6/ant/nblib/org-netbeans-modules-ant-browsetask.jar!/org/netbeans/modules/ant/browsetask/antlib.xml
parsing buildfile
jar:file:/D:/opt/netbeans/profiler1/ant/nblib/org-netbeans-modules-profiler.jar!/org/netbeans/modules/profiler/antlib.xml
with URI =
jar:file:/D:/opt/netbeans/profiler1/ant/nblib/org-netbeans-modules-profiler.jar!/org/netbeans/modules/profiler/antlib.xml
parsing buildfile
jar:file:/D:/opt/netbeans/profiler1/ant/nblib/org-netbeans-modules-profiler.jar!/org/netbeans/modules/profiler/antlib.xml
with URI =

Re: Nightly builds broken

2006-01-24 Thread Jeanfrancois Arcand

Hi Yoav,

Yoav Shapira wrote:

Hi,
A user pointed out to me that our nightly builds appear to be broken. 
First I assumed it was due Gump unhappiness and I thought Mark T and

Bill B fixed it over the past few days.  But last night's builds still
look whacky: http://cvs.apache.org/builds/jakarta-tomcat-5/nightly/. 
The binaries are 45 bytes in size and invalid, the sources are about

200MB in size (Tomcat 5.5.15 correct source distros are a little less
than 6MB).

Also as an aside, all the Gump files still call it jakarta-tomcat-*
rather than apache-tomcat-* or just tomcat-*.  I'm not a Gump expert,
but I believe the Gump project file we want to update is (at least)
http://svn.apache.org/repos/asf/gump/metadata/project/jakarta-tomcat-catalina.xml.


I've posted here a long time ago asking if we still want to have nightly 
or not, and got no response, So I stopped the script since it was still 
using CVS. Do we want nightly build? If yes, then I will fix the script 
and re-add it to the cron job.


-- Jeanfrancois




Thoughts?

--
Yoav Shapira
System Design and Management Fellow
MIT Sloan School of Management
Cambridge, MA, USA
[EMAIL PROTECTED] / www.yoavshapira.com

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Nightly builds broken

2006-01-24 Thread Peter Rossbach

+1
to have nightly builds!

Regards
Peter



Am 24.01.2006 um 16:52 schrieb Jeanfrancois Arcand:


Hi Yoav,

Yoav Shapira wrote:

Hi,
A user pointed out to me that our nightly builds appear to be  
broken. First I assumed it was due Gump unhappiness and I thought  
Mark T and
Bill B fixed it over the past few days.  But last night's builds  
still
look whacky: http://cvs.apache.org/builds/jakarta-tomcat-5/ 
nightly/. The binaries are 45 bytes in size and invalid, the  
sources are about

200MB in size (Tomcat 5.5.15 correct source distros are a little less
than 6MB).
Also as an aside, all the Gump files still call it jakarta-tomcat-*
rather than apache-tomcat-* or just tomcat-*.  I'm not a Gump expert,
but I believe the Gump project file we want to update is (at least)
http://svn.apache.org/repos/asf/gump/metadata/project/jakarta- 
tomcat-catalina.xml.


I've posted here a long time ago asking if we still want to have  
nightly or not, and got no response, So I stopped the script since  
it was still using CVS. Do we want nightly build? If yes, then I  
will fix the script and re-add it to the cron job.


-- Jeanfrancois



Thoughts?
--
Yoav Shapira
System Design and Management Fellow
MIT Sloan School of Management
Cambridge, MA, USA
[EMAIL PROTECTED] / www.yoavshapira.com
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]





-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Nightly builds broken

2006-01-24 Thread Yoav Shapira
Hi,
Mmm, I'm neutral on nightly builds.  They can lead to more trouble
than it's worth:

- Users using a nightly build of unknown stability instead of an
actual release, then asking for support
- Extra load on infrastructure
- Extra load on Gump people to keep the relevant scripts and Gumpy
files up to date

And for what?  This user was the first request I've seen in a long
long time for nightly builds, and he didn't even want the whole build,
just the jsp examples webapp containing the XSS fix.  So I don't think
they're used that much, don't think they add much value.  Anyone doing
testing, be it individuals or organizations like JBoss, Covalent,
Spikesource, whatever, isn't going to test, much less ship, off a
nightly build.  And if someone really wants a build at any time, it's
easy (and documented) to build individually on the client side using
our existing build file and its download target.

Actually, maybe I'm closer to -1 than 0 on nightly builds ;)

Yoav

On 1/24/06, Peter Rossbach [EMAIL PROTECTED] wrote:
 +1
 to have nightly builds!

 Regards
 Peter



 Am 24.01.2006 um 16:52 schrieb Jeanfrancois Arcand:

  Hi Yoav,
 
  Yoav Shapira wrote:
  Hi,
  A user pointed out to me that our nightly builds appear to be
  broken. First I assumed it was due Gump unhappiness and I thought
  Mark T and
  Bill B fixed it over the past few days.  But last night's builds
  still
  look whacky: http://cvs.apache.org/builds/jakarta-tomcat-5/
  nightly/. The binaries are 45 bytes in size and invalid, the
  sources are about
  200MB in size (Tomcat 5.5.15 correct source distros are a little less
  than 6MB).
  Also as an aside, all the Gump files still call it jakarta-tomcat-*
  rather than apache-tomcat-* or just tomcat-*.  I'm not a Gump expert,
  but I believe the Gump project file we want to update is (at least)
  http://svn.apache.org/repos/asf/gump/metadata/project/jakarta-
  tomcat-catalina.xml.
 
  I've posted here a long time ago asking if we still want to have
  nightly or not, and got no response, So I stopped the script since
  it was still using CVS. Do we want nightly build? If yes, then I
  will fix the script and re-add it to the cron job.
 
  -- Jeanfrancois
 
 
  Thoughts?
  --
  Yoav Shapira
  System Design and Management Fellow
  MIT Sloan School of Management
  Cambridge, MA, USA
  [EMAIL PROTECTED] / www.yoavshapira.com
  -
  To unsubscribe, e-mail: [EMAIL PROTECTED]
  For additional commands, e-mail: [EMAIL PROTECTED]
 
  -
  To unsubscribe, e-mail: [EMAIL PROTECTED]
  For additional commands, e-mail: [EMAIL PROTECTED]
 
 


 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




--
Yoav Shapira
System Design and Management Fellow
MIT Sloan School of Management
Cambridge, MA, USA
[EMAIL PROTECTED] / www.yoavshapira.com

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Build Problems

2006-01-24 Thread Marsh David W Maj AFIT/ENG
 log.txt I'm ready to poke around the code and, following the build
directions (and many variations!) at
http://tomcat.apache.org/tomcat-5.5-doc/building.html, am still unable
to get the code downloaded and compiled.  I'm on an XP box behind a
firewall.  The verbose log is attached.  Assistance greatly appreciated!

David
Detected Java version: 1.5 in: C:\Program Files\Java\jdk1.5.0_02\jre
Detected OS: Windows XP
parsing buildfile D:\tomcatsource\build.xml with URI = 
file:///D:/tomcatsource/build.xml
Project base dir set to: D:\tomcatsource
 [property] Loading D:\Documents and Settings\dmarsh\build.properties
 [property] Loading D:\tomcatsource\build.properties
 [property] Unable to find property file: D:\tomcatsource\build.properties
 [property] Loading D:\tomcatsource\build.properties.default
Override ignored for property version
Build sequence for target(s) `build' is [check.source, get.source, build]
Complete build sequence is [check.source, get.source, build, checkout, clean, ]

check.source:
[available] Unable to find dir build to set property source.exists

get.source:
Project base dir set to: D:\tomcatsource
  [antcall] calling target(s) [checkout] in build file D:\tomcatsource\build.xml
parsing buildfile D:\tomcatsource\build.xml with URI = 
file:///D:/tomcatsource/build.xml
Project base dir set to: D:\tomcatsource
 [property] Loading D:\Documents and Settings\dmarsh\build.properties
Override ignored for property base.path
Override ignored for property proxy.host
Override ignored for property proxy.use
Override ignored for property proxy.port
 [property] Loading D:\tomcatsource\build.properties
 [property] Unable to find property file: D:\tomcatsource\build.properties
 [property] Loading D:\tomcatsource\build.properties.default
Override ignored for property commons-httpclient.home
Override ignored for property commons-launcher.bin
Override ignored for property commons-logging.jar
Override ignored for property jmx-remote.jar
Override ignored for property rhino.home
Override ignored for property commons-launcher.jar
Override ignored for property jdt.lib
Override ignored for property commons-el.lib
Override ignored for property jsse.lib
Override ignored for property activation.lib
Override ignored for property jdt.loc
Override ignored for property commons-el.loc
Override ignored for property activation.home
Override ignored for property commons-el.home
Override ignored for property jsp-api.jar
Override ignored for property jsp-api.home
Override ignored for property commons-modeler.jar
Override ignored for property jmx.home
Override ignored for property jdt.home
Override ignored for property jta.lib
Override ignored for property jmx-tools.jar
Override ignored for property commons-fileupload.home
Override ignored for property struts.lib
Override ignored for property mail.jar
Override ignored for property struts.loc
Override ignored for property commons-logging.home
Override ignored for property commons-digester.home
Override ignored for property commons-collections.home
Override ignored for property commons-dbcp.version
Override ignored for property jta.home
Override ignored for property servlet-api.home
Override ignored for property commons-fileupload.jar
Override ignored for property commons-beanutils.lib
Override ignored for property commons-collections.lib
Override ignored for property commons-launcher.bootstrap.class
Override ignored for property nsis.installoptions.dll
Override ignored for property servlet-api.lib
Override ignored for property commons-beanutils.loc
Override ignored for property commons-collections.loc
Override ignored for property version
Override ignored for property jcert.jar
Override ignored for property commons-daemon.lib
Override ignored for property compile.optimize
Override ignored for property jnet.jar
Override ignored for property commons-dbcp-src.loc
Override ignored for property jdt.jar
Override ignored for property commons-el.jar
Override ignored for property commons-daemon.loc
Override ignored for property puretls.lib
Override ignored for property jsse.jar
Override ignored for property activation.jar
Override ignored for property struts.home
Override ignored for property commons-httpclient.lib
Override ignored for property jta.jar
Override ignored for property commons-httpclient.loc
Override ignored for property base-xml.loc
Override ignored for property struts.jar
Override ignored for property base-logging.loc
Override ignored for property junit.lib
Override ignored for property log4j.lib
Override ignored for property junit.loc
Override ignored for property commons-daemon.home
Override ignored for property base-tomcat.loc
Override ignored for property log4j.loc
Override ignored for property version.major
Override ignored for property xercesImpl.jar
Override ignored for property xml-apis.jar
Override ignored for property commons-pool.home
Override ignored for property nsis.exe
Override ignored for property commons-beanutils.jar
Override ignored for property 

Re: Build Problems

2006-01-24 Thread Keith Wannamaker
Hi David, in order to build tomcat from svn, you need svn installed and 
on the path.

Thanks,
Keith


Marsh David W Maj AFIT/ENG wrote:

java.io.IOException: CreateProcess: svn checkout 
http://svn.apache.org/repos/asf//tomcat/current/tc5.5.x D:\tomcatsource error=2
at java.lang.ProcessImpl.create(Native Method)



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



RE: Build Problems

2006-01-24 Thread Marsh David W Maj AFIT/ENG
Sorry.  Yeah, it's on the path.  Relevant feedback:

checkout:
 [echo] If the checkout fails, - todo - 
 [exec] Current OS is Windows XP
 [exec] Executing 'svn' with arguments:
 [exec] 'checkout'
 [exec] 'http://svn.apache.org/repos/asf//tomcat/current/tc5.5.x'
 [exec] 'D:\tomcat.source'
 [exec] 
 [exec] The ' characters around the executable and arguments are
 [exec] not part of the command.
 [exec] svn: PROPFIND request failed on
'/repos/asf/tomcat/current/tc5.5.x'
 [exec] svn: PROPFIND of '/repos/asf/tomcat/current/tc5.5.x': 501
Not Implemented (http://svn.apache.org)
 [exec] Result: 1
  [antcall] Exiting D:\tomcat.source\build.xml.

Thanks.
David
-Original Message-
From: Keith Wannamaker [mailto:[EMAIL PROTECTED] 
Sent: Tuesday, January 24, 2006 11:55 AM
To: Tomcat Developers List
Subject: Re: Build Problems

Hi David, in order to build tomcat from svn, you need svn installed and
on the path.
Thanks,
Keith


Marsh David W Maj AFIT/ENG wrote:
 java.io.IOException: CreateProcess: svn checkout
http://svn.apache.org/repos/asf//tomcat/current/tc5.5.x D:\tomcatsource
error=2
   at java.lang.ProcessImpl.create(Native Method)


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



RE: svn commit: r371765 - /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Response.java

2006-01-24 Thread Bill Barker
 

 -Original Message-
 From: Remy Maucherat [mailto:[EMAIL PROTECTED] 
 Sent: Tuesday, January 24, 2006 12:52 AM
 To: Tomcat Developers List
 Subject: Re: svn commit: r371765 - 
 /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catali
 na/connector/Response.java
 
 Bill Barker wrote:
  Author: remm
  Date: Mon Jan 23 17:13:19 2006
  New Revision: 371765
 
  URL: http://svn.apache.org/viewcvs?rev=371765view=rev
  Log:
  - Remove nonsensical systematic inclusion on ISO-8859-1 
  charset in the content type, which is noth
useless and inefficient.
 
  
  -1
  Sending the charset used by the Writer is very clearly 
 required by the
  servlet spec. 
 
 Thanks, I expected no less coming from you :) I will revert my patch.
 
 A couple questions for your enjoyment:
 1) Is this relevant or irrelevant from the HTTP specification 
 perspective ?

It's relevant to the browser trying to display the code.  If you've
configured your browser's default encoding to EUC-JP, without the charset
you'll see a big mess when you hit a latin-1 page ;-).

 2) Does this mean we're running the following ultra efficient code (I 
 don't even know why I accepted this stuff back then. It must 
 have been 
 that this has been done gradually through many many commits) for each 
 request that uses a writer ?
 

Yup, that's what it means :).  I'm sure you've played the blame-game by now,
and I'm not interested enough to do it myself.  It looks like it's trying to
avoid computing the entire header value each time the characterEncoding
changes.

  public String getContentType() {
 
  String ret = contentType;
 
  if (ret != null
   characterEncoding != null
   charsetSet) {
  ret = ret + ;charset= + characterEncoding;
  }
 
  return ret;
  }
 
 Rémy
 
 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 
 
 



This message is intended only for the use of the person(s) listed above as the 
intended recipient(s), and may contain information that is PRIVILEGED and 
CONFIDENTIAL.  If you are not an intended recipient, you may not read, copy, or 
distribute this message or any attachment. If you received this communication 
in error, please notify us immediately by e-mail and then delete all copies of 
this message and any attachments.

In addition you should be aware that ordinary (unencrypted) e-mail sent through 
the Internet is not secure. Do not send confidential or sensitive information, 
such as social security numbers, account numbers, personal identification 
numbers and passwords, to us via ordinary (unencrypted) e-mail.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



RE: Nightly builds broken

2006-01-24 Thread Bill Barker
 

 -Original Message-
 From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On 
 Behalf Of Yoav Shapira
 Sent: Tuesday, January 24, 2006 7:49 AM
 To: Tomcat-dev
 Subject: Nightly builds broken
 
 Hi,
 A user pointed out to me that our nightly builds appear to be broken. 
 First I assumed it was due Gump unhappiness and I thought Mark T and
 Bill B fixed it over the past few days.  But last night's builds still
 look whacky: http://cvs.apache.org/builds/jakarta-tomcat-5/nightly/. 
 The binaries are 45 bytes in size and invalid, the sources are about
 200MB in size (Tomcat 5.5.15 correct source distros are a little less
 than 6MB).

Actually, Gump runs (or at least the one that nags) on vmgump.apache.org
(http://vmgump.apache.org/gump/public/index.html).  Also, Gump hasn't
published the jars it builds for quite some time (to avoid licensing
issues).

Also Gump builds aren't true nightlies, since they build everything against
the HEAD of everything else, not against the version specified in
build.properties.default.

 
 Also as an aside, all the Gump files still call it jakarta-tomcat-*
 rather than apache-tomcat-* or just tomcat-*.  I'm not a Gump expert,
 but I believe the Gump project file we want to update is (at least)
 http://svn.apache.org/repos/asf/gump/metadata/project/jakarta-
 tomcat-catalina.xml.
 

Actually the closest to a nightly would be jakarta-tomcat-5.xml, but that
one hasn't built in about 6 months (currently because of axis, before that
xdoclet).  The jakarta-tomcat-catalina one is only the 5.5 container part of
the build.

I was too lazy to change the gump project names on the svn move :).  There
are about seven jakarta-tomcat*.xml project files, and three
jakarta-servlet-api*.xml files that would need to be moved.  You'd also need
to change the reference to the files in profile/gump.xml, and track down all
of the other projects that depend on them (e.g. 67 for jakarta-servletapi-4)
and change it there as well.

All ASF committers have karma for the Gump metadata, so if anybody feels
strongly about the names, knock yourself out :).

 Thoughts?
 
 --
 Yoav Shapira
 System Design and Management Fellow
 MIT Sloan School of Management
 Cambridge, MA, USA
 [EMAIL PROTECTED] / www.yoavshapira.com
 
 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 
 
 



This message is intended only for the use of the person(s) listed above as the 
intended recipient(s), and may contain information that is PRIVILEGED and 
CONFIDENTIAL.  If you are not an intended recipient, you may not read, copy, or 
distribute this message or any attachment. If you received this communication 
in error, please notify us immediately by e-mail and then delete all copies of 
this message and any attachments.

In addition you should be aware that ordinary (unencrypted) e-mail sent through 
the Internet is not secure. Do not send confidential or sensitive information, 
such as social security numbers, account numbers, personal identification 
numbers and passwords, to us via ordinary (unencrypted) e-mail.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: svn commit: r371765 - /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Response.java

2006-01-24 Thread Remy Maucherat

Bill Barker wrote:

It's relevant to the browser trying to display the code.  If you've
configured your browser's default encoding to EUC-JP, without the charset
you'll see a big mess when you hit a latin-1 page ;-).


Obviously, this would only impact the case where ;charset=ISO-8859-1 
would be forcefully added to the content-type header for no good reason 
when the user didn't specify any. This is the HTTP default encoding, and 
will not change the behavior from the user perspective.



Yup, that's what it means :).  I'm sure you've played the blame-game by now,
and I'm not interested enough to do it myself.  It looks like it's trying to
avoid computing the entire header value each time the characterEncoding
changes.


This whole thing is a huge mess right now. Hopefully, it's doing what it 
should. You can also for example compare 
o.a.c.connector.Response.setContentType with 
o.a.coyote.Response.setContentType. I have to suppose substring and 
concatenation is a very cool activity.


Rémy

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Purpose of the SVN /bin directory?

2006-01-24 Thread Yoav Shapira
Hi,
It's an intermediate staging area for some compiled files.  It's not
part of the SVN repository, as you can see at
http://svn.apache.org/viewcvs.cgi/tomcat/.  It gets generated during
the build and cleaned up as part of the clean target.

Yoav

On 1/24/06, Glen Mazza [EMAIL PROTECTED] wrote:
 Hello,

 I've downloaded Tomcat off of SVN but am unsure what the tomcat/bin/
 directory indicates, the svn page[1] doesn't describe it.  It appears to
 be a bunch of compiled .class files--may I ask what they're for, and why
 they are in SVN?

 Thanks,
 Glen

 [1] http://tomcat.apache.org/svn.html


 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




--
Yoav Shapira
System Design and Management Fellow
MIT Sloan School of Management
Cambridge, MA, USA
[EMAIL PROTECTED] / www.yoavshapira.com

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 38373] - [PATCH] Patch to change to ASF logo on FAQ pages

2006-01-24 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38373.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38373





--- Additional Comments From [EMAIL PROTECTED]  2006-01-25 03:33 ---
Created an attachment (id=17496)
 -- (http://issues.apache.org/bugzilla/attachment.cgi?id=17496action=view)
Patch to switch from Jakarta to ASF logo on FAQ pages


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 38376] New: - Body content stack may not be properly maintained in the faces of exceptions

2006-01-24 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38376.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38376

   Summary: Body content stack may not be properly maintained in the
faces of exceptions
   Product: Tomcat 5
   Version: 5.5.14
  Platform: All
OS/Version: other
Status: NEW
  Severity: normal
  Priority: P2
 Component: Jasper
AssignedTo: tomcat-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


The code that increments _jspx_push_body_count_XXX[0], which is the count of the
number of outstanding calls to _jspx_page_context.pushBody() within a try block,
is properly within the conditional that is true only if doStartTag() returns
EVAL_BODY_INCLUDE. The corresponding code that decrements the same variable,
however, is not within the corresponding conditional, although the call to
_jspx_page_context.popBody() is. The outstanding count may therefore be wrong,
and the code that pops these extra BodyContents in the finally block pops too
few. In the following Jasper-generated code snippet, note that
_jspx_push_body_count_rwc_dbTry_0[0]++ on the third line is conditional, but
_jspx_push_body_count_rwc_dbTry_0[0]-- on the last line is not.

Jasper-generated code snippet:

if (_jspx_eval_rwc_formPhase_0 != 
javax.servlet.jsp.tagext.Tag.EVAL_BODY_INCLUDE) {
out = _jspx_page_context.pushBody();
_jspx_push_body_count_rwc_dbTry_0[0]++;
_jspx_th_rwc_formPhase_0.setBodyContent((javax.servlet.jsp.tagext.BodyContent) 
out);
_jspx_th_rwc_formPhase_0.doInitBody();
}
do {
...

int evalDoAfterBody = _jspx_th_rwc_formPhase_0.doAfterBody();
if (evalDoAfterBody != javax.servlet.jsp.tagext.BodyTag.EVAL_BODY_AGAIN)
break;
} while (true);
if (_jspx_eval_rwc_formPhase_0 != 
javax.servlet.jsp.tagext.Tag.EVAL_BODY_INCLUDE)
out = _jspx_page_context.popBody();
_jspx_push_body_count_rwc_dbTry_0[0]--;  // THIS SHOULD BE IN THE IF BLOCK!

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 38376] - Body content stack may not be properly maintained in the faces of exceptions

2006-01-24 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38376.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38376





--- Additional Comments From [EMAIL PROTECTED]  2006-01-25 05:50 ---
Created an attachment (id=17500)
 -- (http://issues.apache.org/bugzilla/attachment.cgi?id=17500action=view)
Change to Generator.java


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: svn commit: r371765 - /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Response.java

2006-01-24 Thread Bill Barker

Remy Maucherat [EMAIL PROTECTED] wrote in message 
news:[EMAIL PROTECTED]
 Bill Barker wrote:
 It's relevant to the browser trying to display the code.  If you've
 configured your browser's default encoding to EUC-JP, without the charset
 you'll see a big mess when you hit a latin-1 page ;-).

 Obviously, this would only impact the case where ;charset=ISO-8859-1 would 
 be forcefully added to the content-type header for no good reason when the 
 user didn't specify any. This is the HTTP default encoding, and will not 
 change the behavior from the user perspective.


Yes, RFC 2616 does specify iso-latin-1 as the default for HTTP/1.1 clients. 
However, section 3.4.1 is also relevant for HTTP/1.0 clients (like, say, the 
TCK :).  In any case, it doesn't matter since section 5.4 of the servlet 
spec says must.  Complaints go to the expert group;  here we just develop 
Tomcat.

 Yup, that's what it means :).  I'm sure you've played the blame-game by 
 now,
 and I'm not interested enough to do it myself.  It looks like it's trying 
 to
 avoid computing the entire header value each time the characterEncoding
 changes.

 This whole thing is a huge mess right now. Hopefully, it's doing what it 
 should. You can also for example compare 
 o.a.c.connector.Response.setContentType with 
 o.a.coyote.Response.setContentType. I have to suppose substring and 
 concatenation is a very cool activity.


Yeah, the spec is a mess wrt characterEncoding.  Complaints to the same 
place as above :).  The problem is that we need to deal with such 
pathological cases as:
   response.setContentType(text/html; charset=EUC-JP);
   // Oops, I want French instead of Japanese
   response.setCharacterEncoding(iso-8859-1);

Since you can change your mind (according to the spec) many times before you 
actually grab the Writer, I don't really see a way around substring and 
concatenation being cool :).  Of course, I would love to be proven wrong :).

 Rémy 




-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]