Fix dependency with catkin 0.7.2

[jspricke]

since ros/catkin#813 catkin checks that the
dependency was actually found.

[wjwwood]

It's still not passing.

@dirk-thomas any input on what to change here? I remember some issue comment traffic about things found with pkgconfig. Should we just remove it from the list and pass the include dirs and libraries separately?

[jspricke]

I guess libpcre3-dev is missing in the build depends, at least it's not in the jankins logs.

[dirk-thomas]

Part of the referenced change has been reverted in ros/catkin#825. But the CMake warning indicates that for either of the two exported DEPENDS pcrecpp / libpcrecpp there are no CMake variables defined for include directories and libraries (which makes the export a no-op). I don't know what variables you expect to be defined but currently the DEPENDS (with as well as without this patch) doesn't do what you expect it to do.

[dirk-thomas]

If this dependency you might want to find it with REQUIRED to ensure the configure steps fails if the module was not found. And then it could be that a necessary build dependency is not correctly declared as @jspricke pointed out.

[wjwwood]

@jspricke will you allow me to push to your pr branch? See: https://github.com/blog/2247-improving-collaboration-with-forks

[jspricke]

@wjwwood I found an easier fix, what do you think?

[wjwwood]

Yeah, even better. Looks like it was missed when removing pcre stuff from in this pr: #113

@k-okada fyi