Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(23)

Issue 989043: test binaries are in the builddir, not the top_srcdir (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years ago by Pierre Phaneuf (Google)
Modified:
14 years ago
Base URL:
http://google-glog.googlecode.com/svn/trunk/
Visibility:
Public.

Description

This is so you can build and run tests out-of-tree ("svn co ... glog && mkdir build && cd build && ../glog/configure && make && make check"). Unfortunately, it's not enough, seems some tests are relying on FLAGS_test_srcdir (which, somehow, should be set to either $(srcdir) or $(top_srcdir), I think?). This only gets it going a bit further, only to fail on those other issues, but it's a step in the right direction...

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M Makefile.am View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4
Pierre Phaneuf (Google)
We're using glog in http://dnproxy.googlecode.com/, and we build out-of-tree all the time, noticed that "make ...
14 years ago (2010-04-27 19:35:38 UTC) #1
jeffbailey
lgtm
14 years ago (2010-04-27 19:43:32 UTC) #2
Pierre Phaneuf (Google)
Ping? I actually can't submit this myself, hamaji has to do it... Thanks!
14 years ago (2010-05-14 19:13:54 UTC) #3
shinichiro.hamaji
14 years ago (2010-05-14 21:47:20 UTC) #4
On 2010/05/14 19:13:54, pphaneuf-google wrote:
> Ping? I actually can't submit this myself, hamaji has to do it...
> 
> Thanks!

Sorry for the big latency. I've somehow missed this.

Committed in: http://code.google.com/p/google-glog/source/detail?r=77
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b