Skip to content

was able to test on Linux successfully #5

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

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

Conversation

cielavenir
Copy link

Debian GNU/Linux 9
p7zip Version 16.02

@pombredanne
Copy link

@harvimt ping?

@cielavenir
Copy link
Author

just for my info: 7zip 23+ is incompatible with CDEF_IUnknown patch.

@harvimt
Copy link
Owner

harvimt commented Apr 15, 2025

I thought I had already commented this, but to other's who are seeing this in searches or whatever, the reason this isn't merged is the way it's written it drops support for Windows... this was originally intended as a Windows-only project. I'm not opposed to adding Linux support, but the same code needs to work on Linux and Windows.

Also I'm not really working on this project anymore, but people do seem to find it useful as an example if nothing else.

@cielavenir
Copy link
Author

Thank you for the comment~

After my company moves to Debian 13, 7z.so will become v24 and I will be able to remove CDEF_IUnknown patch and Windows can work with the same code

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.

3 participants