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

Drop loading of xparse lib in documentation lib #254

Closed
muzimuzhi opened this issue Nov 12, 2023 · 2 comments
Closed

Drop loading of xparse lib in documentation lib #254

muzimuzhi opened this issue Nov 12, 2023 · 2 comments
Assignees

Comments

@muzimuzhi
Copy link
Contributor

I think documentation lib only needs the xparse utilities already part of the LaTeX format (called the ltcmd module) and the main tcolorbox package (see #180), hence there's no need to load xparse lib (which will load xparse package) in it.

From c38ef64db2943da4847776fcdef2b661660b0988 Mon Sep 17 00:00:00 2001
From: Yukai Chou <muzimuzhi@gmail.com>
Date: Sun, 12 Nov 2023 18:42:35 +0800
Subject: [PATCH] Drop loading xparse lib in documentation lib

---
 tex/latex/tcolorbox/tcbdocumentation.code.tex | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tex/latex/tcolorbox/tcbdocumentation.code.tex b/tex/latex/tcolorbox/tcbdocumentation.code.tex
index ecdcf95..4a4c6eb 100644
--- a/tex/latex/tcolorbox/tcbdocumentation.code.tex
+++ b/tex/latex/tcolorbox/tcbdocumentation.code.tex
@@ -19,7 +19,7 @@
 %%
 \tcb@set@library@version{6.2.0pre2}
 
-\tcbuselibrary{listings,skins,xparse,external,raster}
+\tcbuselibrary{listings,skins,external,raster}
 
 \ifdefined\printindex\else%
   \RequirePackage{makeidx}[2000/03/29]%
-- 
2.42.0
@T-F-S
Copy link
Owner

T-F-S commented Nov 13, 2023

I think documentation lib only needs the xparse utilities already part of the LaTeX format (called the ltcmd module) and the main tcolorbox package (see #180), hence there's no need to load xparse lib (which will load xparse package) in it.

Yes, you are right. I will remove the loading of xparse.

@T-F-S T-F-S self-assigned this Nov 15, 2023
@T-F-S
Copy link
Owner

T-F-S commented Jan 10, 2024

@T-F-S T-F-S closed this as completed Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants