No, I tweaked the text message that mentioned the download size from 300 MB
to 500 MB.
I am going to use your branch to build.
I'll respond shortly.
Cheers,
David
On Mon, Dec 23, 2013 at 3:45 PM, intrigeri <intrigeri@???> wrote:
> Hi,
>
> David Wolinsky wrote (23 Dec 2013 19:29:48 GMT) :
> > I've pushed a working patch into tails/davidiw/devel that fixes 6221 and
> > 6514.
>
> Thanks!
>
> However.. you might have missed my <85y53emafi.fsf@???> email,
> where I asked Julien and you to test and review
> bugfix/6221-support-newer-vagrant, that does just the same + has some
> improvements on top.
>
> Note that bugfix/6221-support-newer-vagrant is based on our stable
> branch, which is preferred for bugfixes that we may want to merge into
> stable. It seems that your branch is forked off the devel branch.
> If it's better than the one I've proposed, please rebase your branch
> on stable first.
>
> Also, having a bit finer grained commits (e.g. at least one for each
> independent bugfix) is preferred to squashing all into a single commit
> as you did.
>
> So, what should we do? As far as the Git history and ancestors is
> concerned, my branch is quite better. But perhaps yours has some fixes
> that I haven't?
>
> Cheers,
> --
> intrigeri
> | GnuPG key @ https://gaffer.ptitcanardnoir.org/intrigeri/intrigeri.asc
> | OTR fingerprint @ https://gaffer.ptitcanardnoir.org/intrigeri/otr.asc
> _______________________________________________
> tails-dev mailing list
> tails-dev@???
> https://mailman.boum.org/listinfo/tails-dev
>
