jagtesh / googlemock

Automatically exported from code.google.com/p/googlemock
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

gmock-config --ldflags missing space typo #49

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Install google mock
2. run `gmock-config --ldflags`

What is the expected output? What do you see instead?
Should say something like -L/usr/local/lib -L/usr/local/lib, in fact it says: 
"-L/usr/local/lib-L/usr/local/lib"

Note the lack of a space between the two -L values.

This means that using pkg-config macros in the configure step of a build
process would fail. My workaround is to use `gtest-config --ldflags` and
then check for the existence of gmock-config without actually using its
output. I assume that both libraries are installed to the same place.

It looks like a straightforward fix. On line 262 in scripts/gmock-config.in
when gmock_ldflags is set there is a space missing.

What version of the product are you using? On what operating system?
Version 1.1.0 on Linux, with gtest 1.3.0.

Original issue reported on code.google.com by richard.quirk on 2 Jun 2009 at 8:08

GoogleCodeExporter commented 9 years ago

Original comment by zhanyong...@gmail.com on 5 Jun 2009 at 12:18

GoogleCodeExporter commented 9 years ago
Thanks for the report!  Fixed in the trunk r166.

Original comment by zhanyong...@gmail.com on 5 Jun 2009 at 6:42