From e1d4af646bc887ebc5fdaee350394d0fefb25d99 Mon Sep 17 00:00:00 2001 From: Oliverpool Date: Wed, 28 Oct 2015 08:15:29 +0100 Subject: [PATCH 1/5] Compilation bug if # is present in nolyrics environment --- test/test_compilation/datadir.tex.control | 7 +++++++ test/test_compilation/datadir_datadir/songs/datadir.sgc | 2 ++ 2 files changed, 9 insertions(+) diff --git a/test/test_compilation/datadir.tex.control b/test/test_compilation/datadir.tex.control index a003cd42..9444d4de 100644 --- a/test/test_compilation/datadir.tex.control +++ b/test/test_compilation/datadir.tex.control @@ -114,6 +114,13 @@ Chordpro}[ \image{img/datadir.png} + +\ifchorded +\begin{verse*} + \musicnote {\nolyrics \[Am C#m]} +\end{verse*} +\fi + \endsong %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% diff --git a/test/test_compilation/datadir_datadir/songs/datadir.sgc b/test/test_compilation/datadir_datadir/songs/datadir.sgc index 73a6a15f..11bb1f54 100644 --- a/test/test_compilation/datadir_datadir/songs/datadir.sgc +++ b/test/test_compilation/datadir_datadir/songs/datadir.sgc @@ -5,3 +5,5 @@ {partition: datadir.ly} {image: datadir.png} + +[Am C#m] From a6a7edcfc873f40b3a8833c7c8a4cfaa857a96f1 Mon Sep 17 00:00:00 2001 From: Oliverpool Date: Wed, 28 Oct 2015 08:16:34 +0100 Subject: [PATCH 2/5] Replace # with {\shrp} in nolyrics environment --- patacrep/data/ast_templates/chordpro/latex/content_verse | 2 +- test/test_chordpro/nolyrics.tex | 2 +- test/test_compilation/datadir.tex.control | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/patacrep/data/ast_templates/chordpro/latex/content_verse b/patacrep/data/ast_templates/chordpro/latex/content_verse index ae3aa34f..72859732 100644 --- a/patacrep/data/ast_templates/chordpro/latex/content_verse +++ b/patacrep/data/ast_templates/chordpro/latex/content_verse @@ -6,7 +6,7 @@ \ifchorded \begin{verse*} (* for line in content.lines *) - \musicnote {\nolyrics (( render(line) ))} + \musicnote {\nolyrics (( render(line)|replace("#", "{\shrp}") ))} (* endfor *) \end{verse*} \fi diff --git a/test/test_chordpro/nolyrics.tex b/test/test_chordpro/nolyrics.tex index 343f4e63..3ce5d6b3 100644 --- a/test/test_chordpro/nolyrics.tex +++ b/test/test_chordpro/nolyrics.tex @@ -14,7 +14,7 @@ \ifchorded \begin{verse*} - \musicnote {\nolyrics \[A B#]} + \musicnote {\nolyrics \[A B{\shrp}]} \end{verse*} \fi diff --git a/test/test_compilation/datadir.tex.control b/test/test_compilation/datadir.tex.control index 9444d4de..0e37b1e7 100644 --- a/test/test_compilation/datadir.tex.control +++ b/test/test_compilation/datadir.tex.control @@ -117,7 +117,7 @@ Chordpro}[ \ifchorded \begin{verse*} - \musicnote {\nolyrics \[Am C#m]} + \musicnote {\nolyrics \[Am C{\shrp}m]} \end{verse*} \fi From 59fbf0e46b84c0d46c692020e128643191954d99 Mon Sep 17 00:00:00 2001 From: Oliverpool Date: Wed, 4 Nov 2015 08:42:39 +0100 Subject: [PATCH 3/5] Move syntax tests into a separate songbook --- test/test_compilation/datadir.tex.control | 7 -- .../datadir_datadir/songs/datadir.sgc | 2 - test/test_compilation/syntax.sb | 4 + test/test_compilation/syntax.tex.control | 116 ++++++++++++++++++ .../syntax_datadir/songs/musicnote.sgc | 4 + 5 files changed, 124 insertions(+), 9 deletions(-) create mode 100644 test/test_compilation/syntax.sb create mode 100644 test/test_compilation/syntax.tex.control create mode 100644 test/test_compilation/syntax_datadir/songs/musicnote.sgc diff --git a/test/test_compilation/datadir.tex.control b/test/test_compilation/datadir.tex.control index 0e37b1e7..a003cd42 100644 --- a/test/test_compilation/datadir.tex.control +++ b/test/test_compilation/datadir.tex.control @@ -114,13 +114,6 @@ Chordpro}[ \image{img/datadir.png} - -\ifchorded -\begin{verse*} - \musicnote {\nolyrics \[Am C{\shrp}m]} -\end{verse*} -\fi - \endsong %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% diff --git a/test/test_compilation/datadir_datadir/songs/datadir.sgc b/test/test_compilation/datadir_datadir/songs/datadir.sgc index 11bb1f54..73a6a15f 100644 --- a/test/test_compilation/datadir_datadir/songs/datadir.sgc +++ b/test/test_compilation/datadir_datadir/songs/datadir.sgc @@ -5,5 +5,3 @@ {partition: datadir.ly} {image: datadir.png} - -[Am C#m] diff --git a/test/test_compilation/syntax.sb b/test/test_compilation/syntax.sb new file mode 100644 index 00000000..fb16ce10 --- /dev/null +++ b/test/test_compilation/syntax.sb @@ -0,0 +1,4 @@ +{ +"datadir": ["syntax_datadir"], +"lang": "en" +} diff --git a/test/test_compilation/syntax.tex.control b/test/test_compilation/syntax.tex.control new file mode 100644 index 00000000..2794e2fa --- /dev/null +++ b/test/test_compilation/syntax.tex.control @@ -0,0 +1,116 @@ + + + + + + +%% Automatically generated document. +%% You may edit this file but all changes will be overwritten. +%% If you want to change this document, have a look at +%% the templating system. +%% +%% Generated using Songbook + +\makeatletter +\def\input@path{ % + {@TEST_FOLDER@/syntax_datadir/latex/} % + {@TEST_FOLDER@/latex/} % + {@DATA_FOLDER@/latex/} % +} +\makeatother + +\documentclass[ + ]{article} + +\usepackage[ + chorded, +diagram, +pictures, +guitar, + ]{patacrep} + +\usepackage{lmodern} + + +\PassOptionsToPackage{english}{babel} +\usepackage[english]{babel} +\lang{english} + +\usepackage{graphicx} +\graphicspath{ % + {@TEST_FOLDER@/syntax_datadir/} % + {@TEST_FOLDER@/} % + {@DATA_FOLDER@/} % +} + + +\makeatletter +\@ifpackageloaded{hyperref}{}{ + \usepackage{url} + \newcommand{\phantomsection}{} + \newcommand{\hyperlink}[2]{#2} + \newcommand{\href}[2]{\expandafter\url\expandafter{#1}} +} +\makeatother + + +\usepackage{chords} + +\title{Guitar songbook} +\author{The Patacrep Team} + +\newindex{titleidx}{syntax_title} +\newauthorindex{authidx}{syntax_auth} + + +\notenamesout{A}{B}{C}{D}{E}{F}{G} + + +\begin{document} + +\maketitle + + +\showindex{\songindexname}{titleidx} +\showindex{\authorindexname}{authidx} + +% list of chords +\ifchorded + \ifdiagram + \phantomsection + \addcontentsline{toc}{section}{\chordlistname} + \chords + \fi +\fi + +\phantomsection +\addcontentsline{toc}{section}{\songlistname} + +\begin{songs}{titleidx,authidx} +%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% +%% songs/./musicnote.sgc + +\selectlanguage{english} + +\beginsong{Song with Sharp in musicnote}[ + by={ + }, +] + + + + +\ifchorded +\begin{verse*} + \musicnote {\nolyrics \[Am C{\shrp}m]} +\end{verse*} +\fi + +\endsong + +\end{songs} + + + + +\end{document} \ No newline at end of file diff --git a/test/test_compilation/syntax_datadir/songs/musicnote.sgc b/test/test_compilation/syntax_datadir/songs/musicnote.sgc new file mode 100644 index 00000000..830f14d9 --- /dev/null +++ b/test/test_compilation/syntax_datadir/songs/musicnote.sgc @@ -0,0 +1,4 @@ +{title : Song with Sharp in musicnote} +{lang: en} + +[Am C#m] From 48840847e539a0b45f7cf7696b255bb9c1c7b861 Mon Sep 17 00:00:00 2001 From: Oliverpool Date: Wed, 4 Nov 2015 08:57:36 +0100 Subject: [PATCH 4/5] Dynamically skip tex.control if non-existent --- test/test_compilation/test_compilation.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/test_compilation/test_compilation.py b/test/test_compilation/test_compilation.py index 11067c55..9bf24cc2 100644 --- a/test/test_compilation/test_compilation.py +++ b/test/test_compilation/test_compilation.py @@ -38,9 +38,6 @@ class FileTest(unittest.TestCase, metaclass=dynamic.DynamicTest): '*.sb', ))): base = songbook[:-len(".sb")] - control = "{}.tex.control".format(base) - if not os.path.exists(control): - continue yield ( "test_generation_{}".format(os.path.basename(base)), cls._create_generation_test(base), @@ -63,6 +60,9 @@ class FileTest(unittest.TestCase, metaclass=dynamic.DynamicTest): # Check generated tex control = "{}.tex.control".format(base) + if not os.path.exists(control): + raise unittest.SkipTest('No control file for {}'.format(songbook)) + tex = "{}.tex".format(base) with open_read(control) as expectfile: with open_read(tex) as latexfile: From 1d6f8c2bbdbeccfa3d9a2c7f8b4ff127af06c6f4 Mon Sep 17 00:00:00 2001 From: Oliverpool Date: Wed, 4 Nov 2015 08:59:47 +0100 Subject: [PATCH 5/5] Rename compilation test to force better ordering --- test/test_compilation/test_compilation.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/test_compilation/test_compilation.py b/test/test_compilation/test_compilation.py index 9bf24cc2..3af70d19 100644 --- a/test/test_compilation/test_compilation.py +++ b/test/test_compilation/test_compilation.py @@ -39,11 +39,11 @@ class FileTest(unittest.TestCase, metaclass=dynamic.DynamicTest): ))): base = songbook[:-len(".sb")] yield ( - "test_generation_{}".format(os.path.basename(base)), + "test_latex_generation_{}".format(os.path.basename(base)), cls._create_generation_test(base), ) yield ( - "test_compilation_{}".format(os.path.basename(base)), + "test_pdf_compilation_{}".format(os.path.basename(base)), cls._create_compilation_test(base), )