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

efiver.py Doesn't highlight my machine #7

Closed
greyhound3 opened this issue Oct 7, 2017 · 3 comments
Closed

efiver.py Doesn't highlight my machine #7

greyhound3 opened this issue Oct 7, 2017 · 3 comments
Labels

Comments

@greyhound3
Copy link

greyhound3 commented Oct 7, 2017

Thanks for this gem, but I don't know if the error is on my side.
I think this is my problem, the showSystemData never gets executed (Version is 2.0 of efiver.py).
grafik

One question about the line 258, is the ";" after break necessary?

@Piker-Alpha
Copy link
Owner

Piker-Alpha commented Oct 7, 2017

Hi,

Ah yes. That's a stupid error. Need to convert the data object to a string and then strip the trailing NULL byte. Should be fine in v2.1 Please give it a try.

The semicolon is a bad habit. A C programmers thing. Removed. Thanks!

Thank you for reporting this issue here!

@greyhound3
Copy link
Author

greyhound3 commented Oct 8, 2017

Hi, thanks a lot, highlighting works now!!
But for information: to run the v2.2 script now it is necessary to run it with sudo, just entering the password like before doesn't work anymore. I don't know if this is intended.
But now it works!
Thanks for your work!!

without sudo it gives just this (even after deleting the /tmp/InstallAss.... dir) :
Traceback (most recent call last):
File "./efiver.py", line 562, in
main()
File "./efiver.py", line 503, in main
if extractPayloadToDirectory() == True:
File "./efiver.py", line 458, in extractPayloadToDirectory
if not convertPayloadToZX(payloadPath):
File "./efiver.py", line 433, in convertPayloadToZX
with open(payloadPath, 'rb') as sourceFile:
IOError: [Errno 13] Permission denied: '/tmp/InstallAssistantAuto/Payload'

But for me it's not a problem.

@Piker-Alpha
Copy link
Owner

Hi,

The next update of installSeed.py runs pkgutil without sudo. For now. Remove it from this line:

subprocess.call(["sudo", "pkgutil", "--expand", targetFilename, unpackPackage])

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants