Add two comments to the InstallGuestAdditions, explaining why its return
code will not be checked with `log_check_call()` or even manually
grabbing the status informed by `log_call()` itself.
Previously discussed on the #55 issue and 43e3d96 commit.
I've been discussing that with Anders for over a week and we agreed that
checking the return code of the Guest Additions installation is just a
waste of time. It seems to return 1 no matter what happen, like:
- It has been launched without administrator privileges: OK
- Requirements like bzip2 and gcc are missing: OK
- Kernel headers or dkms are missing: OK
- It was installed successfully, but without X11 drivers: ???
There isn't even a way (at least until the current 4.3.x version) to
disable the X11 drivers installation. The `--nox11` option, just "Do not
spawn an xterm".
And the worst part: until the version 4.2.x, it returned 0 if it could
be installed with no errors beside this one about the missing X11
drivers. This means that this verification will most likely fail on any
other version older than 4.3.x, preventing the build from finishing in a
successful way. So, we can't really rely on its return code whatsoever.
That said, neither him or me likes the idea of ignoring the return code
of a command. It just happens that we can't do anything about that.
New phase introduced "package installation" (fixes#114)
Apt source lines are now parsed, this allows to verify the target release of added packages.
All packages (except locales) are now installed *after* bootstrapping (fixes#123)
Added env argument to log_(check_)call
HostDependencies have been refactored a little
