macadmins / installapplications

A tool for dynamically using installapplication
Apache License 2.0
285 stars 62 forks source link

userland rootscript does not wait for userland #48

Closed MichalMMac closed 4 years ago

MichalMMac commented 5 years ago

Currently code for checking if we are not in setupassistant is only in package installation if/else branch.

When there is no package in userland section of installapplications.json (only rootscripts), installapplications.py does not wait for user to log in. Rootscripts in userland section are executed during setup assistant as if they were in setupassistant section.

Is there a reason for this or should we improve the code?

I created PR #49 with simple change to deal with this. (I did not study entire installapplications.py source code).

erikng commented 5 years ago

Definitely a bug.

Thanks, Erik Gomez


From: Michal Moravec notifications@github.com Sent: Tuesday, April 23, 2019 5:44:23 AM To: erikng/installapplications Cc: Subscribed Subject: [erikng/installapplications] userland rootscript does not wait for userland (#48)

Currently codehttps://github.com/erikng/installapplications/blob/699e3ab550193892216bb0bc247a6211515f1d31/payload/Library/Application%20Support/installapplications/installapplications.py#L641 for checking if we are not in setupassistant is only in package installation if/else branch.

When there is no package in userland section of installapplications.json (only rootscripts), installapplications.py does not wait for user to log in. Rootscripts in userland section are executed during setup assistant as if they were in setupassistant section.

Is there a reason for this or should we improve the code?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/erikng/installapplications/issues/48, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABLL6GGIA2QZ2CB5ANSYK3LPR3SAPANCNFSM4HHWXUEA.

erikng commented 4 years ago

Now addressed in new version