vmware / photon-os-installer

Photon OS Installer
Other
12 stars 18 forks source link

Update installer.py #19

Open spyroot opened 1 year ago

spyroot commented 1 year ago

additional_rpms is json list, not just a single entity. Hence it makes sense to iterate for each dir and invoke the rpm. It also fixes the issue if the rpm_paths is a list; exists will raise an exception and crash since it can't process the list.

vmwclabot commented 1 year ago

@spyroot, VMware has rejected your signed contributor license agreement. The merge can not proceed until the agreement has been resigned. Click here to resign the agreement. Reject reason:

VMware employees are not required to execute VMware CLAs.”

vmwclabot commented 1 year ago

@spyroot, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed.

vmwclabot commented 1 year ago

@spyroot, VMware has rejected your signed contributor license agreement. The merge can not proceed until the agreement has been resigned. Click here to resign the agreement. Reject reason:

Please provide full street address

vmwclabot commented 1 year ago

@spyroot, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed.

vmwclabot commented 1 year ago

@spyroot, VMware has approved your signed contributor license agreement.