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.

Sign up for GitHub

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

Jump to bottom

'cp a.txt b.txt' fails on Windows #115

Closed
kal247 opened this issue Jan 4, 2023 · 1 comment
Closed

'cp a.txt b.txt' fails on Windows #115

kal247 opened this issue Jan 4, 2023 · 1 comment
Assignees
@briandfoy
Labels
Platform: windows an existing feature does not work Status: fixed the fix has been merged into master Status: stalled something is blocking progress Type: bug an existing feature does not work

Comments

@kal247
Copy link
Contributor

kal247 commented Jan 4, 2023

On Windows, cp a.txt b.txt fails at line 252 with message "cp: can not access B.TXT ... skipping".

Overall, this script has problems with case sensitivity, for example cp a.txt dir will create dir/A.TXT.
@briandfoy briandfoy added Type: bug an existing feature does not work Status: needs help needs outside expertise or capacity Windows labels Jan 4, 2023
@briandfoy
Copy link
Owner

Okay, noted.

I agree that many of the scripts have problems, but I also remember that they were often written in the previous millennium.

If someone wants to wholly replace it with something that works (using only core modules), I can merge that patch.

@briandfoy briandfoy added Platform: windows an existing feature does not work Status: stalled something is blocking progress and removed Windows labels Mar 7, 2023
briandfoy added a commit that referenced this issue Jun 21, 2023
@briandfoy
Initial cp test for #115
e937650
briandfoy added a commit that referenced this issue Jun 21, 2023
@briandfoy
Turn off case sensitive tracking in bin/cp (#115)
b3a0f1e
@briandfoy briandfoy mentioned this issue Jun 21, 2023
Closed
@briandfoy briandfoy added Status: verified issue has been verified (is reproducible) and removed Status: needs help needs outside expertise or capacity labels Jun 21, 2023
briandfoy added a commit that referenced this issue Jun 21, 2023
@briandfoy
For #115, cp has case problems on Windows.
d62f527 
Squashed commit of the following:

commit b3a0f1e
Author: brian d foy <brian.d.foy@gmail.com>
Date:   Wed Jun 21 09:49:15 2023 -0400

    Turn off case sensitive tracking in bin/cp (#115)

commit e937650
Author: brian d foy <briandfoy@pobox.com>
Date:   Wed Jun 21 09:36:01 2023 -0400

    Initial cp test for #115
@briandfoy briandfoy added Status: fixed the fix has been merged into master and removed Status: verified issue has been verified (is reproducible) labels Jun 21, 2023
@briandfoy briandfoy self-assigned this Jun 21, 2023
briandfoy added a commit that referenced this issue Jun 22, 2023
@briandfoy
Use File::Copy for most uses in bin/cp (#175)
a8cce79 
This gets rid of all of the previous code, which nostly didn't work
on Windows (see #115).

Squashed commit of the following:

commit c21cce7
Author: brian d foy <brian.d.foy@gmail.com>
Date:   Thu Jun 22 08:52:16 2023 -0400

    Update MANIFEST

commit 8c99471
Author: brian d foy <brian.d.foy@gmail.com>
Date:   Thu Jun 22 08:38:08 2023 -0400

    Convert the return value of File::Copy to the right success value

commit d3d81fa
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 06:07:37 2023 -0400

    Set filehandles through variables

commit 3a91cf4
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 06:01:25 2023 -0400

    Document options

commit ffe2674
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 06:00:48 2023 -0400

    Document exit codes

commit 49d6db2
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 05:59:57 2023 -0400

    Update copyright

commit 54af6c2
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 05:59:29 2023 -0400

    Remove modules I'm not using

commit 3c1cdee
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 01:42:38 2023 -0400

    Return the package name when required

commit df1568a
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 01:41:55 2023 -0400

    Add the rest of the modulino stuff

commit cf6eda5
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 00:34:20 2023 -0400

    unlink the right file

commit a28ff5b
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 00:34:08 2023 -0400

    Show files in the subdir too

commit c9f7dc6
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 00:33:52 2023 -0400

    final semicolon

commit 0c9e09a
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 00:27:37 2023 -0400

    Add missing semicolon

commit 105cc7c
Author: brian d foy <briandfoy@pobox.com>
Date:   Thu Jun 22 00:00:07 2023 -0400

    Don't try to treat Windows specially. It's a bit hopeless.

commit 6bacaf6
Author: brian d foy <briandfoy@pobox.com>
Date:   Wed Jun 21 23:59:39 2023 -0400

    Remove old program stuff

commit 0bbdac5
Author: brian d foy <briandfoy@pobox.com>
Date:   Wed Jun 21 23:57:31 2023 -0400

    Check that target file is not there at the start and gone at the end

commit 8397a40
Author: brian d foy <briandfoy@pobox.com>
Date:   Wed Jun 21 23:09:13 2023 -0400

    cmd /c didn't work

commit 96defe8
Author: brian d foy <briandfoy@pobox.com>
Date:   Wed Jun 21 22:46:47 2023 -0400

    Use cmd /c

commit 71e105e
Author: brian d foy <briandfoy@pobox.com>
Date:   Wed Jun 21 22:28:32 2023 -0400

    Try harder with diag

commit dabdd9a
Author: brian d foy <briandfoy@pobox.com>
Date:   Wed Jun 21 22:20:44 2023 -0400

    show the files in the dir after the operation

commit 746d690
Author: brian d foy <briandfoy@pobox.com>
Date:   Wed Jun 21 21:51:23 2023 -0400

    output some error info if system fails

commit 7229037
Author: brian d foy <briandfoy@pobox.com>
Date:   Wed Jun 21 19:56:12 2023 -0400

    shell out to copy/cp if it's around, otherwise File::Copy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@briandfoy briandfoy

Labels
Platform: windows an existing feature does not work Status: fixed the fix has been merged into master Status: stalled something is blocking progress Type: bug an existing feature does not work
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@briandfoy @kal247