<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/distcc-git.git/src/emaillog.c, branch python3</title>
<subtitle>github.com: distcc/distcc.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/'/>
<entry>
<title>Fix warnings about unused return values.  This includes some code changes</title>
<updated>2011-04-06T14:35:28+00:00</updated>
<author>
<name>fergus.henderson</name>
<email>fergushenderson@users.noreply.github.com</email>
</author>
<published>2011-04-06T14:35:28+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=2e8b156f51821b9a6b3dbf38fd978d3581fe0afa'/>
<id>2e8b156f51821b9a6b3dbf38fd978d3581fe0afa</id>
<content type='text'>
to do a better job of error checking.

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
to do a better job of error checking.

</pre>
</div>
</content>
</entry>
<entry>
<title>Apply patch from Akim Demaille &lt;demaille@gostai.com&gt;:</title>
<updated>2010-03-02T08:13:44+00:00</updated>
<author>
<name>fergus.henderson</name>
<email>fergushenderson@users.noreply.github.com</email>
</author>
<published>2010-03-02T08:13:44+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=8b8887432173894c3793a8d78672e2072f54b9ef'/>
<id>8b8887432173894c3793a8d78672e2072f54b9ef</id>
<content type='text'>
       * src/emaillog.c (dcc_maybe_send_email): Pacify GCC's warnings
       about format strings by getting rid of a useless temporary
       string.

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
       * src/emaillog.c (dcc_maybe_send_email): Pacify GCC's warnings
       about format strings by getting rid of a useless temporary
       string.

</pre>
</div>
</content>
</entry>
<entry>
<title>Ensure that all ".c" files start with "#include &lt;config.h&gt;".</title>
<updated>2009-02-17T23:10:59+00:00</updated>
<author>
<name>fergus.henderson</name>
<email>fergushenderson@users.noreply.github.com</email>
</author>
<published>2009-02-17T23:10:59+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=99f367ba44092015293497f8540524d592c52a50'/>
<id>99f367ba44092015293497f8540524d592c52a50</id>
<content type='text'>
This will hopefully fix issue 39 &lt;http://code.google.com/p/distcc/issues/detail?id=39&gt;.
Some header files were using #if statements that depended on macros defined in config.h
without #including config.h first.  The result was a compilation error, because
"#if !HAVE_DECL_SNPRINTF" was evaluating to true because config.h hadn't been included,
causing us to declare snprintf() when we shouldn't, leading to a conflict with the
declaration in the standard header files.

(The alternative solution would have been to add "#include &lt;config.h&gt;"
to every header file that uses '#if'.  I decided not to do that, just
for consistency with the existing code.)

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This will hopefully fix issue 39 &lt;http://code.google.com/p/distcc/issues/detail?id=39&gt;.
Some header files were using #if statements that depended on macros defined in config.h
without #including config.h first.  The result was a compilation error, because
"#if !HAVE_DECL_SNPRINTF" was evaluating to true because config.h hadn't been included,
causing us to declare snprintf() when we shouldn't, leading to a conflict with the
declaration in the standard header files.

(The alternative solution would have been to add "#include &lt;config.h&gt;"
to every header file that uses '#if'.  I decided not to do that, just
for consistency with the existing code.)

</pre>
</div>
</content>
</entry>
<entry>
<title>Fix some of the written-on dates for some google patches.  Pointed out</title>
<updated>2008-05-29T01:46:50+00:00</updated>
<author>
<name>Craig Silverstein</name>
<email>csilvers@khanacademy.org</email>
</author>
<published>2008-05-29T01:46:50+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=218ce6d045d52a75682178ee4dcd5a1849130779'/>
<id>218ce6d045d52a75682178ee4dcd5a1849130779</id>
<content type='text'>
by fergus.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
by fergus.
</pre>
</div>
</content>
</entry>
<entry>
<title>Normalize the copyright text, to the latest version of the GPLv2 text</title>
<updated>2008-05-23T22:24:20+00:00</updated>
<author>
<name>Craig Silverstein</name>
<email>csilvers@khanacademy.org</email>
</author>
<published>2008-05-23T22:24:20+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=51f440cfad4a7b861b774dffb0d8e92db09ca84c'/>
<id>51f440cfad4a7b861b774dffb0d8e92db09ca84c</id>
<content type='text'>
(typically the only change is in the FSF street address).  Add Google
copyright line in some places it was missing.  Add GPLv2 notice to
avahi patches.

OKed by fergus
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
(typically the only change is in the FSF street address).  Add Google
copyright line in some places it was missing.  Add GPLv2 notice to
avahi patches.

OKed by fergus
</pre>
</div>
</content>
</entry>
<entry>
<title>Change C++-style // comments to /**/.</title>
<updated>2008-05-20T18:16:38+00:00</updated>
<author>
<name>Craig Silverstein</name>
<email>csilvers@khanacademy.org</email>
</author>
<published>2008-05-20T18:16:38+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=458b09d3c0df853a872a2fb214dfbe2aaf7117f8'/>
<id>458b09d3c0df853a872a2fb214dfbe2aaf7117f8</id>
<content type='text'>
Also, fix the emacs var-setting line: it was missing a semicolon
before, which means the entire line was being ignored.

No contentful change.

Reviewed by fergus@google.com
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Also, fix the emacs var-setting line: it was missing a semicolon
before, which means the entire line was being ignored.

No contentful change.

Reviewed by fergus@google.com
</pre>
</div>
</content>
</entry>
<entry>
<title>Turned all tabs into 4 spaces.  Got rid of whitespace at the end of</title>
<updated>2008-05-20T06:13:28+00:00</updated>
<author>
<name>Craig Silverstein</name>
<email>csilvers@khanacademy.org</email>
</author>
<published>2008-05-20T06:13:28+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=41b794c77d881e29b260d5fccd14293af499b768'/>
<id>41b794c77d881e29b260d5fccd14293af499b768</id>
<content type='text'>
lines.  Fixed up resulting mis-indented code I noticed (mostly in
files that used 8 space indents, or used 4-space and 8-space indents
in the same file (!)).  Added the emacs tab-var setting for all files,
not just some of them.

I also added in copyright notices for files I noticed that didn't have
them.  We'll want to do another pass-through to fix these up properly,
though.

I used the following perl snippet to check for mis-indented code after
converting tabs to whitespace:

  $ for i in *.{c,h}; do echo $i; perl -nle 'if ($indent &gt; 0) {$sp=" " x $indent; /^$sp[^ ]/ &amp;&amp; print "$.: $_"; $indent=0;}; if (/^( *).*{/ ) {$indent=length($1);} else {$indent=0;}' $i; done | less

It had false positives, but hopefully didn't miss anything.

Reviewed by klarlund@google.com
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
lines.  Fixed up resulting mis-indented code I noticed (mostly in
files that used 8 space indents, or used 4-space and 8-space indents
in the same file (!)).  Added the emacs tab-var setting for all files,
not just some of them.

I also added in copyright notices for files I noticed that didn't have
them.  We'll want to do another pass-through to fix these up properly,
though.

I used the following perl snippet to check for mis-indented code after
converting tabs to whitespace:

  $ for i in *.{c,h}; do echo $i; perl -nle 'if ($indent &gt; 0) {$sp=" " x $indent; /^$sp[^ ]/ &amp;&amp; print "$.: $_"; $indent=0;}; if (/^( *).*{/ ) {$indent=length($1);} else {$indent=0;}' $i; done | less

It had false positives, but hopefully didn't miss anything.

Reviewed by klarlund@google.com
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix polarity of condition.</title>
<updated>2008-05-20T03:32:22+00:00</updated>
<author>
<name>klarlund</name>
<email>klarlund@gmail.com</email>
</author>
<published>2008-05-20T03:32:22+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=9268767b63b4ffeb6180be675e4735c4161c1614'/>
<id>9268767b63b4ffeb6180be675e4735c4161c1614</id>
<content type='text'>
Reviewers: fergus@google.com
Tests: manual (both with and without DISTCC_EMAILLOG_WHOM_TO_BLAME set).

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reviewers: fergus@google.com
Tests: manual (both with and without DISTCC_EMAILLOG_WHOM_TO_BLAME set).

</pre>
</div>
</content>
</entry>
<entry>
<title>These changes are enough to get code to compile, and at least some of</title>
<updated>2008-04-28T19:23:31+00:00</updated>
<author>
<name>Craig Silverstein</name>
<email>csilvers@khanacademy.org</email>
</author>
<published>2008-04-28T19:23:31+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=f1373cdf3c33752ec326c9b79378057367836c4c'/>
<id>f1373cdf3c33752ec326c9b79378057367836c4c</id>
<content type='text'>
the unittests to pass, on FreeBSD 6.0, Solaris 10 x86, and OS X
Leopard.  You have to use gmake instead of standard bsd make, though,
because neither bsd nor solari make understand 'include */*.d' and
'CFLAGS += $(POPT_FLAGS).'  These may be fixable later.

Most changes fall into four categories:
1) #include differences
2) New errors due to some #ifdef paths being taken differently
3) Undefined functions (see, eg, the new HAVE_STRSEP)
4) Type differences (eg tv_usec is an int on os x, not a long int)

As one concrete example, snprintf.c is an empty file on linux, where
all the functionality is part of glibc.  But on Solaris, some of its
functionality is useful.  This turned up a bug where if you have
vsnprintf on your system but not vasnprintf, then dopr() was not being
used but was being defined, leading to an "unused static function"
warning in gcc.

As another, solaris would complain about "index" being used as a
variable, since it's also a function name.  The var was renamed idx.

Tested by compiling on FreeBSD, Solaris 10, and OS X leopard (x86).  I
also compiled and ran unittests under Linux Ubuntu, to make sure
this change didn't break anything there.

Reviewed by fergus@google.com
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
the unittests to pass, on FreeBSD 6.0, Solaris 10 x86, and OS X
Leopard.  You have to use gmake instead of standard bsd make, though,
because neither bsd nor solari make understand 'include */*.d' and
'CFLAGS += $(POPT_FLAGS).'  These may be fixable later.

Most changes fall into four categories:
1) #include differences
2) New errors due to some #ifdef paths being taken differently
3) Undefined functions (see, eg, the new HAVE_STRSEP)
4) Type differences (eg tv_usec is an int on os x, not a long int)

As one concrete example, snprintf.c is an empty file on linux, where
all the functionality is part of glibc.  But on Solaris, some of its
functionality is useful.  This turned up a bug where if you have
vsnprintf on your system but not vasnprintf, then dopr() was not being
used but was being defined, leading to an "unused static function"
warning in gcc.

As another, solaris would complain about "index" being used as a
variable, since it's also a function name.  The var was renamed idx.

Tested by compiling on FreeBSD, Solaris 10, and OS X leopard (x86).  I
also compiled and ran unittests under Linux Ubuntu, to make sure
this change didn't break anything there.

Reviewed by fergus@google.com
</pre>
</div>
</content>
</entry>
<entry>
<title>The first step of moving everything in the distcc directory to the top</title>
<updated>2008-04-11T22:36:40+00:00</updated>
<author>
<name>Craig Silverstein</name>
<email>csilvers@khanacademy.org</email>
</author>
<published>2008-04-11T22:36:40+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/distcc-git.git/commit/?id=596cf4e2f019a766965ad0566495c56b7d295fe4'/>
<id>596cf4e2f019a766965ad0566495c56b7d295fe4</id>
<content type='text'>
level.  I'm doing this in two stages, because I don't understand svn
enough to be confident to do it in one.  This first stage just copies
all the files from distcc/FOO to FOO.  Now there are two copies of
each file under distcc; the Makefile/etc uses the one in distcc and
ignores the one at the top level.

The next commit will delete everything under distcc, and rewrite the
Makefile/etc to use the top-level versions instead.


</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
level.  I'm doing this in two stages, because I don't understand svn
enough to be confident to do it in one.  This first stage just copies
all the files from distcc/FOO to FOO.  Now there are two copies of
each file under distcc; the Makefile/etc uses the one in distcc and
ignores the one at the top level.

The next commit will delete everything under distcc, and rewrite the
Makefile/etc to use the top-level versions instead.


</pre>
</div>
</content>
</entry>
</feed>
