-
Notifications
You must be signed in to change notification settings - Fork 129
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
cutadapt cannot read from stdin with xopen 2.0.0 #774
Comments
Something is broken indeed:
Xopen 1.9.0 performs as it should (the extra newline is added by print) xopen2.0.0 misses some data? Which is really weird as all the xopen tests pass. I will see if I can fix this issue. |
I will yank the 2.0.0 release. This is quite serious. Ping @marcelm |
Ah - I hadn't taken the next step of seeing if this was an xopen bug vs cutadapt needing a tweak for an xopen change. Let's close this and focus on pycompression/xopen#157 I've excluded v2.0.0 on my development branch so my CI works, but that is only a stopgap: Yanking the xopen 2.0.0 release seems prudent, thanks - this is more serious that just a cutadapt issue as I first assumed 👍 |
Working with xopen 1.9.0 (and older), running here on macOS:
Using sample file tests/ncbi-import/multiple_hmm.fasta with this command which outputs 5 of FASTA entries via stdin:
Broken when update to xopen 2.0.0 (released 2024-03-26 https://pypi.org/project/xopen/#history - yesterday):
Also using
/dev/stdin
is broken:This might be related to #772, but the timing doesn't fit with xopen 2.0.0 being released yesterday.
The text was updated successfully, but these errors were encountered: