[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