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

zos-files download dataset: no CRLF for every line, only LF (0A) for some lines #1458

Closed
markklerks opened this issue Jun 24, 2022 · 5 comments
Assignees

Comments

@markklerks
Copy link

markklerks commented Jun 24, 2022

Using Zowe command: zowe zos-files download data-set "NL09412C.BTC00.E4012.SCHEMA" -f testimport.txt
leads to a file ("testimport.txt") where not all lines contain a CRLF (in hex: 0D0A), at least 1 line just has a LF.

Downloading it via the APIML or Zowe Explorer in VS Code, does not show this behavior.

dataset attributes:
dsname: NL09412C.BTC00.E4012.SCHEMA
blksz: 27920
catnm: SYSZ.NLBTCT00
cdate: 2021/03/31
dev: 3390
dsorg: PS
edate: None
extx: 4
lrecl: 80
migr: NO
mvol: N
ovf: NO
rdate: 2022/06/24
recfm: FB
sizex: 393
spacu: TRACKS
used: 100
vol: PJC001
vols: PJC001

@t1m0thyj
Copy link
Member

@markklerks Thank you for reporting this, I've been able to reproduce the issue on Windows where occasionally LF does not get replaced with CRLF. We are continuing to investigate this and will let you know when there is a fix available.

@markklerks
Copy link
Author

great news, the issue can be reproduced!
In case you didn't know this already: to find the 'lonely' LF's in a file you can use a regular expression in Notepad++: "^(.*?|^\r)\n"

@VitGottwald
Copy link

Thank you @t1m0thyj! This was released to npm as part of 7.3.1. Running npm -g upgrade @zowe/cli should upgrade to the latest version 7.4.0 as of this morning.

I tested the fix and I no longer see broken CRLF.

@markklerks
Copy link
Author

thanks @t1m0thyj , looks perfect 👍
let's close this one

@markklerks
Copy link
Author

👍

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

3 participants