Merge lp:~flameeyes/gearmand/gcc47 into lp:gearmand/1.0

Proposed by Diego Elio Pettenò
Status: Merged
Merge reported by: Brian Aker
Merged at revision: not available
Proposed branch: lp:~flameeyes/gearmand/gcc47
Merge into: lp:gearmand/1.0
Diff against target: 11 lines (+1/-0)
1 file modified
util/logfile.cc (+1/-0)
To merge this branch: bzr merge lp:~flameeyes/gearmand/gcc47
Reviewer Review Type Date Requested Status
Clint Byrum (community) Approve
Tangent Pending
Review via email: mp+111420@code.launchpad.net

Description of the change

This is a fix for the GCC 4.7 building bug I reported.

To post a comment you must log in.
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Diego, thanks for the submission, it looks good to me.

I wonder if we have any GCC 4.7 targets in our CI system. Seems like we should add Ubuntu quantal and maybe Debian wheezy since they default to 4.7.

I'll let Brian do the merge, as I think I'd like to also make sure we're actually testing against 4.7 before going any further.

review: Approve
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Actually it looks like Brian beat you to the punch a couple days ago:

revno: 578
committer: Brian Aker <email address hidden>
branch nick: gearmand-build
timestamp: Mon 2012-06-18 10:47:16 -0400
message:
  Update required include.
=== modified file 'util/logfile.cc'
--- util/logfile.cc 2012-04-28 08:01:43 +0000
+++ util/logfile.cc 2012-06-18 14:47:16 +0000
@@ -49,6 +49,7 @@
 #include <sstream>
 #include <sys/stat.h>
 #include <sys/types.h>
+#include <unistd.h>

 namespace datadifferential {
 namespace util {

Revision history for this message
Diego Elio Pettenò (flameeyes) wrote :

Why do I not see it when I branch trunk?

--
Diego Elio Pettenò

Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Excerpts from flameeyes's message of 2012-06-21 16:43:19 UTC:
> Why do I not see it when I branch trunk?
>

Its in lp:gearmand/build .. which is where things go to be CI tested
before merge into trunk.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'util/logfile.cc'
2--- util/logfile.cc 2012-04-28 08:01:43 +0000
3+++ util/logfile.cc 2012-06-21 15:03:28 +0000
4@@ -49,6 +49,7 @@
5 #include <sstream>
6 #include <sys/stat.h>
7 #include <sys/types.h>
8+#include <unistd.h>
9
10 namespace datadifferential {
11 namespace util {

Subscribers

People subscribed via source and target branches