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

Removed PCRE library support #2525

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Removed PCRE library support #2525

wants to merge 2 commits into from

Conversation

unxed
Copy link
Contributor

@unxed unxed commented Nov 23, 2024

There are several reasons for that.

  1. 7zip 23.01 supports RAW format itself, we do not need fls and icat for it now

  2. Two different regexp engines (PCRE and Far internal one) in one app is abmigious and confuse users.

  3. custom.ini is here since 2016, and the only use for it is actually wrapping around 7zip. More of this, nowadays we can just link to 7z.so in distros and use 7zip as a library.

  4. Any parsing of archiver output can be done directly in the command used to invoke it, using tools like awk or perl. You can then provide multiarc with, say, CSV format, which can be parsed using the standard multiarc method.

  5. Minus one unneeded dependency.

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

Successfully merging this pull request may close these issues.

1 participant