Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add missing dependency on pkg-config #19

Merged
merged 1 commit into from
Feb 8, 2018
Merged

add missing dependency on pkg-config #19

merged 1 commit into from
Feb 8, 2018

Conversation

dirk-thomas
Copy link
Member

After being merged this patch needs to be cherry-picked into the ardent branch and a new patch release has to be made.

@dirk-thomas dirk-thomas added the bug label Feb 8, 2018
@dirk-thomas dirk-thomas self-assigned this Feb 8, 2018
Copy link
Member

@wjwwood wjwwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, but please always describe why it is needed, explained by it or linking to failing jobs or something. Just declaring it needs to be done and leaving the rest up to the reviewer to figure out is not acceptable pr etiquette in my opinion.

@wjwwood wjwwood merged commit fd11fad into ros2 Feb 8, 2018
@wjwwood wjwwood deleted the pkg-config-dep branch February 8, 2018 16:57
@dirk-thomas
Copy link
Member Author

Sure, the packages CMake extra file require pkg-config to be available (

message(FATAL_ERROR "Could not find pkg-config")
) but the manifest doesn't list it as a dependency.

wjwwood pushed a commit that referenced this pull request Feb 8, 2018
wjwwood added a commit that referenced this pull request Feb 8, 2018
@nuclearsandwich
Copy link
Member

resource_retriever builds on top of this were failing. 🔴 Abin_uX64__resource_retriever.. #9 is an example. A snippet of the build log is below.

23:01:31 cmake: /opt/ros/ardent/opt/libcurl_vendor/lib/libcurl.so.4: no version information available (required by cmake)
23:01:31 -- The C compiler identification is GNU 5.4.0
23:01:31 -- The CXX compiler identification is GNU 5.4.0
23:01:31 -- Check for working C compiler: /usr/lib/ccache/x86_64-linux-gnu-gcc
23:01:31 -- Check for working C compiler: /usr/lib/ccache/x86_64-linux-gnu-gcc -- works
23:01:31 -- Detecting C compiler ABI info
23:01:31 -- Detecting C compiler ABI info - done
23:01:31 -- Detecting C compile features
23:01:31 -- Detecting C compile features - done
23:01:31 -- Check for working CXX compiler: /usr/lib/ccache/x86_64-linux-gnu-g++
23:01:31 -- Check for working CXX compiler: /usr/lib/ccache/x86_64-linux-gnu-g++ -- works
23:01:31 -- Detecting CXX compiler ABI info
23:01:31 -- Detecting CXX compiler ABI info - done
23:01:31 -- Detecting CXX compile features
23:01:31 -- Detecting CXX compile features - done
23:01:31 -- Found ament_cmake_ros: 0.4.0 (/opt/ros/ardent/share/ament_cmake_ros/cmake)
23:01:31 -- Found PythonInterp: /usr/bin/python3 (found suitable version "3.5.2", minimum required is "3") 
23:01:31 -- Using PYTHON_EXECUTABLE: /usr/bin/python3
23:01:31 -- Found ament_index_cpp: 0.4.0 (/opt/ros/ardent/share/ament_index_cpp/cmake)
23:01:31 -- Found libcurl_vendor: 2.0.1 (/opt/ros/ardent/share/libcurl_vendor/cmake)
23:01:31 -- Could NOT find PkgConfig (missing:  PKG_CONFIG_EXECUTABLE) 
23:01:31 CMake Error at /opt/ros/ardent/share/libcurl_vendor/cmake/libcurl_vendor-extras.cmake:18 (message):
23:01:31   Could not find pkg-config
23:01:31 Call Stack (most recent call first):
23:01:31   /opt/ros/ardent/share/libcurl_vendor/cmake/libcurl_vendorConfig.cmake:30 (include)
23:01:31   CMakeLists.txt:16 (find_package)

nuclearsandwich added a commit to ros2/ros2 that referenced this pull request Feb 9, 2018
nuclearsandwich added a commit to ros2/ros2 that referenced this pull request Feb 9, 2018
nuclearsandwich pushed a commit to ros/rosdistro that referenced this pull request Apr 19, 2019
…20969)

Increasing version of package(s) in repository `resource_retriever` to `2.1.0-1`:

- upstream repository: https://github.com/ros/resource_retriever.git
- release repository: https://github.com/ros2-gbp/resource_retriever-release.git
- distro file: `dashing/distribution.yaml`
- bloom version: `0.8.0`
- previous version for package: `null`

## libcurl_vendor

```
* depend on curl (mapping to curl, libcurl4-openssl-dev) for packaging (#25 <ros/resource_retriever#25>)
* add missing dependency on pkg-config (#19 <ros/resource_retriever#19>)
* [libcurl_vendor] convert to ament and setup env hooks for library paths (#14 <ros/resource_retriever#14>)
* Contributors: Dirk Thomas, Mikael Arguedas, William Woodall
```

## resource_retriever

```
* Make sure to export the include directory for resource_retriever. (#22 <ros/resource_retriever#22>)
* Contributors: Chris Lalancette
```
andre-rosa pushed a commit to andre-rosa/rosdistro that referenced this pull request May 3, 2019
…os#20969)

Increasing version of package(s) in repository `resource_retriever` to `2.1.0-1`:

- upstream repository: https://github.com/ros/resource_retriever.git
- release repository: https://github.com/ros2-gbp/resource_retriever-release.git
- distro file: `dashing/distribution.yaml`
- bloom version: `0.8.0`
- previous version for package: `null`

## libcurl_vendor

```
* depend on curl (mapping to curl, libcurl4-openssl-dev) for packaging (ros#25 <ros/resource_retriever#25>)
* add missing dependency on pkg-config (ros#19 <ros/resource_retriever#19>)
* [libcurl_vendor] convert to ament and setup env hooks for library paths (ros#14 <ros/resource_retriever#14>)
* Contributors: Dirk Thomas, Mikael Arguedas, William Woodall
```

## resource_retriever

```
* Make sure to export the include directory for resource_retriever. (ros#22 <ros/resource_retriever#22>)
* Contributors: Chris Lalancette
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants