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

[cmake] pass compiler to & use ccache for FFmpeg #13548

Merged
merged 2 commits into from
Feb 14, 2018

Conversation

Rechi
Copy link
Member

@Rechi Rechi commented Feb 14, 2018

Motivation and Context

use ccache on FreeBSD jenkins builder for FFmpeg

How Has This Been Tested?

tested on linux without setting ENABLE_CCACHE and with setting ENABLE_CCACHE to ON & OFF
also tested with setting clang as compiler

Types of change

  • Bug fix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves existing functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the Code guidelines of this project
  • My change requires a change to the documentation, either Doxygen or wiki
  • I have updated the documentation accordingly
  • I have read the CONTRIBUTING document
  • I have added tests to cover my change
  • All new and existing tests passed

@Rechi Rechi added this to the L 18.0-alpha1 milestone Feb 14, 2018
@Rechi Rechi requested a review from wsnipex February 14, 2018 13:01
@Rechi
Copy link
Member Author

Rechi commented Feb 14, 2018

@wsnipex comment added

Copy link
Member

@wsnipex wsnipex left a comment

Choose a reason for hiding this comment

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

lgtm

@Rechi Rechi merged commit b16ba16 into xbmc:master Feb 14, 2018
@Rechi Rechi deleted the ffmpegCompiler branch February 14, 2018 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants