You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by "Peter M. Goldstein" <pe...@yahoo.com> on 2002/07/27 21:07:17 UTC

[PATCH] Bug #11233

All,

The attached files constitute a patch to James to restore thread safety
to its date formatting, addressing the issues discussed in bug #11233.

This is the second revision of these changes, spurred mostly by
Harmeet's response to the first, simpler revision.  Please see my email
of 7/2/2002 to this list with the subject "RE: Possible Synchronization
Issue" for further details of that discussion.

Please note that bug #8774 may be related to this issue.

--Peter

[PATCH] Bug #6812 (take 2)

Posted by "Peter M. Goldstein" <pe...@yahoo.com>.
All,

Sorry.  I inadvertently introduced a bug in the absolute URL handling in
the previous revision of this fix by stripping off the leading '/'.
That'll cause a problem on non-Windows systems.  Here's a corrected fix.

--Peter

-----Original Message-----
From: Peter M. Goldstein [mailto:peter_m_goldstein@yahoo.com] 
Sent: Saturday, July 27, 2002 3:07 PM
To: 'James Developers List'
Subject: [PATCH] Bug #6812



All,

The attached files are a patch for bug #6812, improper handling of
absolute file URLs.  Basically they consist of a check to see whether
the URL starts with '/' after the initial file:// protocol prefix.  If
so, the initial '/' is stripped off and the resulting string is taken as
the file path.  If not, the base directory obtained from the block
context is prepended to the string to obtain the file path.

--Peter 

RE: [PATCH] Bug #6812

Posted by Danny Angus <da...@apache.org>.
What so a *nix path has to be file:////usr/?
I think file:///usr is what we want.


> -----Original Message-----
> From: Peter M. Goldstein [mailto:peter_m_goldstein@yahoo.com]
> Sent: 27 July 2002 23:07
> To: 'James Developers List'
> Subject: [PATCH] Bug #6812
> 
> 
> 
> 
> All,
> 
> The attached files are a patch for bug #6812, improper handling of
> absolute file URLs.  Basically they consist of a check to see whether
> the URL starts with '/' after the initial file:// protocol prefix.  If
> so, the initial '/' is stripped off and the resulting string is taken as
> the file path.  If not, the base directory obtained from the block
> context is prepended to the string to obtain the file path.
> 
> --Peter 
> 

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>


[PATCH] Bug #6812

Posted by "Peter M. Goldstein" <pe...@yahoo.com>.

All,

The attached files are a patch for bug #6812, improper handling of
absolute file URLs.  Basically they consist of a check to see whether
the URL starts with '/' after the initial file:// protocol prefix.  If
so, the initial '/' is stripped off and the resulting string is taken as
the file path.  If not, the base directory obtained from the block
context is prepended to the string to obtain the file path.

--Peter 

RE: [PATCH] Bug #11233

Posted by Danny Angus <da...@apache.org>.
Thanks Peter, I'll get these commited asap.

> -----Original Message-----
> From: Peter M. Goldstein [mailto:peter_m_goldstein@yahoo.com]
> Sent: 27 July 2002 20:07
> To: 'James Developers List'
> Subject: [PATCH] Bug #11233
> 
> 
> 
> All,
> 
> The attached files constitute a patch to James to restore thread safety
> to its date formatting, addressing the issues discussed in bug #11233.
> 
> This is the second revision of these changes, spurred mostly by
> Harmeet's response to the first, simpler revision.  Please see my email
> of 7/2/2002 to this list with the subject "RE: Possible Synchronization
> Issue" for further details of that discussion.
> 
> Please note that bug #8774 may be related to this issue.
> 
> --Peter
> 

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>