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

Interested ? #1

Open
jief666 opened this issue Jul 17, 2018 · 6 comments
Open

Interested ? #1

jief666 opened this issue Jul 17, 2018 · 6 comments

Comments

@jief666
Copy link
Owner

jief666 commented Jul 17, 2018

@tomkoen : here is my first version that integrate HFS & APFS disk images, encrypted or clear, sparsebundle or plain files.
I didn't test every single possibility (APF or HSF, encrypted or clear, different compression or not, GPT or Apple map or no map make quite a lot of possibilities), but I checked quite a few.
It's a work in progress branch. There is some cleaning, duplicates...

@LubosD @bugaevc : I can do some pull request if you are interested by some features (encryption, sparsebundle, etc.). Just let me know.

@tomkoen
Copy link

tomkoen commented Jul 17, 2018

@jief666 well. I'm interested in new features but better as a part of darling-dmg, it's a mature project which I use for quite a while...

Whould be great to see new pull requests, but one by one in order to be able to test.

@jief666
Copy link
Owner Author

jief666 commented Jul 17, 2018

Sure, I'd like that too. My point is NOT to get away from darling-dmg. I just wanted to commit something so they can see what they want (and how I did it). I made enough pull requests lately, so I thought I can temporarily commit in a new repo and give them a break.
I also want to do some more cleaning, support openssl as well as embedded crypto.

I'll be more than happy if they want all, so I don't have to keep in sync.
Let see what they'd like.

@jief666
Copy link
Owner Author

jief666 commented Jul 17, 2018

I also saw that they never took your pull request about encryption, so they maybe not interested by that.

@LubosD
Copy link

LubosD commented Jul 17, 2018

We are interested in supporting encrypted images, but the PR felt kind of like "stealing" because it copied a lot of code from someone else's project :-D
I'd also prefer using OpenSSL for crypto functions instead of having an implementation inside darling-dmg.

@jief666
Copy link
Owner Author

jief666 commented Jul 17, 2018

I think tomkoen started from https://github.com/sgan81/apfs-fuse/blob/master/ApfsLib/DeviceDMG.cpp.
I've merge something I already had and his version to make an EncryptedFileReader that support openssl. If you take it, I suppose the original author should be mentioned ?
If it's ok, I'll do the pull request (without embedded crypto).

@jief666
Copy link
Owner Author

jief666 commented Sep 3, 2018

@tomkoen ,
Hi,
here is an updated version (in branch wip). It include Sparsebundle and encryption, APFS.
In case you wonder, the goal of this project is not to fork and go his own way. In fact the HFS part is a darling-dmg submodule, and APFS is also a submodule. Compare source tree, it's barely touched.

I will continue to submit pull request to original projects, ideally to not have to fork darling-dmg or apfs-fuse for minor adjustment. But you know very well that pull requests can sometimes hang for a long time with no explanations. To speed up, I commit in this project while waiting for pull requests to be merged.

If you make a Visual C project, I'll be happy to integrate.

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