Skip to content

Conversation

@devjgm
Copy link
Contributor

@devjgm devjgm commented Apr 14, 2021

Fixes: #6247

ldconfig doesn't always add stuff in /usr/local/lib directories, so
in Dockerfiles when we install a bunch of stuff that might land in
/usr/local, we should be sure to run ldconfig on /usr/local before we
attempt to use those things (or expect pkg-config to find them).


This change is Reviewable

Fixes: googleapis#6247

`ldconfig` doesn't always add stuff in `/usr/local/lib` directories, so
in Dockerfiles when we install a bunch of stuff that might land in
`/usr/local`, we should be sure to run ldconfig on /usr/local before we
attempt to use those things (or expect pkg-config to find them).
@devjgm devjgm requested a review from a team as a code owner April 14, 2021 00:46
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Apr 14, 2021
@codecov
Copy link

codecov bot commented Apr 14, 2021

Codecov Report

Merging #6248 (fd92e03) into master (be5767c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6248   +/-   ##
=======================================
  Coverage   95.67%   95.67%           
=======================================
  Files        1177     1177           
  Lines      105228   105228           
=======================================
  Hits       100682   100682           
  Misses       4546     4546           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be5767c...fd92e03. Read the comment docs.

@devjgm devjgm merged commit 54725bf into googleapis:master Apr 14, 2021
@devjgm devjgm deleted the gcb-fix-demo-ldconfig branch April 14, 2021 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[GCB] demo-fedora-{pr,ci} builds are failing when triggered, but pass locally

2 participants