On Fri, Dec 13, 2013 at 2:15 PM, Jan Nijtmans <jan.nijtm...@gmail.com>wrote:

> 2013/12/13 Mark Janssen <mpc.jans...@gmail.com>:
> > With the updated version [85528ef507] I still get the same error.
> > xfer_run_script still fails with error:
> >
> > url must be http:// or https://
> >
> > At xfer.c line 869
> >
> > Maybe there is something else I am doing wrong?
>
> More likely that I did something wrong, like
> an incomplete commit. Please try again ;-)
>
> Regards,
>        Jan Nijtmans
> _______________________________________________
> fossil-users mailing list
> fossil-users@lists.fossil-scm.org
> http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users
>


Hi Jan,

Two remarks:

1) I am fairly sure that:
    memset(&urlData, '0', sizeof(urlData));
  Should be
    memset(&urlData, 0, sizeof(urlData));

2) I think url_parse_local should be fixed to properly fill all the urlData
fields instead of having to memset the structure before.

Mark

BTW with memset 0 it works :)
_______________________________________________
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users

Reply via email to