Hi,

when grepping for things which look like version numbers I noticed some
version 2.12 specific code in scripts/build/create-weblinks-itexi.py.

The attached patch removes code which won't be executed in lilypond
versions other than 2.12.

Felix
diff --git a/scripts/build/create-weblinks-itexi.py b/scripts/build/create-weblinks-itexi.py
index 7e0c655..95342d0 100644
--- a/scripts/build/create-weblinks-itexi.py
+++ b/scripts/build/create-weblinks-itexi.py
@@ -66,7 +66,6 @@ translations = {
         'PDF of MusicXML regtests for ': 'PDF der MusicXML Regressionstests für ',
 
         'Doc tarball for ': 'Dokumentation tar-gepackt für ',
-        ' (did not exist in 2.12)': ' (nicht existent in 2.12)',
         },
     'es': {
         'Source': 'Código fuente',
@@ -97,7 +96,6 @@ translations = {
         'PDF of MusicXML regtests for ': 'Pruebas de MusicXML en PDF para ',
 
         'Doc tarball for ': 'Tarball de la documentación para ',
-        ' (did not exist in 2.12)': ' (no existía en la versión 2.12)',
      },
     'fr': {
         'Source': 'Sources',
@@ -128,7 +126,6 @@ translations = {
         'PDF of MusicXML regtests for ': 'PDF des tests de régression de MusicXML pour ',
 
         'Doc tarball for ': 'Archive de la documentation pour ',
-        ' (did not exist in 2.12)': ' (non disponible pour la version 2.12)',
         },
     'hu': {
         'Source': 'Forrás',
@@ -157,7 +154,6 @@ translations = {
         'PDF of MusicXML regtests for ': 'PDF formátumban - verzió: ',
 
         'Doc tarball for ': 'Tömörített csomag - verzió: ',
-        ' (did not exist in 2.12)': ' (nem létezett a 2.12-es verzióban)',
         },
     'it': {
         'Source': 'Sorgenti',
@@ -187,7 +183,6 @@ translations = {
         'PDF of MusicXML regtests for ': 'PDF dei test di collaudo di MusicXML per ',
 
         'Doc tarball for ': 'Archivio della documentazione per ',
-        ' (did not exist in 2.12)': ' (non disponibile per la versione 2.12)',
         },
     'ja': {
         'Source': 'ソース',
@@ -219,7 +214,6 @@ translations = {
         'PDF of MusicXML regtests for ': 'MusicXML 回帰テスト (PDF 版) バージョン ',
 
         'Doc tarball for ': 'ドキュメント アーカイブ バージョン ',
-        ' (did not exist in 2.12)': ' (バージョン 2.12 には存在しません)',
 
         },
     'nl': {
@@ -251,7 +245,6 @@ translations = {
         'PDF of MusicXML regtests for ': 'MusicXML regressietesten voor ',
 
         'Doc tarball for ': 'Tarball met documentation voor ',
-        ' (did not exist in 2.12)': ' (bestond nog niet in 2.12)',
      },
     'zh': {
         'Source': '源码',
@@ -282,7 +275,6 @@ translations = {
         'PDF of MusicXML regtests for ': 'MusicXML 的 PDF 回归测试 ',
 
         'Doc tarball for ': '为文档压缩包',
-        ' (did not exist in 2.12)': ' (在 2.12 中不存在)',
         },
     }
 
@@ -415,30 +407,7 @@ def translateNameToUrl(manual, version):
     ver_split = version.split('.')
     ver_minor = ver_split[0] + '.' + ver_split[1]
     url = depth + "doc/v" + ver_minor + "/Documentation/"
-
-    if (ver_minor == '2.12'):
-        if (manual=='learning'):
-            return url+'user/lilypond-learning'
-        elif (manual=='internals'):
-            return url+'user/lilypond-internals'
-        elif (manual=='notation'):
-            return url+'user/lilypond'
-        elif (manual=='usage'):
-            return url+'user/lilypond-program'
-        elif (manual=='snippets'):
-            return url+'../input/lsr/lilypond-snippets'
-        elif (manual=='changes'):
-            return url+'topdocs/NEWS.html'
-        elif (manual=='music-glossary'):
-            return url+'user/music-glossary'
-        elif (manual=='essay'):
-            return url+'user/lilypond-learning/Background.html'
-        elif (manual=='extending'):
-            return url+'user/lilypond/Interfaces-for-programmers.html'
-        else:
-            return ''
-    else:
-        return url+manual
+    return url+manual
 
 def addLang(url, lang):
     if lang:
@@ -464,52 +433,19 @@ def make_manual_links(name, version, lang):
             manual = 'Music glossary'
         url = translateNameToUrl(m, version)
 
-        if (url == ''):
-            # can't have a comma here due to texinfo
-            make_ver_link(macroLang("manual"+name+mshort+'Pdf',lang),
-                "http://lilypond.org";,
-                mshort+getTrans(" (did not exist in 2.12)",lang))
-            make_ver_link(macroLang("manual"+name+mshort+'Split',lang),
-                "http://lilypond.org";,
-                mshort+getTrans(" (did not exist in 2.12)",lang))
-            make_ver_link(macroLang("manual"+name+mshort+'Big',lang),
-                "http://lilypond.org";,
-                mshort+getTrans(" (did not exist in 2.12)",lang))
-            make_ver_link(macroLang("manual"+name+mshort+'SplitNoName',lang),
-                "http://lilypond.org";,
-                mshort+getTrans(" (did not exist in 2.12)",lang))
-            continue
-        # this is stupid and I shouldn't have bothered trying
-        # to support the 2.12 docs and it will be deleted once
-        # 2.14 is out and the website won't be visible to users
-        # until 2.14 is out.  -gp
-        if (url.endswith('.html')):
-            make_ver_link(macroLang("manual"+name+mshort+'Pdf',lang),
-                      addLang(url, lang),
-                      getTrans(manual.capitalize(),lang) + '.pdf')
-            make_ver_link(macroLang("manual"+name+mshort+'Split',lang),
-                      addLang(url, lang),
-                      getTrans(manual.capitalize(),lang) +
-                      getTrans(' (split HTML)',lang))
-            make_ver_link(macroLang("manual"+name+mshort+'Big',lang),
-                      addLang(url, lang),
-                      getTrans(manual.capitalize(),lang) +
-                      getTrans(' (big HTML)',lang))
-            newurl = url
-        else:
-            make_ver_link(macroLang("manual"+name+mshort+'Pdf',lang),
-                      # TODO: this is an even stupider way of doing it
-                      addLang(url+'.pdf', lang),
-                      getTrans(manual.capitalize(),lang) + '.pdf')
-            make_ver_link(macroLang("manual"+name+mshort+'Split',lang),
-                      addLang(url + '/index.html', lang),
-                      getTrans(manual.capitalize(),lang) +
-                      getTrans(' (split HTML)',lang))
-            make_ver_link(macroLang("manual"+name+mshort+'Big',lang),
-                      addLang(url + '-big-page.html', lang),
-                      getTrans(manual.capitalize(),lang) +
-                      getTrans(' (big HTML)',lang))
-            newurl = url + '/index.html'
+        make_ver_link(macroLang("manual"+name+mshort+'Pdf',lang),
+                  # TODO: this is an even stupider way of doing it
+                  addLang(url+'.pdf', lang),
+                  getTrans(manual.capitalize(),lang) + '.pdf')
+        make_ver_link(macroLang("manual"+name+mshort+'Split',lang),
+                  addLang(url + '/index.html', lang),
+                  getTrans(manual.capitalize(),lang) +
+                  getTrans(' (split HTML)',lang))
+        make_ver_link(macroLang("manual"+name+mshort+'Big',lang),
+                  addLang(url + '-big-page.html', lang),
+                  getTrans(manual.capitalize(),lang) +
+                  getTrans(' (big HTML)',lang))
+        newurl = url + '/index.html'
         make_ver_link(macroLang("manual"+name+mshort+'SplitNoName',lang),
                   addLang(newurl, lang),
                   getTrans(manual.capitalize(),lang))
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to