Opened 2 years ago

Last modified 2 years ago

#6695 reopened change

Download recent supported version of V8 and use it for local builds by default.

Reported by: sergz Assignee:
Priority: P2 Milestone:
Module: Libadblockplus Keywords:
Cc: hfiguiere Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):



In the codereview there was a request to make the building and using of the library more user friendly, regarding binaries. Currently one has to dig through the CI scripts in order to download prebuilt V8 or build it manually (what takes a lot of time and disk space).

JIC, in order to download the versions for today one can use the following commands on linux URL_PREFIX= TARGET_OS=android Configuration=release TARGET_ARCH=arm .travis/ or URL_PREFIX= TRAVIS_OS_NAME=linux .travis/, etc.

and get the downloaded prebuilt V8 in third_party/prebuilt-v8/.

What to change

  • Taking into account #6694, create a file with the list of supported/tested versions of V8 and use that list for CI.
  • provide a script downloading the recent version by default or a version specified in the command line arguments
  • integrate that script either into the building system or dependencies ( or even switch to depot_tools and DEPS or check what meson can provide with).

Change History (9)

comment:1 Changed 2 years ago by hfiguiere

  • Cc hfiguiere added

comment:2 Changed 2 years ago by hfiguiere

  • Owner set to hfiguiere

comment:3 Changed 2 years ago by hfiguiere

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

comment:4 Changed 2 years ago by abpbot

A commit referencing this issue has landed:
Issue 6695 - Allow automatically downloading V8

comment:5 Changed 2 years ago by hfiguiere

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

comment:6 Changed 2 years ago by sergz

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:7 Changed 2 years ago by sergz

  • Review URL(s) modified (diff)

comment:8 Changed 2 years ago by abpbot

comment:9 Changed 2 years ago by hfiguiere

  • Owner hfiguiere deleted
Note: See TracTickets for help on using tickets.