Comment on attachment 600506
Use YARR interpreter instead of PCRE on platforms where YARR JIT is not 
supported (m-c patch)

Review of attachment 600506:
-----------------------------------------------------------------

Stealing the review because Chris doesn't have time. Looks fine--just
one little note about this and bug 731110 but you probably already know
anyway.

::: js/src/Makefile.in
@@ +351,5 @@
> +                OSAllocatorWin.cpp \
> +                PageBlock.cpp \
> +                YarrInterpreter.cpp \
> +                YarrPattern.cpp \
> +                YarrSyntaxChecker.cpp \

This probably conflicts with patch 3 in bug 731110. Not sure exactly how
you guys want to deal with that, just pointing it out.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to thunderbird in Ubuntu.
https://bugs.launchpad.net/bugs/908508

Title:
  Firefox/Thunderbird 10 FTBFS on powerpc in js/src/yarr/pcre

Status in The Mozilla Firefox Browser:
  In Progress
Status in “firefox” package in Ubuntu:
  Fix Released
Status in “thunderbird” package in Ubuntu:
  Fix Released

Bug description:
  Debian has the fix, so I'll add it to the branch and it should work
  with the next upload.

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/908508/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to