Merge pull request #34 from YunoHost/fix-license-test

Fix license test + improve message
This commit is contained in:
Laurent Peuch 2018-01-14 16:55:28 +01:00 committed by GitHub
commit 589e6ac516
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -180,11 +180,10 @@ def check_manifest(path):
print_warning("[YEP-1.3] The use of '%s' in license field implies to " print_warning("[YEP-1.3] The use of '%s' in license field implies to "
"write something about the license in your " "write something about the license in your "
"README.md" % (license)) "README.md" % (license))
if license == "non-free" or "dep-non-free": if license in ["non-free", "dep-non-free"]:
print_warning("[YEP-1.3] 'non-free' apps can't be officialised." print_warning("[YEP-1.3] 'non-free' apps can't be officialized."
"The way to integrate them or not is always " "Their integration is still being discussed,"
"in discussion notably about apps with non " "especially for apps with non-free dependencies")
"free dependencies")
elif code_license not in urlopen(link)['content']: elif code_license not in urlopen(link)['content']:
print_warning("[YEP-1.3] The license '%s' is not registered in " print_warning("[YEP-1.3] The license '%s' is not registered in "
"https://spdx.org/licenses/ . It can be a typo error." "https://spdx.org/licenses/ . It can be a typo error."