Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

travis: build both debug and release to avoid debug build fail #874

Merged
merged 1 commit into from
Jan 10, 2019

Conversation

xuguangxin
Copy link
Contributor

No description provided.

.travis.yml Outdated
- ./autogen.sh --prefix=/opt/yami --enable-h264enc --enable-jpegenc --enable-vp8enc --enable-vp9enc --enable-h265enc --enable-mpeg2dec --enable-vp8dec --enable-vp9dec --enable-jpegdec --enable-h264dec --enable-h265dec --enable-vc1dec --enable-v4l2
- make -j8 ; sudo make install
#make sure we did not break debug build
- ./autogen.sh --prefix=/opt/yami --enable-debug --enable-h264enc --enable-jpegenc --enable-vp8enc --enable-vp9enc --enable-h265enc --enable-mpeg2dec --enable-vp8dec --enable-vp9dec --enable-jpegdec --enable-h264dec --enable-h265dec --enable-vc1dec --enable-v4l2 && make -j8
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add --enable-tests (maybe in both cases), too, since I've seen this break several times before.

@xuguangxin xuguangxin force-pushed the travis_debug branch 5 times, most recently from 4ae8716 to ac3d6d6 Compare January 10, 2019 07:13
@xuguangxin
Copy link
Contributor Author

good suggestion. done.

Copy link
Contributor

@uartie uartie left a comment

Choose a reason for hiding this comment

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

The use of COMPILE_OPTS variable is a good idea. Although, I'm not sure where this setting is configured. Nonetheless, seems travis ci is working with this.

LGTM

@uartie uartie merged commit 01d5f7b into intel:apache Jan 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants