Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(219)

Issue 2460523002: libsrtp: Update license location in README.chromium (Closed)

Created:
4 years, 1 month ago by Raphael Kubo da Costa (rakuco)
Modified:
4 years, 1 month ago
CC:
chromium-reviews
Target Ref:
refs/heads/master
Visibility:
Public.

Description

libsrtp: Update license location in README.chromium The LICENSE file is now located in the top-level directory following commit 7baef70 ("Update libsrtp to version 2.0"). R=kjellander@chromium.org, mattdr@chromium.org, phoglund@chromium.org Committed: https://chromium.googlesource.com/chromium/deps/libsrtp/+/0e0936f3013fe5884eac82f95e370c8d460a179f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M README.chromium View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7 (1 generated)
Raphael Kubo da Costa (rakuco)
4 years, 1 month ago (2016-10-27 10:21:10 UTC) #1
kjellander_chromium
lgtm confirmed: https://cs.chromium.org/chromium/src/third_party/libsrtp/LICENSE
4 years, 1 month ago (2016-10-27 13:02:24 UTC) #2
mattdr-at-chromium
lgtm, thanks!
4 years, 1 month ago (2016-10-27 17:25:24 UTC) #3
Raphael Kubo da Costa (rakuco)
Hmm, I may need some help landing this, as there's no commit queue for Chromium ...
4 years, 1 month ago (2016-10-28 07:40:19 UTC) #4
kjellander_chromium
Committed patchset #1 (id:1) manually as 0e0936f3013fe5884eac82f95e370c8d460a179f (presubmit successful).
4 years, 1 month ago (2016-10-28 07:52:36 UTC) #6
kjellander_chromium
4 years, 1 month ago (2016-10-28 07:52:58 UTC) #7
Message was sent while issue was closed.
On 2016/10/28 07:40:19, Raphael Kubo da Costa (rakuco) wrote:
> Hmm, I may need some help landing this, as there's no commit queue for
Chromium
> dependencies.

Done :)

Powered by Google App Engine
This is Rietveld 408576698