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

Issue 29825642: Noissue - add 7z requirement into README.md (Closed)

Created:
July 9, 2018, 2:02 p.m. by sergei
Modified:
Aug. 20, 2018, 10:16 a.m.
Reviewers:
hub
CC:
anton
Base URL:
https://github.com/adblockplus/libadblockplus@d361cdf75c6c5af2aaa9b484092ded6c0a1dc441
Visibility:
Public.

Description

Since we document how to obtain prebuilt V8 binaries for *nix we should mention also tools required for that. Additionally it can help to find out all missing requirements in the derived projects.

Patch Set 1 #

Total comments: 3

Patch Set 2 : address comments #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -1 line) Patch
M README.md View 1 1 chunk +6 lines, -1 line 2 comments Download

Messages

Total messages: 7
sergei
https://codereview.adblockplus.org/29825642/diff/29825647/README.md File README.md (right): https://codereview.adblockplus.org/29825642/diff/29825647/README.md#newcode32 README.md:32: one can use `p7zip-full` package and `p7zip` on rpm ...
July 9, 2018, 2:11 p.m. (2018-07-09 14:11:17 UTC) #1
hub
https://codereview.adblockplus.org/29825642/diff/29825647/README.md File README.md (right): https://codereview.adblockplus.org/29825642/diff/29825647/README.md#newcode32 README.md:32: one can use `p7zip-full` package and `p7zip` on rpm ...
July 9, 2018, 2:44 p.m. (2018-07-09 14:44:39 UTC) #2
sergei
https://codereview.adblockplus.org/29825642/diff/29825647/README.md File README.md (right): https://codereview.adblockplus.org/29825642/diff/29825647/README.md#newcode32 README.md:32: one can use `p7zip-full` package and `p7zip` on rpm ...
July 9, 2018, 3:20 p.m. (2018-07-09 15:20:32 UTC) #3
hub
https://codereview.adblockplus.org/29825642/diff/29825659/README.md File README.md (right): https://codereview.adblockplus.org/29825642/diff/29825659/README.md#newcode32 README.md:32: one can use `p7zip-full` package and `p7zip-plugins` on rpm-based. ...
July 13, 2018, 2:39 p.m. (2018-07-13 14:39:36 UTC) #4
hub
otherwise LGTM.
July 13, 2018, 4:11 p.m. (2018-07-13 16:11:28 UTC) #5
hub
this one should be checked in
Aug. 16, 2018, 6:18 p.m. (2018-08-16 18:18:15 UTC) #6
sergei
Aug. 20, 2018, 10:07 a.m. (2018-08-20 10:07:34 UTC) #7
Since I made a change, I have moved it to gitlab,
https://gitlab.com/eyeo/adblockplus/libadblockplus/merge_requests/12. Let's
continue there.

https://codereview.adblockplus.org/29825642/diff/29825659/README.md
File README.md (right):

https://codereview.adblockplus.org/29825642/diff/29825659/README.md#newcode32
README.md:32: one can use `p7zip-full` package and `p7zip-plugins` on rpm-based.
On 2018/07/13 14:39:36, hub wrote:
> I'd write 
> 
> "one can use `p7zip-full` package, or `p7zip-plugins` on rpm-based"
> 
> BTW I'm unsure about "RPM based" since I don't know what SuSE does use for
their
> package name. It possibly differ from Fedora based (or RHEL)

Done.

Powered by Google App Engine
This is Rietveld