Skip to content

substitutions.txt: add missing |JSL|#211

Merged
lgirdwood merged 1 commit intothesofproject:masterfrom
marc-hb:missing-JSL
Apr 4, 2020
Merged

substitutions.txt: add missing |JSL|#211
lgirdwood merged 1 commit intothesofproject:masterfrom
marc-hb:missing-JSL

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Apr 3, 2020

Fixes commit f4ada42 ("getting_started: Add build documenation for
ICL and JSL+") that added four "WARNING: Undefined substitution
referenced: JSL" and for literal "|JSL|" instances on
https://thesofproject.github.io/latest/getting_started/build-guide/build-from-scratch.html,
example:

Current target Intel platforms include: Bay Trail, Cherry Trail,
Haswell, Broadwell, Apollo Lake, Cannon Lake, Ice Lake and |JSL|.

Signed-off-by: Marc Herbert marc.herbert@intel.com

Fixes commit f4ada42 ("getting_started: Add build documenation for
ICL and JSL+") that added four "WARNING: Undefined substitution
referenced: JSL" and for literal "|JSL|" instances on
https://thesofproject.github.io/latest/getting_started/build-guide/build-from-scratch.html,
example:

  Current target Intel platforms include: Bay Trail, Cherry Trail,
  Haswell, Broadwell, Apollo Lake, Cannon Lake, Ice Lake and |JSL|.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb
Copy link
Collaborator Author

marc-hb commented Apr 3, 2020

The other warnings are unrelated.

Does github have the concept of a warning? It would be much better to make continuous-integration/travis-ci/pr yellow and probably avoid late fixes like this one. The check is not just green but collapsed! Need to click on "show all checks" so hard to blame anyone who doesn't pay attention (at least authors should see most of these warnings locally too)

@marc-hb
Copy link
Collaborator Author

marc-hb commented Apr 4, 2020

Does github have the concept of a warning?

Not sure about github, but Travis apparently not. Travis is either green or red. I'm trying some experiment anyway in PR #214 if you're interested.

@lgirdwood lgirdwood merged commit 9b2c84c into thesofproject:master Apr 4, 2020
@marc-hb marc-hb deleted the missing-JSL branch April 8, 2020 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants