<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/git-cvsserver.perl, branch v2.16.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.16.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.16.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2017-09-11T05:52:29Z</updated>
<entry>
<title>cvsserver: use safe_pipe_capture for `constant commands` as well</title>
<updated>2017-09-11T05:52:29Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2017-09-11T05:45:54Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=46203ac24dc7e6b5a8d4f1b024ed93591705d47b'/>
<id>urn:sha1:46203ac24dc7e6b5a8d4f1b024ed93591705d47b</id>
<content type='text'>
This is not strictly necessary, but it is a good code hygiene.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>cvsserver: use safe_pipe_capture instead of backticks</title>
<updated>2017-09-11T05:52:29Z</updated>
<author>
<name>joernchen</name>
<email>joernchen@phenoelit.de</email>
</author>
<published>2017-09-11T05:45:09Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=27dd73871f814062737c327103ee43f1eb7f30d9'/>
<id>urn:sha1:27dd73871f814062737c327103ee43f1eb7f30d9</id>
<content type='text'>
This makes the script pass arguments that are derived from end-user
input in safer way when invoking subcommands.

Reported-by: joernchen &lt;joernchen@phenoelit.de&gt;
Signed-off-by: joernchen &lt;joernchen@phenoelit.de&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>cvsserver: move safe_pipe_capture() to the main package</title>
<updated>2017-09-11T05:52:29Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2017-09-11T05:44:24Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=fce13af5d20cad8dcb2d0e47bcf01b6960f08e55'/>
<id>urn:sha1:fce13af5d20cad8dcb2d0e47bcf01b6960f08e55</id>
<content type='text'>
As a preparation for replacing `command` with a call to this
function from outside GITCVS::updater package, move it to the main
package.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>typofix: assorted typofixes in comments, documentation and messages</title>
<updated>2016-05-06T20:16:37Z</updated>
<author>
<name>Li Peng</name>
<email>lip@dtdream.com</email>
</author>
<published>2016-05-06T12:36:46Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=832c0e5e63a0f61c3788847d4a7abb82d9e86ef4'/>
<id>urn:sha1:832c0e5e63a0f61c3788847d4a7abb82d9e86ef4</id>
<content type='text'>
Many instances of duplicate words (e.g. "the the path") and
a few typoes are fixed, originally in multiple patches.

    wildmatch: fix duplicate words of "the"
    t: fix duplicate words of "output"
    transport-helper: fix duplicate words of "read"
    Git.pm: fix duplicate words of "return"
    path: fix duplicate words of "look"
    pack-protocol.txt: fix duplicate words of "the"
    precompose-utf8: fix typo of "sequences"
    split-index: fix typo
    worktree.c: fix typo
    remote-ext: fix typo
    utf8: fix duplicate words of "the"
    git-cvsserver: fix duplicate words

Signed-off-by: Li Peng &lt;lip@dtdream.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-cvsserver.perl: fix typo</title>
<updated>2016-02-17T18:00:53Z</updated>
<author>
<name>GyuYong Jung</name>
<email>obliviscence@gmail.com</email>
</author>
<published>2016-02-17T02:14:58Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=527d4a638e330f4efe459442bbfb0799d29fd73a'/>
<id>urn:sha1:527d4a638e330f4efe459442bbfb0799d29fd73a</id>
<content type='text'>
Signed-off-by: GyuYong Jung &lt;obliviscence@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'ak/cvsserver-stabilize-use-of-hash-keys'</title>
<updated>2013-11-04T22:58:05Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-11-04T22:58:05Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=ec787db662af05eb6b7dcd71bed1548055cf437a'/>
<id>urn:sha1:ec787db662af05eb6b7dcd71bed1548055cf437a</id>
<content type='text'>
* ak/cvsserver-stabilize-use-of-hash-keys:
  cvsserver: Determinize output to combat Perl 5.18 hash randomization
</content>
</entry>
<entry>
<title>cvsserver: Determinize output to combat Perl 5.18 hash randomization</title>
<updated>2013-10-30T17:30:30Z</updated>
<author>
<name>Anders Kaseorg</name>
<email>andersk@MIT.EDU</email>
</author>
<published>2013-10-30T08:44:43Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=9462953ad25f9ad6ef933908c317dcbbfd4397ed'/>
<id>urn:sha1:9462953ad25f9ad6ef933908c317dcbbfd4397ed</id>
<content type='text'>
Perl 5.18 randomizes the seed used by its hash function, so iterating
through hashes results in different orders from run to run:
  http://perldoc.perl.org/perl5180delta.html#Hash-overhaul

This usually broke t9400 (gitcvs.dbname, gitcvs.ext.dbname, when
running cmp on two .sqlite files) and t9402 (check [cvswork3] diff,
when running test_cmp on two diffs).

To fix this, hide the internal order of hashes with sort when sending
output or running database queries.

(An alternative workaround is PERL_HASH_SEED=0, but this seems nicer.)

Signed-off-by: Anders Kaseorg &lt;andersk@mit.edu&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'jc/cvsserver-perm-bit-fix'</title>
<updated>2013-09-18T18:48:02Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-09-18T18:48:01Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=2f46b53957dec3aaca206c6590f5e7f538fabde2'/>
<id>urn:sha1:2f46b53957dec3aaca206c6590f5e7f538fabde2</id>
<content type='text'>
"git cvsserver" computed the permission mode bits incorrectly for
executable files.

* jc/cvsserver-perm-bit-fix:
  cvsserver: pick up the right mode bits
</content>
</entry>
<entry>
<title>cvsserver: pick up the right mode bits</title>
<updated>2013-09-11T16:32:30Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-09-10T22:33:06Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=1b48d56cfb006baf12d34b5e3150ba2ec1900e08'/>
<id>urn:sha1:1b48d56cfb006baf12d34b5e3150ba2ec1900e08</id>
<content type='text'>
When determining the file mode from either ls-tree or diff-tree
output, we used to grab these octal mode string (typically 100644 or
100755) and then did

	$git_perms .= "r" if ( $mode &amp; 4 );
	$git_perms .= "w" if ( $mode &amp; 2 );
	$git_perms .= "x" if ( $mode &amp; 1 );

which was already wrong, as (100644 &amp; 4) is very different from
oct("100644") &amp; 4.  An earlier refactoring 2c3af7e7 (cvsserver:
factor out git-log parsing logic, 2012-10-13) further changed it to
pick the third octal digit (10*0*644 or 10*0*755) from the left and
then do the above conversion, which does not make sense, either.

Let's use the third digit from the last of the octal mode string to
make sure we get the executable and read bits right.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
Tested-by: Michael Cronenworth &lt;mike@cchtml.com&gt;
</content>
</entry>
<entry>
<title>use 'tree-ish' instead of 'treeish'</title>
<updated>2013-09-04T22:02:56Z</updated>
<author>
<name>Richard Hansen</name>
<email>rhansen@bbn.com</email>
</author>
<published>2013-09-04T19:04:30Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=bb8040f9f9274d739b894230959134584b156ae5'/>
<id>urn:sha1:bb8040f9f9274d739b894230959134584b156ae5</id>
<content type='text'>
Replace 'treeish' in documentation and comments with 'tree-ish' to
match gitglossary(7).

The only remaining instances of 'treeish' are:
  * variable, function, and macro names
  * "(also treeish)" in the definition of tree-ish in gitglossary(7)

Signed-off-by: Richard Hansen &lt;rhansen@bbn.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
