Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(5478)

Issue 326040043: [l2tpreg] Updated code to work with plaso 20171030 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 9 months ago by Joachim Metz
Modified:
6 years, 6 months ago
Reviewers:
onager
CC:
kiddi, log2timeline-dev_googlegroups.com
Visibility:
Public.

Description

[l2tpreg] Updated code to work with plaso 20171030

Patch Set 1 : Updated code to work with plaso 20171030 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+925 lines, -327 lines) Patch
A .pylintrc View 1 chunk +459 lines, -0 lines 0 comments Download
M .travis.yml View 2 chunks +2 lines, -0 lines 0 comments Download
M appveyor.yml View 2 chunks +2 lines, -1 line 0 comments Download
M config/travis/install.sh View 1 chunk +1 line, -1 line 0 comments Download
M dependencies.ini View 2 chunks +2 lines, -1 line 0 comments Download
M l2tpreg/front_end.py View 5 chunks +18 lines, -10 lines 0 comments Download
A l2tpreg/hexdump.py View 1 chunk +76 lines, -0 lines 0 comments Download
M requirements.txt View 1 chunk +2 lines, -2 lines 0 comments Download
M scripts/preg.py View 16 chunks +42 lines, -22 lines 0 comments Download
M setup.cfg View 1 chunk +1 line, -1 line 0 comments Download
A test_data/artifacts/artifacts.yaml View 1 chunk +252 lines, -0 lines 0 comments Download
M tests/front_end.py View 5 chunks +21 lines, -6 lines 0 comments Download
M utils/dependencies.py View 4 chunks +41 lines, -9 lines 0 comments Download
D utils/pylintrc View 1 chunk +0 lines, -273 lines 0 comments Download
M utils/review.py View 4 chunks +6 lines, -1 line 0 comments Download

Messages

Total messages: 6
Joachim Metz
6 years, 9 months ago (2017-07-13 05:31:31 UTC) #1
Joachim Metz
For reviewer, no need to review this CL yet
6 years, 9 months ago (2017-07-13 06:04:51 UTC) #2
Joachim Metz
Code updated.
6 years, 6 months ago (2017-10-13 17:51:51 UTC) #3
Joachim Metz
PTAL
6 years, 6 months ago (2017-10-13 17:52:24 UTC) #4
onager
LGTM
6 years, 6 months ago (2017-10-13 20:51:15 UTC) #5
Joachim Metz
6 years, 6 months ago (2017-10-14 04:26:03 UTC) #6
merged via github
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b