Ticket Hash: | 2c565a39d9faf823c50dd137ea49c2906ed16268 | |||
Title: | os8-run cpto /A option breaks script | |||
Status: | Closed | Type: | Code Defect | |
Severity: | Minor | Priority: | High | |
Subsystem: | os8-run | Resolution: | Fixed | |
Last Modified: | 2020-07-10 17:51:54 | |||
Version Found In: | ||||
User Comments: | ||||
tangent added on 2020-06-14 23:14:11:
(text/x-markdown)
The os8-run manual claims you can add /A to the end of a `cpto` command, even though this is the default copying mode, but if you do, you get this bogus error: ``` sim> attach -r ptr /usr/local/src/pidp8i/e8-integration/src/e8/e8defs.pa-8027.pt_temp attach -r ptr /usr/local/src/pidp8i/e8-integration/src/e8/e8defs.pa-8027.pt ^H_temp^M PTR: unit is read only^M sim> go 7600 go 7600^M ^M .R PIP^MR PIP^M *DTA0:/A<PTR:^MDTA0:/A<PTR:^M CAN'T OPEN OUTPUT FILE^M ``` Note the missing file in the command to PIP. For this case, it should be like: ``` *DTA0:E8DEFS.PA/A<PTR:^MDTA0:/A<PTR:^M ``` The simple workaround is to remove the `/A`, but that just means that either the manual is wrong or `os8-run` should be changed to obey the manual. tangent added on 2020-06-15 00:14:21: (text/x-markdown) Does [this commit](02b6db5df2) address this bug? poetnerd added on 2020-06-15 01:47:35: (text/x-markdown) Yes. Not a bug per-se, but a failure to adequately document the requirement that the option be preceded with whitespace. tangent added on 2020-07-10 17:51:54: (text/x-markdown) Fixed by documentation in [](02b6db5df2). |