Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29807589: Issue 6695 - Allow automatically downloading V8 (Closed)

Created:
June 15, 2018, 1:52 a.m. by hub
Modified:
June 18, 2018, 4:25 p.m.
Reviewers:
sergei
Base URL:
https://hg.adblockplus.org/libadblockplus/
Visibility:
Public.

Description

Issue 6695 - Allow automatically downloading V8

Patch Set 1 #

Total comments: 8

Patch Set 2 : Changes in README. Renamed Makefile target. #

Patch Set 3 : prebuilt, not prebuild #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -1 line) Patch
M .gitignore View 1 chunk +1 line, -0 lines 0 comments Download
M .hgignore View 1 chunk +1 line, -0 lines 0 comments Download
M Makefile View 1 2 2 chunks +12 lines, -0 lines 0 comments Download
M README.md View 1 2 1 chunk +16 lines, -1 line 0 comments Download

Messages

Total messages: 6
hub
June 15, 2018, 1:52 a.m. (2018-06-15 01:52:48 UTC) #1
hub
it needs to be manually invoked. Probably needs refinements to support multiple targets.
June 15, 2018, 1:53 a.m. (2018-06-15 01:53:57 UTC) #2
sergei
Although I would like to rather have scripts which don't know about CI and CI ...
June 18, 2018, 10:16 a.m. (2018-06-18 10:16:25 UTC) #3
hub
I can change this to not use the CI scripts, but rather to have the ...
June 18, 2018, 12:51 p.m. (2018-06-18 12:51:10 UTC) #4
hub
update patch
June 18, 2018, 1:40 p.m. (2018-06-18 13:40:11 UTC) #5
sergei
June 18, 2018, 1:54 p.m. (2018-06-18 13:54:42 UTC) #6
LGTM.

Powered by Google App Engine
This is Rietveld