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

Issue 131343010: [NaCl SDK Docs] Remove /tmp path from generated docs. (Closed)

Created:
6 years, 11 months ago by binji
Modified:
6 years, 11 months ago
Reviewers:
Sam Clegg
CC:
chromium-reviews, native-client-reviews_googlegroups.com, Sam Clegg
Visibility:
Public.

Description

[NaCl SDK Docs] Remove /tmp path from generated docs. When running generate_docs.py, Doxygen would generate the docs from source files in a directory like /tmp/pepper_stableXXXXX. These paths were being displayed in the generated docs. This change makes the filenames relative to ppapi/c or ppapi/cpp. This CL also includes the changes to the generated docs; I haven't uploaded them to Rietveld because there are too many files. BUG=none R=sbc@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=246968

Patch Set 1 #

Patch Set 2 : example of change #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -3 lines) Patch
M native_client_sdk/doc_generated/pepper_beta/c/pp__array__output_8h.html View 1 2 chunks +3 lines, -3 lines 0 comments Download
M native_client_sdk/doc_generated/pepper_beta/c/pp__array__output_8h__incl.png View 1 Binary file 0 comments Download
M native_client_sdk/src/doc/doxygen/Doxyfile.c.template View 1 chunk +1 line, -0 lines 0 comments Download
M native_client_sdk/src/doc/doxygen/Doxyfile.cpp.template View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
binji
6 years, 11 months ago (2014-01-24 18:05:22 UTC) #1
Sam Clegg
Can you add one (or a sample) of the generated docs to the CL so ...
6 years, 11 months ago (2014-01-24 18:09:12 UTC) #2
binji
Done.
6 years, 11 months ago (2014-01-24 18:22:25 UTC) #3
Sam Clegg
lgtm
6 years, 11 months ago (2014-01-24 18:25:19 UTC) #4
binji
6 years, 11 months ago (2014-01-24 21:21:48 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r246968 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698