Change DEPS hooks paths to include 'pdfium/'.
authorOliver Chang <ochang@chromium.org>
Wed, 21 Oct 2015 19:05:57 +0000 (12:05 -0700)
committerOliver Chang <ochang@chromium.org>
Wed, 21 Oct 2015 19:05:57 +0000 (12:05 -0700)
This will break existing checkouts based on the instructions provided.

Instead of having a single pdfium directory, checkouts will now compromise of:
("repo" can be named anything)

repo/.gclient
repo/pdfium/.git
repo/pdfium/others...

To convert an existing checkout, do something like:

mkdir repo
mv pdfium repo
rm repo/pdfium/.gclient_entries # will be regenerated, with a warning
mv repo/pdfium/.gclient repo
edit repo/.gclient and change "name: '.'" to "name: 'pdfium'"

Instructions for getting a new checkout are in README.md in this CL.

R=thestig@chromium.org, tsepez@chromium.org, thakis@chromium.org

Review URL: https://codereview.chromium.org/1406383003 .

DEPS
README.md

diff --git a/DEPS b/DEPS
index 0c75f5e..455150d 100644 (file)
--- a/DEPS
+++ b/DEPS
@@ -40,7 +40,7 @@ hooks = [
     # A change to a .gyp, .gypi, or to GYP itself should run the generator.
     'name': 'gyp',
     'pattern': '.',
-    'action': ['python', 'build/gyp_pdfium'],
+    'action': ['python', 'pdfium/build/gyp_pdfium'],
   },
   # Pull clang-format binaries using checked-in hashes.
   {
@@ -51,7 +51,7 @@ hooks = [
                 '--platform=win32',
                 '--no_auth',
                 '--bucket', 'chromium-clang-format',
-                '-s', 'buildtools/win/clang-format.exe.sha1',
+                '-s', 'pdfium/buildtools/win/clang-format.exe.sha1',
     ],
   },
   {
@@ -62,7 +62,7 @@ hooks = [
                 '--platform=darwin',
                 '--no_auth',
                 '--bucket', 'chromium-clang-format',
-                '-s', 'buildtools/mac/clang-format.sha1',
+                '-s', 'pdfium/buildtools/mac/clang-format.sha1',
     ],
   },
   {
@@ -73,7 +73,7 @@ hooks = [
                 '--platform=linux*',
                 '--no_auth',
                 '--bucket', 'chromium-clang-format',
-                '-s', 'buildtools/linux64/clang-format.sha1',
+                '-s', 'pdfium/buildtools/linux64/clang-format.sha1',
     ],
   },
 ]
index b297e87..67d8448 100644 (file)
--- a/README.md
+++ b/README.md
@@ -10,11 +10,16 @@ Also install Python, Subversion, and Git and make sure they're in your path.
 
 ## Get the code
 
+The name of the top-level directory does not matter. In our examples, we use
+"repo". This directory must not have been used before by `gclient config` as
+each directory can only house a single gclient configuration.
+
 ```
-mkdir pdfium
-cd pdfium
-gclient config --name . --unmanaged https://pdfium.googlesource.com/pdfium.git
+mkdir repo
+cd repo
+gclient config --unmanaged https://pdfium.googlesource.com/pdfium.git
 gclient sync
+cd pdfium
 ```
 
 ## Generate the build files