Mailing-List: contact cygwin-help AT sourceware DOT cygnus DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe AT sources DOT redhat DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin/>
List-Post: <mailto:cygwin AT sources DOT redhat DOT com>
List-Help: <mailto:cygwin-help AT sources DOT redhat DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-owner AT sources DOT redhat DOT com
Delivered-To: mailing list cygwin AT sources DOT redhat DOT com
Message-ID: <3AA91DB0.F1CDE9F@hackdata.com>
Date: Fri, 09 Mar 2001 19:15:12 +0100
From: Hack =?iso-8859-1?Q?Kampbj=F8rn?= <hack AT hackdata DOT com>
X-Mailer: Mozilla 4.76 [en] (Windows NT 5.0; U)
X-Accept-Language: en,da,ca,es
MIME-Version: 1.0
To: cygwin AT sources DOT redhat DOT com
CC: d94-llu AT nada DOT kth DOT se
Subject: Re: wget bug?
References: <984155140 DOT 17264 DOT ezmlm AT sources DOT redhat DOT com>
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by delorie.com id NAA13055

First of all excuse me if this already has been discussed ad libitum.
But I'm only on the digest list 8-(

cygwin-digest-help AT sources DOT redhat DOT com wrote:
> Subject: Re: wget bug?
> Date: Fri, 09 Mar 2001 09:53:35 -0500
> From: "Charles S. Wilson" <cwilson AT ece DOT gatech DOT edu>
> To: d94-llu AT nada DOT kth DOT se
> CC: cygwin AT sources DOT redhat DOT com
> 
> Thanks for the patch.  However, you should probably do something like:
> #if defined(WIN32) || defined(__CYGWIN__)
>   your code
> #endif
> 
> Can you regenerate the patch with those changes and send it both to me,
> and to the official wget maintainer or mailing list (go to www.gnu.org
> to find out how to report patches for wget).
> 
> Thanks,
> --Chuck
> 
> d94-llu AT nada DOT kth DOT se wrote:
> >
> > I found where to change all ? to _.
> >
> > This is an ugly solution not checking that FS is NTFS or anything such. Also
> > other protocols (ftp) should be fixed.

I think url.c is a better place to check for this. Not only that I
actually submited a quick hack for working around this but your is more
general.
http://www.mail-archive.com/wget AT sunsite DOT dk/msg00190.html

But this was turned down. I'm not aware of any archive of the wget-patch
list. So: 
	"Thanks for the patch, but I think I'll refrain from applying
	it for now.  We really should have a more general solution for
	mapping of remote file names to local file names.  Until then,
	the makers of Windows binaries can cons up a similar patch to
	remove disallowed characters from Windows file systems."


If submiting patches to wget please take a look at the Development
section on the wget web-site:
http://sunsite.dk/wget/wgetdev.html

> >
> > Small patch:
> >
> > /usr/local/src/wget-1.6/src$ diff http.c http2.c
> > 749a750,762
> > >       /* Windows doesn't (always) allow /\:*?"<>| in filenames,
> > >          Change to _ and pray for none name-clashes.
> > >          But / and \ are path-indicators. Leave them.
> > >          Leif Lundgren, 2001-03-09
> > >        */
> > >       {
> > >               int i;
> > >               const char bad_chars[] = ":*?\"<>|";
> > >               for ( i=0; u->local[i] != '\0'; i++ )
> > >                       if ( strchr( bad_chars, u->local[i] ) != NULL )
> > >                               u->local[i] = '_';
> > >       }
> > >
> >
> > At  Fri, 09 Mar 2001 12:59:12 +0100 (MET) Leif Lundgren wrote:
> > >
> > > I think I have discoved a bug in the port of wget. (Version 1.6)
> > >
> > > If you download an url containing a '?' you get an error. wget tries to
> > > create a file containing a '?'.
> > >
> > > Example:
> > > wget "http://search.altavista.com/cgi-bin/query?
> > > q=cygwin&kl=XX&pg=q&Translate=on"
> > >
> > > Does the rest of the world have the same problem?
> > >
> > > Regards
> > >   Leif Lundgren
> > >
> > >



Feel free to CC me when answering as I'm only on the digest list 8-)

-- 
Med venlig hilsen / Kind regards

Hack Kampbjørn               hack AT hackdata DOT com
HackLine                     +45 2031 7799

--
Want to unsubscribe from this list?
Check out: http://cygwin.com/ml/#unsubscribe-simple