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

Unified Diff: docs/linux_eclipse_dev.md

Issue 2122603005: Update Linux eclipse docs for GN (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: docs/linux_eclipse_dev.md
diff --git a/docs/linux_eclipse_dev.md b/docs/linux_eclipse_dev.md
index 78cee842f3db3da4f6240898a25383b62d2d3fc3..09946b53ff0cd0ebb6add4c1ca58162975240864 100644
--- a/docs/linux_eclipse_dev.md
+++ b/docs/linux_eclipse_dev.md
@@ -287,20 +287,13 @@ requires the Eclipse knows correct include paths and pre-processor definitions.
After fighting with with a number of approaches, I've found the below to work
best for me.
-*The instrcutions below are out-of-date since it references GYP. Please see
-`gn help gen` for how to generate an Eclipse CDT file in GN. If you use
-Eclipse and make it work, please update this documentation.*
-
1. From a shell in your src directory, run
- `GYP_GENERATORS=ninja,eclipse build/gyp_chromium`
- 1. This generates <project root>/out/Debug/eclipse-cdt-settings.xml which
+ `gn gen --ide=eclipse out/Debug_gn/' (replacing Debug_gn with the output directory you normally use when building).
+ 1. This generates <project root>/out/Debug_gn/eclipse-cdt-settings.xml which
is used below.
1. This creates a single list of include directories and preprocessor
definitions to be used for all source files, and so is a little
inaccurate. Here are some tips for compensating for the limitations:
- 1. Use `-R <target>` to restrict the output to considering only certain
- targets (avoiding unnecessary includes that are likely to cause
- trouble). Eg. for a blink project, use `-R blink`.
Dirk Pranke 2016/07/06 18:19:26 You might replace this with a mention of the --fil
1. If you care about blink, move 'third\_party/Webkit/Source' to the
top of the list to better resolve ambiguous include paths (eg.
`config.h`).
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698