<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/git-p4.py, branch v2.41.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.41.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.41.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2022-08-01T16:58:37Z</updated>
<entry>
<title>Merge branch 'kk/p4-client-name-encoding-fix'</title>
<updated>2022-08-01T16:58:37Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2022-08-01T16:58:37Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=e59acea3f0752ddfdcaab08b54dc0f84c7ab66e6'/>
<id>urn:sha1:e59acea3f0752ddfdcaab08b54dc0f84c7ab66e6</id>
<content type='text'>
"git p4" did not handle non-ASCII client name well, which has been
corrected.
source: &lt;pull.1285.v3.git.git.1658394440.gitgitgadget@gmail.com&gt;

* kk/p4-client-name-encoding-fix:
  git-p4: refactoring of p4CmdList()
  git-p4: fix bug with encoding of p4 client name
</content>
</entry>
<entry>
<title>Merge branch 'mb/p4-utf16-crlf'</title>
<updated>2022-07-27T16:16:55Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2022-07-27T16:16:55Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=745af8c7b5506f17516e21629386191acb26aae7'/>
<id>urn:sha1:745af8c7b5506f17516e21629386191acb26aae7</id>
<content type='text'>
"git p4" working on UTF-16 files on Windows did not implement
CRLF-to-LF conversion correctly, which has been corrected.

* mb/p4-utf16-crlf:
  git-p4: fix CR LF handling for utf16 files
</content>
</entry>
<entry>
<title>git-p4: refactoring of p4CmdList()</title>
<updated>2022-07-21T16:47:06Z</updated>
<author>
<name>Kilian Kilger</name>
<email>kkilger@gmail.com</email>
</author>
<published>2022-07-21T09:07:20Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=d205483695366803e912da6ae24380c33c293467'/>
<id>urn:sha1:d205483695366803e912da6ae24380c33c293467</id>
<content type='text'>
The function p4CmdList executes a Perforce command and
decodes the marshalled python dictionary. Special care has to be
taken for certain dictionary values which contain non-unicode characters.
The old handling contained separate hacks for each of the corresponding
dictionary keys. This commit tries to refactor the coding to handle the
special cases uniformely.

Signed-off-by: Kilian Kilger &lt;kkilger@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: fix error handling in P4Unshelve.renameBranch()</title>
<updated>2022-07-20T20:53:54Z</updated>
<author>
<name>Moritz Baumann</name>
<email>moritz.baumann@sap.com</email>
</author>
<published>2022-07-20T18:55:30Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=198551ca54f6ff1c95c9357ccca9ae37298811bf'/>
<id>urn:sha1:198551ca54f6ff1c95c9357ccca9ae37298811bf</id>
<content type='text'>
The error handling code path is meant to be triggered when the loop does
not exit early via "break". This fails, as the boolean variable "found",
which is used to track whether the loop was exited early, is initialized
incorrectly.

It would be possible to fix this issue by correcting the initialization,
but Python supports a for:-else: control flow construct for this exact
use case (executing code if a loop does not exit early), so it is more
idiomatic to remove the tracking variable entirely.

In addition, the error message no longer refers to a variable that does
not exist.

Signed-off-by: Moritz Baumann &lt;moritz.baumann@sap.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: fix typo in P4Submit.applyCommit()</title>
<updated>2022-07-20T20:53:52Z</updated>
<author>
<name>Moritz Baumann</name>
<email>moritz.baumann@sap.com</email>
</author>
<published>2022-07-20T18:55:29Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=c0d2b07460ab9e42ac398316544ed3eb43f9a998'/>
<id>urn:sha1:c0d2b07460ab9e42ac398316544ed3eb43f9a998</id>
<content type='text'>
Signed-off-by: Moritz Baumann &lt;moritz.baumann@sap.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: fix CR LF handling for utf16 files</title>
<updated>2022-07-20T18:38:06Z</updated>
<author>
<name>Moritz Baumann</name>
<email>moritz.baumann@sap.com</email>
</author>
<published>2022-07-20T18:17:45Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=4d35f744219335d8b32df363891b6336dcf02a6e'/>
<id>urn:sha1:4d35f744219335d8b32df363891b6336dcf02a6e</id>
<content type='text'>
Perforce silently replaces LF with CR LF for "utf16" files if the client
is a native Windows client. Since git's autocrlf logic does not undo
this transformation for UTF-16 encoded files, git-p4 replaces CR LF with
LF during the sync if the file type "utf16" is detected and the Perforce
client platform indicates that this conversion is performed.

Windows only runs on little-endian architectures, therefore the encoding
of the byte stream received from the Perforce client is UTF-16-LE and
the relevant byte sequence is 0D 00 0A 00.

Signed-off-by: Moritz Baumann &lt;moritz.baumann@sap.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-p4: fix bug with encoding of p4 client name</title>
<updated>2022-07-08T14:59:12Z</updated>
<author>
<name>Kilian Kilger</name>
<email>kilian.kilger@sap.com</email>
</author>
<published>2022-07-08T08:01:00Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=34f67c9619b56ab4718427c1a31071641f0dccdb'/>
<id>urn:sha1:34f67c9619b56ab4718427c1a31071641f0dccdb</id>
<content type='text'>
The Perforce client name can contain arbitrary characters
which do not decode to UTF-8. Use the fallback strategy
implemented in metadata_stream_to_writable_bytes() also
for the client name.

Signed-off-by: Kilian Kilger &lt;kkilger@gmail.com&gt;
Reviewed-by: Tao Klerks &lt;tao@klerks.biz&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'tk/p4-metadata-coding-strategies'</title>
<updated>2022-05-20T22:27:00Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2022-05-20T22:27:00Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=3af1df0415741c85f84e78e281339186be1d05d7'/>
<id>urn:sha1:3af1df0415741c85f84e78e281339186be1d05d7</id>
<content type='text'>
"git p4" updates.

* tk/p4-metadata-coding-strategies:
  git-p4: improve encoding handling to support inconsistent encodings
</content>
</entry>
<entry>
<title>Merge branch 'kf/p4-multiple-remotes'</title>
<updated>2022-05-20T22:26:55Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2022-05-20T22:26:55Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=586f23705c639d08fc3c1f1489cdec57e18a1fe9'/>
<id>urn:sha1:586f23705c639d08fc3c1f1489cdec57e18a1fe9</id>
<content type='text'>
"git p4" update.

* kf/p4-multiple-remotes:
  git-p4: fix issue with multiple perforce remotes
</content>
</entry>
<entry>
<title>Merge branch 'tk/p4-with-explicity-sync'</title>
<updated>2022-05-20T22:26:55Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2022-05-20T22:26:55Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=af3a3205d14561e1683ca86b894888f9f1fc0176'/>
<id>urn:sha1:af3a3205d14561e1683ca86b894888f9f1fc0176</id>
<content type='text'>
"git p4" update.

* tk/p4-with-explicity-sync:
  git-p4: support explicit sync of arbitrary existing git-p4 refs
</content>
</entry>
</feed>
