<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/git-p4.py, branch v2.19.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.19.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.19.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2018-08-01T20:37:18Z</updated>
<entry>
<title>git-p4: add the `p4-pre-submit` hook</title>
<updated>2018-08-01T20:37:18Z</updated>
<author>
<name>Chen Bin</name>
<email>chenbin.sh@gmail.com</email>
</author>
<published>2018-07-27T11:22:22Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=251c8c501faf7a7910edb7c9e19692988dcac6a8'/>
<id>urn:sha1:251c8c501faf7a7910edb7c9e19692988dcac6a8</id>
<content type='text'>
The `p4-pre-submit` hook is executed before git-p4 submits code.
If the hook exits with non-zero value, submit process not start.

Signed-off-by: Chen Bin &lt;chenbin.sh@gmail.com&gt;
Reviewed-by: Luke Diamand &lt;luke@diamand.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: python3: fix octal constants</title>
<updated>2018-06-19T16:34:32Z</updated>
<author>
<name>Luke Diamand</name>
<email>luke@diamand.org</email>
</author>
<published>2018-06-19T08:04:11Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=db2d997efa43596fe5860d1b1c0c410e99d96f67'/>
<id>urn:sha1:db2d997efa43596fe5860d1b1c0c410e99d96f67</id>
<content type='text'>
See PEP3127. Works fine with python2 as well.

Signed-off-by: Luke Diamand &lt;luke@diamand.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: python3: use print() function</title>
<updated>2018-06-19T16:34:32Z</updated>
<author>
<name>Luke Diamand</name>
<email>luke@diamand.org</email>
</author>
<published>2018-06-19T08:04:10Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=f2606b1797fdffe40b00a6766ca1df6e1e0f3a60'/>
<id>urn:sha1:f2606b1797fdffe40b00a6766ca1df6e1e0f3a60</id>
<content type='text'>
Replace calls to print ... with the function form, print(...), to
allow use with python3 as well as python2.x.

Converted using 2to3 (and some hand-editing).

Signed-off-by: Luke Diamand &lt;luke@diamand.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: python3: basestring workaround</title>
<updated>2018-06-19T16:34:32Z</updated>
<author>
<name>Luke Diamand</name>
<email>luke@diamand.org</email>
</author>
<published>2018-06-19T08:04:09Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=efdcc99263871818cfe297a5a981d5841c77ebac'/>
<id>urn:sha1:efdcc99263871818cfe297a5a981d5841c77ebac</id>
<content type='text'>
In Python3, basestring no longer exists, so use this workaround.

Signed-off-by: Luke Diamand &lt;luke@diamand.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: python3: remove backticks</title>
<updated>2018-06-19T16:34:32Z</updated>
<author>
<name>Luke Diamand</name>
<email>luke@diamand.org</email>
</author>
<published>2018-06-19T08:04:08Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=4d88519f6a2c7ea170f239a638137ce26d39cb11'/>
<id>urn:sha1:4d88519f6a2c7ea170f239a638137ce26d39cb11</id>
<content type='text'>
Backticks around a variable are a deprecated alias for repr().
This has been removed in python3, so just use the string
representation instead, which is equivalent.

Signed-off-by: Luke Diamand &lt;luke@diamand.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: python3: replace dict.has_key(k) with "k in dict"</title>
<updated>2018-06-19T16:34:31Z</updated>
<author>
<name>Luke Diamand</name>
<email>luke@diamand.org</email>
</author>
<published>2018-06-19T08:04:07Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=dba1c9d9f26ac9fce55d0bfde8a040700fc9ff52'/>
<id>urn:sha1:dba1c9d9f26ac9fce55d0bfde8a040700fc9ff52</id>
<content type='text'>
Python3 does not have the dict.has_key() function, so replace all
such calls with "k in dict". This will still work with python2.6
and python2.7.

Converted using 2to3 (plus some hand-editing)

Signed-off-by: Luke Diamand &lt;luke@diamand.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: python3: replace &lt;&gt; with !=</title>
<updated>2018-06-19T16:34:31Z</updated>
<author>
<name>Luke Diamand</name>
<email>luke@diamand.org</email>
</author>
<published>2018-06-19T08:04:06Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=fc35c9d5dc8fefbc29e9713e6b47eb4c19579d56'/>
<id>urn:sha1:fc35c9d5dc8fefbc29e9713e6b47eb4c19579d56</id>
<content type='text'>
The &lt;&gt; string inequality operator (which doesn't seem to be even
documented) no longer exists in python3. Replace with !=.

This still works with python2.

Signed-off-by: Luke Diamand &lt;luke@diamand.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'ld/git-p4-updates'</title>
<updated>2018-06-18T17:18:41Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2018-06-18T17:18:41Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=e638899470621738f32ef75dc0a092f6475a4a6e'/>
<id>urn:sha1:e638899470621738f32ef75dc0a092f6475a4a6e</id>
<content type='text'>
"git p4" updates.

* ld/git-p4-updates:
  git-p4: auto-size the block
  git-p4: narrow the scope of exceptions caught when parsing an int
  git-p4: raise exceptions from p4CmdList based on error from p4 server
  git-p4: better error reporting when p4 fails
  git-p4: add option to disable syncing of p4/master with p4
  git-p4: disable-rebase: allow setting this via configuration
  git-p4: add options --commit and --disable-rebase
</content>
</entry>
<entry>
<title>git-p4: auto-size the block</title>
<updated>2018-06-12T21:46:09Z</updated>
<author>
<name>Luke Diamand</name>
<email>luke@diamand.org</email>
</author>
<published>2018-06-08T20:32:48Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=3deed5e078f2998573589b1ca03d89f41ea346e9'/>
<id>urn:sha1:3deed5e078f2998573589b1ca03d89f41ea346e9</id>
<content type='text'>
git-p4 originally would fetch changes in one query. On large repos this
could fail because of the limits that Perforce imposes on the number of
items returned and the number of queries in the database.

To fix this, git-p4 learned to query changes in blocks of 512 changes,
However, this can be very slow - if you have a few million changes,
with each chunk taking about a second, it can be an hour or so.

Although it's possible to tune this value manually with the
"--changes-block-size" option, it's far from obvious to ordinary users
that this is what needs doing.

This change alters the block size dynamically by looking for the
specific error messages returned from the Perforce server, and reducing
the block size if the error is seen, either to the limit reported by the
server, or to half the current block size.

That means we can start out with a very large block size, and then let
it automatically drop down to a value that works without error, while
still failing correctly if some other error occurs.

Signed-off-by: Luke Diamand &lt;luke@diamand.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: narrow the scope of exceptions caught when parsing an int</title>
<updated>2018-06-12T21:46:09Z</updated>
<author>
<name>Luke Diamand</name>
<email>luke@diamand.org</email>
</author>
<published>2018-06-08T20:32:47Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=8fa0abf830ef3c8e97846cef0986589b49bcfe6f'/>
<id>urn:sha1:8fa0abf830ef3c8e97846cef0986589b49bcfe6f</id>
<content type='text'>
The current code traps all exceptions around some code which parses an
integer, and then talks to Perforce.

That can result in errors from Perforce being ignored. Change the code
to only catch the integer conversion exceptions.

Signed-off-by: Luke Diamand &lt;luke@diamand.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
