Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug IO::Path method move #6216

Closed
p6rt opened this issue May 2, 2017 · 5 comments
Closed

Bug IO::Path method move #6216

p6rt opened this issue May 2, 2017 · 5 comments

Comments

@p6rt
Copy link

p6rt commented May 2, 2017

Migrated from rt.perl.org#131242 (status was 'resolved')

Searchable as RT131242$

@p6rt
Copy link
Author

p6rt commented May 2, 2017

From @MARTIMM

With Rakudo version 2017.04.3-66-g7648793 built on MoarVM version
2017.04-44-gf0db882
implementing Perl 6.c. on a system (uname -a) Linux h03-fedora
4.10.11-100.fc24.x86_64 #​1 SMP Tue Apr 18 17​:25​:04 UTC 2017 x86_64
x86_64 x86_64 GNU/Linux

I see the following problem

Moving a file where destination is the same as its source, two things
can happen a) program hangs or b) file is deleted. I would expect an
X​::IO​::Move exception with a proper message.

Example.​: 'abc.txt'.IO.move('abc.txt');

Regards,

Marcel

@p6rt
Copy link
Author

p6rt commented May 2, 2017

From @zoffixznet

On Tue, 02 May 2017 03​:30​:44 -0700, mt1957@​gmail.com wrote​:

With Rakudo version 2017.04.3-66-g7648793 built on MoarVM version
2017.04-44-gf0db882
implementing Perl 6.c. on a system (uname -a) Linux h03-fedora
4.10.11-100.fc24.x86_64 #​1 SMP Tue Apr 18 17​:25​:04 UTC 2017 x86_64
x86_64 x86_64 GNU/Linux

I see the following problem

Moving a file where destination is the same as its source, two things
can happen a) program hangs or b) file is deleted. I would expect an
X​::IO​::Move exception with a proper message.

Example.​: 'abc.txt'.IO.move('abc.txt');

Regards,

Marcel

Thank you for the report. This is now fixed.

Fix​: rakudo/rakudo@08a8075f91
Tests​: Raku/roast@4fdb8504cd
Docs​: Raku/doc@eca21ff851

-- IO grant

@p6rt
Copy link
Author

p6rt commented May 2, 2017

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented May 2, 2017

@zoffixznet - Status changed from 'open' to 'resolved'

@p6rt p6rt closed this as completed May 2, 2017
@p6rt
Copy link
Author

p6rt commented May 2, 2017

From @MARTIMM

Thanks very much
Marcel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant