Differential D20221

AppArmor: don't highlight variable assignments and alias rules within profiles
ClosedPublic
Actions

Authored by nibags on Apr 3 2019, 6:15 AM.

Details

Reviewers
dhaumann
cullmann
Group Reviewers
Framework: Syntax Highlighting
Commits
R216:1726c4f868f6: AppArmor: don't highlight variable assignments and alias rules within profiles
Summary

Depends on D20220

There are some minor changes to the AppArmor highlighter.

Changes:

  • Don't highlight variable assignments and alias rules inside profiles.
  • Add new keywords of "tunables/share" variables.
  • Change style of "Other Option" attribute.

Diff Detail

Repository
R216 Syntax Highlighting
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
nibags requested review of this revision.Apr 3 2019, 6:15 AM
nibags created this revision.
nibags updated this revision to Diff 55367.Apr 3 2019, 8:42 PM
  • Remove "alias" in the "_end_rule" context and add some comments
Restricted Application added projects: Kate, Frameworks. · View Herald TranscriptApr 3 2019, 8:42 PM
Restricted Application added subscribers: kde-frameworks-devel, kwrite-devel. · View Herald Transcript
nibags updated this revision to Diff 55368.Apr 3 2019, 8:52 PM
  • Fix some indentations
  • Remove selinux-*.xml changes
nibags updated this revision to Diff 55378.Apr 3 2019, 9:29 PM
  • Apply changes in apparmor.xml, remove selinux-*.xml changes
nibags updated this revision to Diff 55382.Apr 3 2019, 9:55 PM
  • Update tests
nibags updated this revision to Diff 55457.Apr 5 2019, 8:19 AM
  • Don't highlight as Error the "alias" keyword within rules

Better not to highlight anything in rules.

cullmann accepted this revision.Apr 7 2019, 1:18 PM

See no obvious issues, please commit, thanks!

This revision is now accepted and ready to land.Apr 7 2019, 1:18 PM
Closed by commit R216:1726c4f868f6: AppArmor: don't highlight variable assignments and alias rules within profiles (authored by nibags, committed by cullmann). · Explain WhyApr 12 2019, 7:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 56092

View Options

autotests/folding/usr.bin.apparmor-profile-test.fold

Loading...
View Options

autotests/html/usr.bin.apparmor-profile-test.html

Loading...
View Options

autotests/input/usr.bin.apparmor-profile-test

Loading...
View Options

autotests/reference/usr.bin.apparmor-profile-test.ref

Loading...
View Options

data/syntax/apparmor.xml

Loading...