Re: [Tails-dev] Help needed on #5972: Fix virtualbox guest m…

Delete this message

Reply to this message
Autore: anonym
Data:  
To: The Tails public development discussion list
Oggetto: Re: [Tails-dev] Help needed on #5972: Fix virtualbox guest modules build
19/05/14 00:57, sajolida@??? wrote:
> anonym:
>> 16/05/14 09:52, sajolida@??? wrote:
>>>>> Anyway, I'm absolutely not sure I'll be able to fix that before the
>>>>> freeze. I'd rather concentrate on the documentation upgrades. So if any
>>>>> VirtualBox user want to have a look...
>>>>
>>>> I haven't done a code review yet, but it looks very promising to me so
>>>> expect this to be merged in time for 1.1.
>>
>> Done now, all looks good. My only non-blocker issues are the following:
>>
>> 1. To avoid binary bloat in Git, why not generate the fake gcc-4.8
>>    package (with `equivs-build`) in the hook? I know we cannot access
>>    `deb-src/gcc-4.8/control` inside live-build's chroot (we will when/if
>>    we move to 3.x) but we could just put it somewhere in
>>    `config/chroot_local-includes/` and remove it in the
>>    `98-remove_unwanted_files` hook.

>
> Ok, that's commit 0a11342 now.
>
>> 2. Why not `apt-get remove` the fake gcc-4.8 after we're done with the
>>    DKMS stuff? It's not needed after that, and won't break anything,
>>    unless I've misunderstood something. The presence of that fake
>>    package could actually confuse users that want to install gcc in
>>    Tails.

>
> That's commit 9182bbc now.


Both commits looks good. Thank you!

>>>> A potential follow-up is how to document the shortcomings. Known issue,
>>>> perhaps?
>
> That's commit 348945e.


Reading this, I get the impression that *all* guest features are broken,
but at least Shared folders work (but not auto-mounting since
`VBoxService` is broken). This exception may be worth mentioning, but I
won't block on it...

> I want to rewrite the whole virtualization documentation as part of
> #7143, so I just updated the known issues on that one.


... and neither on this. So merged! Congrats!

Cheers!