You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2009/06/19 17:00:18 UTC

svn commit: r786530 - /tomcat/current/tc5.5.x/STATUS.txt

Author: markt
Date: Fri Jun 19 15:00:18 2009
New Revision: 786530

URL: http://svn.apache.org/viewvc?rev=786530&view=rev
Log:
Votes

Modified:
    tomcat/current/tc5.5.x/STATUS.txt

Modified: tomcat/current/tc5.5.x/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/current/tc5.5.x/STATUS.txt?rev=786530&r1=786529&r2=786530&view=diff
==============================================================================
--- tomcat/current/tc5.5.x/STATUS.txt (original)
+++ tomcat/current/tc5.5.x/STATUS.txt Fri Jun 19 15:00:18 2009
@@ -97,7 +97,7 @@
   For zip/tgz distributives it was already fixed in 6.0.17, 5.5.27.
   Now, apply the same fix to exe distributive.
   http://svn.apache.org/viewvc?rev=776945&view=rev
-  +1: kkolinko, fhanik
+  +1: kkolinko, fhanik, markt
   -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43343
@@ -111,14 +111,14 @@
 * Additional patches:
   http://svn.apache.org/viewvc?rev=784453&view=rev
   http://svn.apache.org/viewvc?rev=784602&view=rev
-  +1: kkolinko
+  +1: kkolinko, markt
   -1:
 
 * Fix wrong download location of tcnative 1.1.16
   Patch for "build/build.properties.default":
   http://people.apache.org/~kkolinko/patches/2009-05-25_fix_tcnative_download.patch
   +1: kkolinko, fhanik
-  -1:
+  -1: markt - the updated location is still wrong
 
 * Make 64-bit binaries/installer available for 5.5.x as well as 6.0.x
   http://svn.apache.org/viewvc?rev=777624&view=rev 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Re: svn commit: r786530 - /tomcat/current/tc5.5.x/STATUS.txt

Posted by Konstantin Kolinko <kn...@gmail.com>.
2009/6/19 Mark Thomas <ma...@apache.org>:
> Konstantin Kolinko wrote:
>>>  * Fix wrong download location of tcnative 1.1.16
>>>   Patch for "build/build.properties.default":
>>>   http://people.apache.org/~kkolinko/patches/2009-05-25_fix_tcnative_download.patch
>>>   +1: kkolinko, fhanik
>>> -  -1:
>>> +  -1: markt - the updated location is still wrong
>>
>> It points to
>> http://tomcat.apache.org/dev/dist/tomcat-connectors/native/source/1.1.16/tomcat-native-1.1.16-src.tar.gz
>
> That is a really bad place for it to point at. That is the location for
> the test builds rather than releases.
>
>> because
>> base-tomcat.loc=http://tomcat.apache.org/dev/dist
>>
>> Maybe it should be, as is in 6.0,
>> base-tomcat.loc=http://archive.apache.org/dist/tomcat
>
> Yep. That is the right location.
>

Ok, fixed.
http://svn.apache.org/viewvc?rev=786792&view=rev

Best regards,
Konstantin Kolinko

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Re: svn commit: r786530 - /tomcat/current/tc5.5.x/STATUS.txt

Posted by Mark Thomas <ma...@apache.org>.
Konstantin Kolinko wrote:
>>  * Fix wrong download location of tcnative 1.1.16
>>   Patch for "build/build.properties.default":
>>   http://people.apache.org/~kkolinko/patches/2009-05-25_fix_tcnative_download.patch
>>   +1: kkolinko, fhanik
>> -  -1:
>> +  -1: markt - the updated location is still wrong
> 
> It points to
> http://tomcat.apache.org/dev/dist/tomcat-connectors/native/source/1.1.16/tomcat-native-1.1.16-src.tar.gz

That is a really bad place for it to point at. That is the location for
the test builds rather than releases.

> because
> base-tomcat.loc=http://tomcat.apache.org/dev/dist
> 
> Maybe it should be, as is in 6.0,
> base-tomcat.loc=http://archive.apache.org/dist/tomcat

Yep. That is the right location.

> The original link to the native sources will work then.
> 
> The current value of base-tomcat.loc in build.properties.default of tc5.5
> originates from this rev.:
> http://svn.apache.org/viewvc?view=rev&revision=613225
> 
> Any concerns?

Only that I have no idea why I thought that was a good idea.

> I agree that it is strange to point to /dev, but maybe we had an unreleased
> version placed there at that time, or maybe that value just slipped in.

Maybe. I can't remember why I did that. It looks plain wrong with hindsight.

Mark



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Re: svn commit: r786530 - /tomcat/current/tc5.5.x/STATUS.txt

Posted by Konstantin Kolinko <kn...@gmail.com>.
>  * Fix wrong download location of tcnative 1.1.16
>   Patch for "build/build.properties.default":
>   http://people.apache.org/~kkolinko/patches/2009-05-25_fix_tcnative_download.patch
>   +1: kkolinko, fhanik
> -  -1:
> +  -1: markt - the updated location is still wrong

It points to
http://tomcat.apache.org/dev/dist/tomcat-connectors/native/source/1.1.16/tomcat-native-1.1.16-src.tar.gz

because
base-tomcat.loc=http://tomcat.apache.org/dev/dist

Maybe it should be, as is in 6.0,
base-tomcat.loc=http://archive.apache.org/dist/tomcat

The original link to the native sources will work then.

The current value of base-tomcat.loc in build.properties.default of tc5.5
originates from this rev.:
http://svn.apache.org/viewvc?view=rev&revision=613225

Any concerns?

I agree that it is strange to point to /dev, but maybe we had an unreleased
version placed there at that time, or maybe that value just slipped in.

Best regards,
Konstantin Kolinko

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org