Overview
Request 1205188 accepted
- Created by Guillaume_G
- In state accepted
Request History
Guillaume_G created request
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
anag+factory added as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:48"
anag+factory accepted review
Picked "openSUSE:Factory:Staging:adi:48"
licensedigger accepted review
The legal review is accepted preliminary. The package may require actions later on.
darix accepted review
Accepted review for by_group opensuse-review-team request 1205188 from user anag+factory
anag+factory accepted review
Staging Project openSUSE:Factory:Staging:adi:48 got accepted.
anag+factory approved review
Staging Project openSUSE:Factory:Staging:adi:48 got accepted.
anag+factory accepted request
Staging Project openSUSE:Factory:Staging:adi:48 got accepted.
Any clue why the dep auto-detection would not work here?
Because the .pc file has no
Requires: libavdevice libavfilter libavformat libavcodec libswresample libswscale libavutil
line in it. The generation of dlib.pc from CMakeLists.txt is really subpar-ly implemented.so why not patch the pkg-config file generation then? that would be a fix we can send upstream even.
also why does building against dlib requires all those other devel packages?
(I'm just a reviewing participant, not the maintainer) dlib headers do
#include <libavcodec/libavcodec.h>
,<gif_lib.h>
etc. and so have a direct dependency on other devel packages.