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

extend platform expressions to support media query syntax - terminals… #21444

Merged
merged 4 commits into from
Nov 17, 2021

Conversation

markle11m
Copy link
Contributor

@markle11m markle11m commented Nov 15, 2021

Update documentation to support media queries syntax in platform expressions.

In media queries syntax (Media Queries):

  • "not" represents logical-not
  • "and" represents logical-and
  • "," represents a logical-or, but with lower precedence than logical-and
  • media queries syntax expresses precedence by treating "," as a list-separator

This change extends platform expressions by using the same list-separator model for "," and by adding the other terminals ("and", "not") as synonyms for the corresponding operators that are already supported ("&", "!"). This change also adds "or" as a synonym for "|" as a potential future extension.

Open question: does the EBNF need to disambiguate "and", "not", "or" as keywords and not identifiers?

… 'and', 'not' and ',' for logical-and, -not and -or operators
@markle11m markle11m marked this pull request as draft November 15, 2021 23:12
@JackBoosY JackBoosY added category:documentation To resolve the issue, documentation will need to be updated requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look. labels Nov 16, 2021
@JackBoosY
Copy link
Contributor

I'm not sure that, needs other guys to review this.

@markle11m markle11m marked this pull request as ready for review November 17, 2021 23:33
@ras0219-msft ras0219-msft merged commit cbc22a3 into microsoft:master Nov 17, 2021
@ras0219-msft
Copy link
Contributor

LGTM, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:documentation To resolve the issue, documentation will need to be updated requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants