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

Issue 20573002: Remove pnacl-nmf and GENNMF now that it is removed from chrome scons tests. (Closed)

Created:
7 years, 4 months ago by jvoung (off chromium)
Modified:
7 years, 4 months ago
Reviewers:
Mark Seaborn
CC:
native-client-reviews_googlegroups.com
Visibility:
Public.

Description

Remove pnacl-nmf and GENNMF now that it is removed from chrome scons tests. Removal from chrome: https://codereview.chromium.org/20370002/ Reduce confusion about which driver tools are supported. This is no longer supported, and the SDK has create_nmf.py, which works for PNaCl. BUG=none, cleanup R=mseaborn@chromium.org Committed: https://src.chromium.org/viewvc/native_client?view=rev&revision=11864

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -222 lines) Patch
D pnacl/driver/pnacl-nmf.py View 1 chunk +0 lines, -220 lines 0 comments Download
M site_scons/site_tools/naclsdk.py View 2 chunks +0 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
jvoung (off chromium)
7 years, 4 months ago (2013-07-25 23:06:29 UTC) #1
Mark Seaborn
"Reduce confusing about which driver tools are supported." - Is there a noun missing here? ...
7 years, 4 months ago (2013-07-25 23:15:38 UTC) #2
jvoung (off chromium)
On 2013/07/25 23:15:38, Mark Seaborn wrote: > "Reduce confusing about which driver tools are supported." ...
7 years, 4 months ago (2013-07-25 23:19:38 UTC) #3
jvoung (off chromium)
7 years, 4 months ago (2013-07-26 16:11:22 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r11864 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698