[darcs-devel] [patch1246] issue822: Improve error message with generalized IO type

Ernesto Rodriguez neto at netowork.me
Thu Feb 5 16:42:57 UTC 2015


Sure, I will write one special test for this issue. It is worth mentioning
that there exists a test that checks invalid urls when using scp which
after my changes failed because it was looking or the old error message. I
updated that test to match the new error message. Nevertheless, I will
write a test specific for this case. I will do it throughout the weekend
because today and tomorrow is really busy for me :(.

Regarding wether it solves issue822. My changes do solve the specific
problem mentioned on issue822. It only covers SCP related error messages
but issue822 only talks about scp. So under my criteria it does solve it.


Cheers,

Ernesto

On Thu, Feb 5, 2015 at 5:37 PM, Ben Franksen <bugs at darcs.net> wrote:

>
> Ben Franksen <benjamin.franksen at helmholtz-berlin.de> added the comment:
>
> So, does this resolve issue822? I think so but to make it perfect I
> would like to have a test.
>
> netogallo: Could you be bothered to write one? If not, I may do it instead.
>
> ----------
> assignedto:  -> bf
> milestone:  -> 2.10.0 HEAD
> nosy: +bf
> status: needs-review -> review-in-progress
>
> __________________________________
> Darcs bug tracker <bugs at darcs.net>
> <http://bugs.darcs.net/patch1246>
> __________________________________
>



-- 
Ernesto Rodriguez

Masters Student
Computer Science
Utrecht University

www.netowork.me
github.com/netogallo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osuosl.org/pipermail/darcs-devel/attachments/20150205/0dc9fa76/attachment.html>


More information about the darcs-devel mailing list