search

juanjonol / Unlock-Redux

Unlock multiple Core Storage encrypted volumes at boot.
Other
8 stars 2 forks source link

Args and return type of subprocess.run() #9

Closed renauddetry closed 6 years ago

renauddetry commented 6 years ago

I'm working with Python 3.5 on High Sierra. I had to modify the subprocess line of get_uuid() as follows:

result = str(subprocess.run(command, stdout=subprocess.PIPE, check=True).stdout)

I removed the encoding argument which doesn't seem to exist in 3.5, and converted the returned value to a string. I don't know enough about Python 3 to figure out whether this will hold in pre or post 3.5.

Thanks a lot for providing an APFS-compatible Unlock, this repo spared me a lot of time!

juanjonol commented 6 years ago

Hi! I'm glad you find this program useful.

I'm going to test your modifications a little and then I'll push it to the repo. Thank you for your comment!