Travis: switch sof/doc sub-build to Ninja#215
Merged
lgirdwood merged 1 commit intothesofproject:masterfrom Apr 8, 2020
Merged
Conversation
marc-hb
force-pushed
the
ninja-doxygen
branch
3 times, most recently
from
5e49ba6 to
b7014cd
Compare
Since commit a97756a was merged, Travis shows the following harmless but misleading message: # To build doxygen APIs too run this first: # cmake -GNinja -S ../sof/doc -B ../sof/doc Example: https://travis-ci.org/github/thesofproject/sof-docs/builds/670800271 That's because the sof-docs documentation and sof-docs/Makefile now expect a sof/doc/ninja.build. Stop invoking make and let the main build invoke ninja In the future we'll probably want cmake -S sof/doc to use a separate -B build directory. One thing at a time. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
marc-hb
requested review from
deb-intel,
intelkevinputnam and
lgirdwood
as code owners
Collaborator
Author
|
See also thesofproject/sof#2741 |
dbaluta
approved these changes
Apr 8, 2020
lgirdwood
approved these changes
Apr 8, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Since commit a97756a was merged, Travis shows the following harmless but
misleading message:
Example: https://travis-ci.org/github/thesofproject/sof-docs/builds/670800271
That's because the sof-docs documentation and sof-docs/Makefile now
expect a sof/doc/ninja.build. So switch sof-docs/.travis.yml away from
Make and to ninja.
In the future we'll probably want cmake -S sof/doc to use a separate
-B build directory. One thing at a time.
Signed-off-by: Marc Herbert marc.herbert@intel.com