Opened on 02/17/2017 at 11:32:56 AM

Closed on 07/04/2017 at 02:35:50 PM

Last modified on 10/08/2019 at 05:50:55 PM

#4907 closed change (fixed)

Update v8 to 5.7.278 in libadblockplus

Reported by: sergz Assignee: sergz
Priority: P2 Milestone:
Module: Libadblockplus Keywords:
Cc: fhd, eric Blocked By: #4906, #4932, #4933, #4999, #5119, #5125
Blocking: #4194, #5267 Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):


We need a recent version of v8 but we cannot update to the version in #1197 because of linking issues, #4062. However, recently v8 got rid of requirement for stlport and now we can link it statically, the next chosen version is 5.7.278 because it's possible to compile it for windows and android.

What to change

  • code and building infrastructure in libadblockplus
  • satisfy all dependencies, e.g. create local mirrors of required repos

Additional info

  • it requires to switch to MS Visual Studio 2015 (community edition is enough)
  • android ndk 12b

Attachments (0)

Change History (15)

comment:1 Changed on 02/17/2017 at 11:33:12 AM by sergz

  • Component changed from Libadblockplus-Android to Libadblockplus

comment:2 Changed on 02/20/2017 at 03:43:47 PM by

Sergei, about a year ago you were the one that changed #1197 to have a version number of v8 in the title:

It's something of a misrepresentation to say we can't use the version in that ticket, since you were the one that changed it to have a version number in the first place. Please go change #1197 again to use the version number you want to use and close this ticket. It's clouding the actual history here to have another ticket for the same issue.

#1197 languished long enough that Visual Studio 2013 was superseded by Visual Studio 2015 and is about to be superseded by Visual Studio 2017. As long as you're changing #1197, we should update the stated VS requirement to 2017. The compiler continues to improve, with better language support and more correct and efficient STL implementation. VS 2017 will be released March 7, sooner than we'll be ready to update v8.

comment:3 Changed on 02/20/2017 at 03:50:27 PM by

  • Cc eric added

comment:4 Changed on 02/21/2017 at 08:52:41 AM by sergz

I think these tickets are different.
The main aim of #1197 is to change the version of visual studio, and I have added the version of v8 in the title only for reference, however the issue #4907 is to update v8, and updating of visual studio is merely consequences which may not happen.

If I change the title of #1197, I will also have to change its description and the comments may sound a bit inconsistent. In this case, I think, the history will be indeed very cloudy. So, for the sake of simple history I would like to to have them both, and rather add the comment that it has been superseded by #4907 because the latter implies a higher version of Visual studio and either close #1197 or change #1197 to update to visual studio 2017 with removing of version of v8.

comment:5 Changed on 02/24/2017 at 02:10:16 PM by sergz

  • Blocked By 4932 added

comment:6 Changed on 02/24/2017 at 02:15:01 PM by sergz

  • Blocked By 4933 added

comment:7 Changed on 03/16/2017 at 10:00:16 AM by sergz

  • Blocked By 4999 added

comment:8 Changed on 04/11/2017 at 07:59:21 AM by sergz

  • Blocked By 5125 added

comment:9 Changed on 05/23/2017 at 08:26:47 AM by sergz

  • Blocking 5267 added

comment:10 Changed on 05/30/2017 at 08:49:12 AM by sergz

  • Blocking 4194 added

comment:11 Changed on 06/01/2017 at 04:13:25 PM by sergz

  • Blocked By 5119 added

comment:12 Changed on 06/02/2017 at 07:33:46 AM by sergz

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:13 Changed on 07/04/2017 at 02:34:57 PM by abpbot

A commit referencing this issue has landed:
Issue 4907 - Update v8 to 5.7.278 in libadblockplus

comment:14 Changed on 07/04/2017 at 02:35:50 PM by sergz

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:15 Changed on 05/20/2019 at 09:42:34 AM by Arvind098


Last edited on 10/08/2019 at 05:50:55 PM by kzar

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from sergz.
Note: See TracTickets for help on using tickets.