<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
<title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
The code looks great. The only thing I noticed was that there were no
Thread.yield(), which Joe did to fix the selection redraw problem. Is
it needed here?<br>
<br>
I also noticed that it creates a second tab (which I think is a really
cool behavior of using properties to allow different installation
sites). Do we want both an FTP tab and an HTTP tab? I tested it on
WinXP SP2 and it works great. And it seems much faster than FTP. My
guess is that it is because it is a single get operation of a
compressed file.<br>
<br>
Mark Goodwin wrote:
<blockquote cite="midb0ca56fe040820124671bc6e09@mail.gmail.com"
type="cite">
<pre wrap="">BTW; there is one issue I am aware of; because the zip contains the
.conf file, it dumps it in the module data directory. This file is
redundant anyway because a copy is created from mods.d.gz
</pre>
</blockquote>
<br>
I think that this is OK. At some point we may want to remove the
copying of it out of mods.d.gz.<br>
<br>
<blockquote cite="midb0ca56fe040820124671bc6e09@mail.gmail.com"
type="cite">
<pre wrap="">
On Fri, 20 Aug 2004 20:43:00 +0100, Mark Goodwin <a class="moz-txt-link-rfc2396E" href="mailto:goodwinster@gmail.com"><goodwinster@gmail.com></a> wrote:
</pre>
<blockquote type="cite">
<pre wrap="">This patch is an HTTP sword installer.
It's closely based on the existing FTP installer; we might want to
create an implementation superclass for some of the bits; I didn't.
This installer works on all of the test machines that the FTP
installer failed on.
HTH.
MarkG
</pre>
</blockquote>
<pre wrap=""><!---->_______________________________________________
jsword-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:jsword-devel@crosswire.org">jsword-devel@crosswire.org</a>
<a class="moz-txt-link-freetext" href="http://www.crosswire.org/mailman/listinfo/jsword-devel">http://www.crosswire.org/mailman/listinfo/jsword-devel</a>
</pre>
</blockquote>
</body>
</html>