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

Add Beamer Theme Support for LaTeX Scanner #4621

Merged
merged 3 commits into from
Oct 24, 2024

Conversation

kprussing
Copy link
Contributor

This adds the support for recursively scanning beamer themes as discussed in #4620. It updates the LaTeX scanner to look for \use(|color|font|inner|outer)theme{file} and generate the correct file name to interrogate. Similar to the \usepacakge, it also avoids superfluous errors if the user may be relying on a theme that is installed at the system level.

Contributor Checklist:

  • I have created a new test or updated the unit tests to cover the new/changed functionality.
  • I have updated CHANGES.txt and RELEASE.txt (and read the README.rst).
  • I have updated the appropriate documentation

The branch adds support for tracking Beamer theme files.  This defines
the test to show the scanning failed without modification, but succeeds
with the modification.
This adds a regular expression to check for beamer theme files for LaTeX
that may be part of the user's build tree.  Similar to package files
found via `\usepackage`, this will not report a missing file because the
user may just be relying on the system version.
@mwichmann mwichmann added the LaTex LaTex related issues. label Oct 21, 2024
CHANGES.txt Outdated
@@ -66,6 +66,9 @@ RELEASE VERSION/DATE TO BE FILLED IN LATER
/etc/paths.d. This may occur if SCons is invoked in a sandboxed
environment (such as Nix).

From Keith F Prussing:
Copy link
Contributor

Choose a reason for hiding this comment

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

Please adjust. The listings should be alphabetical by contributors last name.

Copy link
Contributor

Choose a reason for hiding this comment

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

NM. I fixed.

@bdbaddog bdbaddog merged commit 3b6e4d1 into SCons:master Oct 24, 2024
@mwichmann mwichmann added this to the NextRelease milestone Oct 25, 2024
@mwichmann mwichmann linked an issue Oct 27, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LaTex LaTex related issues.
Projects
Status: Complete
Development

Successfully merging this pull request may close these issues.

Support Tracking Custom Beamer Themes in LaTeX
3 participants