Travis no longer green on master about libde265 HOT 8 CLOSED

I tried to fix the CMakefile to find this include file, but could not
see where to add the path.

/home/travis/build/strukturag/libde265/dist-cmake-test/libde265-1.0.2/enc265/enc265.cc:30:10:
fatal error:

  'image-io-png.h' file not found

#include "image-io-png.h"

     ^

1 error generated.

2015-10-20 14:12 GMT+02:00 Joachim Bauch [email protected]:

Travis is no longer green on master after commit 0338ac5
0338ac5
:
https://travis-ci.org/strukturag/libde265/builds/85284996

Also the fuzzing streams are failing after commit 8ec4e93
8ec4e93
:
https://travis-ci.org/strukturag/libde265/builds/85519474

Please check and make sure to keep Travis green/stable at least on the
master branch in the future.

—
Reply to this email directly or view it on GitHub
#130.

from libde265.

It is using the result of make dist which then fails to compile, maybe the header is new and not included in the distribution tarball?

from libde265.

That could be the reason.

However, I think it also failed when I tried to build it locally with cmake
without 'make dist'. Then I gave up.

2015-10-20 14:19 GMT+02:00 Joachim Bauch [email protected]:

It is using the result of make dist which then fails to compile, maybe
the header is new and not included in the distribution tarball?

—
Reply to this email directly or view it on GitHub
#130 (comment)
.

from libde265.

cmake without make dist is also tested, that works:
https://travis-ci.org/strukturag/libde265/jobs/85562561

from libde265.

Then it sounds like a simple fix.

2015-10-20 14:25 GMT+02:00 Joachim Bauch [email protected]:

cmake without make dist is also tested, that works:
https://travis-ci.org/strukturag/libde265/jobs/85562561

—
Reply to this email directly or view it on GitHub
#130 (comment)
.

from libde265.

The missing file most likely yes, but I don't know about the fuzzing streams. Maybe you added an assertion somewhere that now fires?

from libde265.

Most probably an unavailable reference to a SPS or PPS header which are now
managed through shared_ptr<>s. Not too concerned about it. Will check later.

2015-10-20 14:31 GMT+02:00 Joachim Bauch [email protected]:

The missing file most likely yes, but I don't know about the fuzzing
streams. Maybe you added an assertion somewhere that now fires?

—
Reply to this email directly or view it on GitHub
#130 (comment)
.

from libde265.

all green again

from libde265.