Accept .tar.xz in the GUI and check BeautifulSoup parser dependencies - #2242
Merged
Merged
Conversation
BeautifulSoup takes its parser as a string, so a module asking for one that requirements.txt does not install imports cleanly and fails only when it parses. RLEAPP 2026.4.1 shipped that. The new test fails when a call names no parser, names a feature several builders share, or names a parser whose library requirements.txt does not install, and it follows calls made through an alias. It passes on the current tree and is the same file in all five cores. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The GUI takes the input type from the last suffix, so a .tar.xz came through as 'xz', which matches no branch, and the run stopped with "Error on argument -o (input type)". It now maps to 'tar', which already reads an xz-compressed tar through tarfile's compression detection, and the file picker lists *.xz the same way it lists *.gz for a .tar.gz. A bare .xz, a .tar.gz and a .tar keep the types they had. The README notes that a compressed tar reads much more slowly than a plain one, since the reader decompresses from the start whenever it needs an earlier file, and says to decompress a large extraction first. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two changes, one commit each.
.tar.xz. Its type came from the last suffix,xz, which no branch handles, so the run stopped with "Error on argument -o (input type)". It now maps totar, which already reads xz, and the file picker lists*.xz. The README adds that a compressed tar reads much more slowly than a plain one: a 5.4 GB Android extraction took 33 s as.tar, 347 s as.tar.gz, and projected at about two hours as.tar.xz.test_bs4_parser_dependencies.pyfails when a BeautifulSoup call names no parser, or one whose libraryrequirements.txtdoes not install. It is the same file in all five cores.🤖 Generated with Claude Code