[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [gftp] Patch that corrects problems with Novell ls parsing
- To: gftp-users@seul.org
- Subject: Re: [gftp] Patch that corrects problems with Novell ls parsing
- From: Brian Masney <masneyb@gftp.org>
- Date: Mon, 1 Nov 2004 06:12:49 -0500
- Delivered-to: archiver@seul.org
- Delivered-to: gftp-users-outgoing@seul.org
- Delivered-to: gftp-users@seul.org
- Delivery-date: Mon, 01 Nov 2004 06:13:09 -0500
- In-reply-to: <4183DADF.70804@klub.com.pl>
- References: <4183DADF.70804@klub.com.pl>
- Reply-to: gftp-users@seul.org
- Sender: owner-gftp-users@seul.org
- User-agent: Mutt/1.5.5.1+cvs20040105i
Hi,
Thanks for this patch, I just merged it into CVS. The tarball
http://www.gftp.org/gftp-test.tar.bz2 has my latest code as well.
Brian
On Sat, Oct 30, 2004 at 08:18:07PM +0200, Thomson wrote:
> Hi there!
>
> My name is Tomek Mrugalski and I'm a long time user of GFTP. I'd like to
> say that it is a great piece of software.
> But it has one bug, which troubled me. In my LAN there's a Novell FTP
> server. It's admin is not so well informed,
> so he is using usernames with spaces. It's a stupid idea, but some
> admins do it anyway. So I had to fix GFTP
> to properly support this.
>
> In function gftp_ls_parse_novell(...) in lib/protocol.c, there are
> problems with parsing when Novell username contains spaces. Oh well.
> GFTP is unable to parse those entries, so there's a patch needed:
>
> fle->group = g_strdup (_("unknown"));
>
> while (*startpos!=0 && !isdigit(*startpos))
> startpos++;
>
> fle->size = gftp_parse_file_size (startpos);
>
>
> And here's output of diff protocol.c and prococol2.c
>
>
> 1582a1583,1585
> > while (*startpos!=0 && !isdigit(*startpos))
> > startpos++;
> >
>
> Well, that's all.
>
> Greetings from Poland
>
> --
> Tomasz Mrugalski, | "I think there is a world market for |
> spam@klub.com.pl | about five computers." |
> | Thomas J. Watson (Chairman, IBM) 1943|
>
>
>
>
>
>