From 9cd6b444f848e2a2f8ed4ca16cebb96390ade179 Mon Sep 17 00:00:00 2001
From: Hans-Christoph Steiner <hans@eds.org>
Date: Mon, 21 Mar 2016 21:51:23 +0100
Subject: [PATCH] standardize on .yml as the file extension for YAML

Though the YAML people recommend .yaml for the file extension, in Android
land it seems clear that .yml has won out:

* .travis.yml
* .gitlab-ci.yml
* .circle.yml
* Ansible main.yml
---
 fdroidserver/common.py                                    | 2 +-
 fdroidserver/import.py                                    | 2 +-
 fdroidserver/metadata.py                                  | 8 ++++----
 fdroidserver/rewritemeta.py                               | 2 +-
 tests/metadata.TestCase                                   | 2 +-
 tests/metadata/org.videolan.vlc.pickle                    | 2 +-
 .../{org.videolan.vlc.yaml => org.videolan.vlc.yml}       | 0
 tests/run-tests                                           | 2 +-
 8 files changed, 10 insertions(+), 10 deletions(-)
 rename tests/metadata/{org.videolan.vlc.yaml => org.videolan.vlc.yml} (100%)

diff --git a/fdroidserver/common.py b/fdroidserver/common.py
index 72af49d8..acaeaffd 100644
--- a/fdroidserver/common.py
+++ b/fdroidserver/common.py
@@ -63,7 +63,7 @@ default_config = {
     'ant': "ant",
     'mvn3': "mvn",
     'gradle': 'gradle',
-    'accepted_formats': ['txt', 'yaml'],
+    'accepted_formats': ['txt', 'yml'],
     'sync_from_local_copy_dir': False,
     'per_app_repos': False,
     'make_current_version_link': True,
diff --git a/fdroidserver/import.py b/fdroidserver/import.py
index 4d342f04..8c7f5107 100644
--- a/fdroidserver/import.py
+++ b/fdroidserver/import.py
@@ -261,7 +261,7 @@ def main():
     app.builds.append(build)
 
     if write_local_file:
-        metadata.write_metadata('.fdroid.yaml', app)
+        metadata.write_metadata('.fdroid.yml', app)
     else:
         # Keep the repo directory to save bandwidth...
         if not os.path.exists('build'):
diff --git a/fdroidserver/metadata.py b/fdroidserver/metadata.py
index 722e1710..71da8453 100644
--- a/fdroidserver/metadata.py
+++ b/fdroidserver/metadata.py
@@ -779,10 +779,10 @@ def read_metadata(xref=True):
     for metadatapath in sorted(glob.glob(os.path.join('metadata', '*.txt'))
                                + glob.glob(os.path.join('metadata', '*.json'))
                                + glob.glob(os.path.join('metadata', '*.xml'))
-                               + glob.glob(os.path.join('metadata', '*.yaml'))
+                               + glob.glob(os.path.join('metadata', '*.yml'))
                                + glob.glob('.fdroid.json')
                                + glob.glob('.fdroid.xml')
-                               + glob.glob('.fdroid.yaml')):
+                               + glob.glob('.fdroid.yml')):
         app = parse_metadata(metadatapath)
         if app.id in apps:
             raise MetaDataException("Found multiple metadata files for " + app.id)
@@ -950,7 +950,7 @@ def parse_metadata(metadatapath):
             parse_json_metadata(mf, app)
         elif ext == 'xml':
             parse_xml_metadata(mf, app)
-        elif ext == 'yaml':
+        elif ext == 'yml':
             parse_yaml_metadata(mf, app)
         else:
             raise MetaDataException('Unknown metadata format: %s' % metadatapath)
@@ -1387,6 +1387,6 @@ def write_metadata(metadatapath, app):
     with open(metadatapath, 'w') as mf:
         if ext == 'txt':
             return write_txt(mf, app)
-        elif ext == 'yaml':
+        elif ext == 'yml':
             return write_yaml(mf, app)
     raise MetaDataException('Unknown metadata format: %s' % metadatapath)
diff --git a/fdroidserver/rewritemeta.py b/fdroidserver/rewritemeta.py
index 75888a23..8335ed9c 100644
--- a/fdroidserver/rewritemeta.py
+++ b/fdroidserver/rewritemeta.py
@@ -64,7 +64,7 @@ def main():
     if options.list and options.to is not None:
         parser.error("Cannot use --list and --to at the same time")
 
-    supported = ['txt', 'yaml']
+    supported = ['txt', 'yml']
 
     if options.to is not None and options.to not in supported:
         parser.error("Must give a valid format to --to")
diff --git a/tests/metadata.TestCase b/tests/metadata.TestCase
index 9dfe2bbc..c287a212 100755
--- a/tests/metadata.TestCase
+++ b/tests/metadata.TestCase
@@ -33,7 +33,7 @@ class MetadataTest(unittest.TestCase):
         config = dict()
         config['sdk_path'] = '/opt/android-sdk'
         config['ndk_paths'] = dict()
-        config['accepted_formats'] = ['json', 'txt', 'xml', 'yaml']
+        config['accepted_formats'] = ['json', 'txt', 'xml', 'yml']
         fdroidserver.common.config = config
 
         apps = fdroidserver.metadata.read_metadata(xref=True)
diff --git a/tests/metadata/org.videolan.vlc.pickle b/tests/metadata/org.videolan.vlc.pickle
index 45af8916..a1810e60 100644
--- a/tests/metadata/org.videolan.vlc.pickle
+++ b/tests/metadata/org.videolan.vlc.pickle
@@ -4618,7 +4618,7 @@ sasS'FlattrID'
 p1324
 NsS'metadatapath'
 p1325
-S'metadata/org.videolan.vlc.yaml'
+S'metadata/org.videolan.vlc.yml'
 p1326
 sS'Disabled'
 p1327
diff --git a/tests/metadata/org.videolan.vlc.yaml b/tests/metadata/org.videolan.vlc.yml
similarity index 100%
rename from tests/metadata/org.videolan.vlc.yaml
rename to tests/metadata/org.videolan.vlc.yml
diff --git a/tests/run-tests b/tests/run-tests
index 8ec03c35..62d27aa7 100755
--- a/tests/run-tests
+++ b/tests/run-tests
@@ -163,7 +163,7 @@ cp $WORKSPACE/tests/metadata/org.smssecure.smssecure.txt $REPOROOT/metadata/
 $fdroid readmeta
 
 # now make a fake duplicate
-touch $REPOROOT/metadata/org.smssecure.smssecure.yaml
+touch $REPOROOT/metadata/org.smssecure.smssecure.yml
 
 set +e
 $fdroid readmeta
-- 
2.30.2