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

Issue 276250043: TBR: Update README.md on Dr. Memory release version

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 5 months ago by zhaoqin
Modified:
8 years, 5 months ago
Reviewers:
bruening
CC:
drmemory-devs_googlegroups.com
Visibility:
Public.

Description

Commit log for first patchset: --------------- Update README.md on Dr. Memory release version - change 1.8.0 to 1.9.0 ---------------

Patch Set 1 #

Patch Set 2 : Committed #

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

Messages

Total messages: 3
zhaoqin
8 years, 5 months ago (2015-11-25 16:32:16 UTC) #1
zhaoqin
Committed as https://github.com/DynamoRIO/drmemory/commit/5f2c907c70963c0ff609a90f2ac99ab76627af25 Final commit log: --------------- Update README.md on Dr. Memory release version - ...
8 years, 5 months ago (2015-11-25 16:32:42 UTC) #2
bruening
8 years, 5 months ago (2015-11-29 15:10:57 UTC) #3
https://codereview.appspot.com/276250043/diff/20001/README.md
File README.md (right):

https://codereview.appspot.com/276250043/diff/20001/README.md#newcode16
README.md:16: [New 1.9.0
release](https://github.com/DynamoRIO/drmemory/wiki/Downloads)
If we're going to update this file on every release, we should update the wiki
page that lists what to do on each release.  Seems best to just remove the ver #
from this file.  I'll do that.

https://codereview.appspot.com/276250043/diff/20001/README.md#newcode18
README.md:18: Mac](https://bintray.com/bruening/DrMemory/DrMemory-MacOS/view).
This is a now-invalid link that needs to be changed.  I will update it.
Sign in to reply to this message.

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