Commit Graph

46 Commits (7f7915cb12b542872e7a288d43a5418399dd1be0)

Author SHA1 Message Date
maruel@chromium.org 5f3eee3bab gclient: remove wildcard import from git_scm
Part of a larger refactoring to abstract SCM-specific bits.

presubmit_support, revert, gcl: modify to import gclient_scm and gclient_utils

Part of a larger refactoring to abstract SCM-specific bits.

revert, gcl: modify to import gclient_scm and gclient_utils

Part of a larger refactoring to abstract SCM-specific bits.

gclient: pull out SCM bits

Pulled out SCMWrapper into gcliet_scm.py as part of a larger refactoring to
abstract SCM-specific bits. Plan is to evenutally add git support.

Pulling out SCMWrapper also required pulling out utility functions into
a gclient_utility.py.

Patch contributed by msb@

TEST=none
BUG=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@26423 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
dpranke@google.com a135e356ef Modify gcl.py so that lint ignores only webkit/api, and not webkit/*
R=pam
   TEST=none
   BUG=none

M    gcl.py



git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@26194 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
bradnelson@google.com 57e7855b1d Making gcl upload do tryupload from the same place as gcl try.
BUG=None
TEST=None

Review URL: http://codereview.chromium.org/200103

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@26044 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
pam@chromium.org f0dfba3f44 Add all modified files to new changelists by default.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/164084

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@22800 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
laforge@chromium.org 8b60d0d4b1 Revert 22561 - Minor change to gcl to automatically put changed files in the changelist (like we do with P4)
R=nsylvain
Review URL: http://codereview.chromium.org/164028

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@22608 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
laforge@chromium.org 1ef869cbb9 Minor change to gcl to automatically put changed files in the changelist (like we do with P4)
R=nsylvain
Review URL: http://codereview.chromium.org/164028

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@22561 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
estade@chromium.org bfd09ce886 Get rid of annoying empty changelists!
1) don't save empty changelists to begin with. This guards against an accidental or canceled "gcl change" creating an empty file.
2) create a "clearempties" command which clears all changelists that don't have any files in them.

Review URL: http://codereview.chromium.org/160139

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@22538 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org a05be0ba8d Fix 'gcl help' when not run inside a svn checkout.
TEST=none
BUG=none

Review URL: http://codereview.chromium.org/150114

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@19628 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 2e50180403 Fix gcl presubmit usage.
TEST=none
BUG=none
TBR=nsylvain

Review URL: http://codereview.chromium.org/125079

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18325 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
nirnimesh@chromium.org 07f0186d08 Really really fix gcl.py
TEST=none
BUG=none
TBR=maruel

Review URL: http://codereview.chromium.org/126057

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18279 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 0b7574dc41 Fix gcl upload for real this time.
I had used the old method name. watchlists needs a unit test.

TEST=none
BUG=none
TBR=nirnimesh

Review URL: http://codereview.chromium.org/125053

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18278 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 5159b2e26c Fix watchlist exception on upload.
TEST=none
BUG=none
TBR=nirnimesh

Review URL: http://codereview.chromium.org/126051

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18274 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 8d5c9a5a52 Fix gcl breakage.
Small refactor of ChangeInfo to not hardcode the use of GetRepositoryRoot().

TEST=unit tests
BUG=none

Review URL: http://codereview.chromium.org/125051

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18272 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 17f59f2dd9 A step closer to make presubmit SCM independent.
Stop presubmit_support.py from calling gcl.GetRepositoryRoot(). Instead, save the repository root in ChangeInfo and name it GetLocalRoot() to make it less confusing.
Add GetFileList() and GetFileNames() to ChangeInfo to make it less confusing and more "interface-like".

TEST=updated unit tests with a lot of pain
BUG=none

Review URL: http://codereview.chromium.org/119442

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18266 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org a005ccdead Fix 'gcl settings' output.
TEST=none
BUG=none

Review URL: http://codereview.chromium.org/126032

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18265 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
nirnimesh@chromium.org b2ab494a8b Add watchlists to 'gcl upload'
Watchlists provides an opportunity for someone interested in a particular
code section to make comments before commit. One can subscribe to watchlists
depending on filepath (regular expressions) and she'll automatically get
cc-ed to changes when the watchlist is triggered.

Additional examples of watchlist_rules in diff repo:
http://codereview.chromium.org/118431
http://codereview.chromium.org/119356
Review URL: http://codereview.chromium.org/118432

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18209 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 21b893b71c Stop showing "Presubmit errors, can't continue (use --no_presubmit to bypass)" when running gcl presubmit!
TEST=none
BUG=none

Review URL: http://codereview.chromium.org/119436

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18078 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org b0dfd35742 Make the presubmit output much more compact.
Disable prompting when using gcl presubmit xxx.

TEST=none
BUG=none

Review URL: http://codereview.chromium.org/118498

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@18054 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 51ee007126 Add --force as a synonym for --no_presubmit as per Nicolas' request.
Factor out some code in FilterFlag() to make the code much more readable.

TEST=none
BUG=none

Review URL: http://codereview.chromium.org/119315

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@17884 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 2c8d4b29b9 TBR: nsylvain
git-cl is broken

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@17844 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 6cfadff1e9 gcl change <change> was broken when creating a new change list.
TEST=none
BUG=none

Review URL: http://codereview.chromium.org/119286

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@17841 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 32ba260f1e Added patchset persistence in gcl, this is necessary to implement a presubmit queue.
Now enforces ChangeInfo.issue (and .patchset) to be a int.
Renamed LoadChangelistInfo() to ChangeInfo.Load().
Removed ChangeInfo.__init__() default parameters.

TEST=none
BUG=none

Review URL: http://codereview.chromium.org/118355

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@17836 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org b044218cca Fix gcl in the case there is no CODEREVIEW_SETTINGS file in the repository.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/118265

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@17727 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
laforge@chromium.org 85532fc2e5 Added some new functionality to gcl:
- Create a new parameter for "gcl change 12354 <filename>" to read comments from a text file.  This allows us to programmatically inject comments into a gcl upload
 - Added helper methods in change to order of which other_files have changed (brings changed files to the top of the list).  This makes it easier to find which files have change, and simply cut and paste the top of the list (after many merges, the list of unmodified files grows to be quite extensive... very useful)

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@17684 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 3b217f537f Remove the tree open check since it's now a presubmit script.
Review URL: http://codereview.chromium.org/115996

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@17330 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 1bb04aa1c0 Remove the use of --non-recursive flag.
Review URL: http://codereview.chromium.org/115994

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@17328 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 0ff1fab804 Add repository root presubmit support.
Review URL: http://codereview.chromium.org/115616

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@16738 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 053ba90ed1 Use the --non-recursive flag when committing.
BUG=12339
TEST=see bug for repro steps.

Review URL: http://codereview.chromium.org/113709

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@16620 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 98fc2b9bf6 Add support to cache arbitrary file.
Rename repository_root to REPOSITORY_ROOT.

Review URL: http://codereview.chromium.org/115615

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@16598 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 374d65e518 Move the changelist files into a subdirectory for cached files implementation.
Review URL: http://codereview.chromium.org/115612

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@16597 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 8c3ccf39eb Fix error printing in gcl.GetRepositoryRoot() and fix its unit tests.
Review URL: http://codereview.chromium.org/113650

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@16499 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 003c269b98 When -s or --server is used, don't try the patch.
Review URL: http://codereview.chromium.org/115534

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@16477 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 1033accb23 Rename presubmit.py to presubmit_support.py to reduce confusion and be able to add a PRESUBMIT.py check into depot_tools.
Review URL: http://codereview.chromium.org/113337

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15954 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org c315020307 Fix a regression on svn cp|ren|mv on gcl upload.
BUG=11884
Review URL: http://codereview.chromium.org/113341

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15953 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
estade@chromium.org 3bcc6cebff Pass "gcl help upload" to upload.py.
Review URL: http://codereview.chromium.org/113303

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15911 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 4810a96600 Deprecate gcl.GetSVNStatus() for gclient.CaptureSVNStatus() and fix some status information that was lost.
This was breaking gclient revert on unversioned files.

TEST=ran "for %a in (tests\*test.py) do python %a" and all tests succeeded.
Review URL: http://codereview.chromium.org/115264

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15894 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 46a9410947 Deprecate gcl.GetSVNFileInfo() for gclient.CaptureSVNInfo().
Review URL: http://codereview.chromium.org/113290

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15891 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org be0d1ca0e4 Add more tests and cleanup ChangeInfo class.
Review URL: http://codereview.chromium.org/113269

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15885 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 11e0fd6997 Revert changes 15823 and 15824 because they broke gclient revert.
TBR=sgk
Review URL: http://codereview.chromium.org/115217

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15827 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 46b9ae6961 Starts reusing functions in gclient.py from gcl.py.
Review URL: http://codereview.chromium.org/113218

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15824 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
thestig@chromium.org 8a62d5b229 Fix incorrect variable in LoadChangelistInfo and recognize directories properly in GenerateDiff.
Review URL: http://codereview.chromium.org/115179

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15753 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
mal@chromium.org 61c5fdf4e3 Don't throw an exception for files with only property changes.
R= maruel
See also: http://codereview.chromium.org/99317

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@15157 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org 207fdf3647 Convert gcl.GetSVNFileInfo() to xml format.
Add gcl's first unit test.
Add version string.
Review URL: http://codereview.chromium.org/100095

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@14771 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@chromium.org c1675e27e6 Add an optional flag to gcl.ReadFile() so that presubmit can support CRLF PRESUBMIT.py scripts.
Also add a version number to gcl.py.
Review URL: http://codereview.chromium.org/99072

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@14644 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@google.com 9b613270be Fix gclient on posix platforms.
Grab the latest gcl.py and presubmit.py scripts.

TBR=nsylvain
Review URL: http://codereview.chromium.org/93108

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@14403 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago
maruel@google.com fb2b8eb2e2 Create the Next Generation of depot_tools. Eh.
Review URL: http://codereview.chromium.org/92087

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@14349 0039d316-1c4b-4281-b951-d872f2087c98
16 years ago