806538 - Update double-conversion and document the version we are at

Status: RESOLVED FIXED

(Core :: MFBT, defect)

Comment 1

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

Attachment: update it

https://tbpl.mozilla.org/?tree=Try&rev=9a92b43c4afd

Comment 2

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

Attachment: there was an "hg rm" missing in the last patch.

Comment 3

[Jeff Walden [:Waldo]]

Comment on attachment 676311 [details] [diff] [review]
there was an "hg rm" missing in the last patch.

Review of attachment 676311 [details] [diff] [review]:
-----------------------------------------------------------------

::: mfbt/double-conversion/update.sh
@@ +4,5 @@
>  # double-conversion source that we need.
>  
> +# This was last updated with git rev e5b34421b763f7bf7e4f9081403db417d5a55a36.
> +
> +set -e

Since we're already touching this...

We might consider putting all the third-party code in mfbt/double-conversion/src, then having update.sh and all the patches in mfbt/double-conversion.  Then update.sh could rm -rf src, then run the git clone/pull/whatever command that included the gitrev directly, then apply our patches.

I'm not going to let perfect be the enemy of good, but if you're feeling especially motivated, this seems like it'd be a nice thing to do, to avoid mixing our stuff directly amidst theirs.

Comment 4

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/72003517a631

Comment 5

[Ed Morley [:emorley]]

Backed out along with several others in order to get inbound green again, after a busted landing meant we lost test coverage for 7 pushes, and now have multiple failures:
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=a145ded68994

Backout:
https://hg.mozilla.org/integration/mozilla-inbound/rev/553fb59b9ca0

Comment 6

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1c6b13fd3b6a

Comment 7

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/1c6b13fd3b6a