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

[enh] Add YEP 1.3 about licensing information #413

Merged
merged 4 commits into from
Jan 12, 2017
Merged

Conversation

zamentur
Copy link
Member

@zamentur zamentur commented Nov 15, 2016

This YEP is a draft about how inform user about app or package license. Some points regarding how to fill licence in manifest.json may be unconsensual.


Cette YEP est un brouillon à propos de la façon dont les licences doivent être indiquées dans les paquets. Certains points à propos de la façon de remplir le champ licence dans le manifeste peuvent faire débat.

@Psycojoker
Copy link
Member

Je trouve cette yep très bien écrite, mais j'ai peur que ça soit trop compliqué et un peu le entre "trop compliqué" et "pas assez compliqué".

Je pense aux fichiers debian où y a des globs pour décrire quels fichiers sont sous quelle licence, ici on a à la fois pas ce niveau de détails et un niveau de détails déjà assez poussé. J'arrive pas à savoir ce qui serait le plus utile en fin de compte.

@zamentur
Copy link
Member Author

Perso, ce que j'aimerais à terme c'est pouvoir afficher des infos dans la webadmin sur la licence de l'application. Le choix free ou non-free me semble un peu juste en terme d'information, d'autant que la notion "free" fait débat. Dans l'idée, il faudrait pouvoir afficher le texte des licences. Le must serait d'avoir un résumé compréhensible façon Creative Commons (à voir si il y a des résumés quelques part).

Pour moi il faudrait au moins que l'admin sache: est ce que c'est entièrement libre ou partiellement libre ou proprio ? Est ce que si je fais une modif sur mon instance je dois la republier forcément ?

@opi
Copy link
Contributor

opi commented Nov 16, 2016

Quelques ressources :

@maniackcrudelis
Copy link
Contributor

YEP très bien rédigée, mais un peu complexe à mon avis.
Toutefois, détailler d'avantage que free/non-free me semble indispensable tant les licences peuvent différer. Je pense notamment à Rainloop avant que la licence ne soit modifiée. RainLoop/rainloop-webmail#625

Cela étant, je ne vois pas comment faire plus simple que le nom abrégé de la license.

Copy link
Contributor

@julienmalik julienmalik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci pour cet effort.

C'est un sujet compliqué. J'ai déjà discuté de ça par le passé, je pense que résumer la license du paquet complet à un petit acronyme est difficile/tendancieux, quand ce n'est pas impossible.

Ta solution est néanmoins, AMHA, proche du "mieux qu'on puisse faire", si on veut absolument ça.
Du coup je validerais bien en l'état (validation mitigée, mais bon)


Les listes d'applications official.json et community.json n'acceptent que les paquets dont la licence est libre, de même pour la licence de l'application contenue. Certaines applications libres nécessitent des dépendances non-libres (exemple: mp3, drivers, etc.). Dans ce cas, il faut ajouter `+dep-non-free` à l'acronyme et si possible donner des précisions dans le README.md du paquet, l'intégration sera dans ce cas acceptée au cas par cas.

Dans le future, YunoHost affichera sans doute des détails sur la licence de l'application. Pour y parvenir, l'acronyme doit être celui issue de cette [liste de licences répertoriées sur gnu.org](https://www.gnu.org/licenses/license-list.fr.html) (si il y a 2 acronymes, il faut prendre celui contenant le numéro de version). Pour plus de cohérence, la casse doit être respectée.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Dans le futur"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"celui issu"


Si deux applications distinctes sont dans le même paquet d'installation et ont des licences distinctes, dans ce cas on peut utiliser le `&` pour séparer les licences.

Dans les deux cas, le mainteneur est encouragé à réfléchir à la possibilité de créer deux paquets distincts. Le manifeste permet des questions de type `app` de façon à faire référence à une autre application déjà installée.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est pas clair : "Le manifeste permet des questions de type app" meme si après y avoir réfléchi une 3eme fois j'ai compris de quoi tu parles (je mentionnerait "paramètres d'installation" par ex., plutot que "questions")

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moi j'ai pas compris de quoi il s'agit...
app dans le manifest?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est vis à vis du type de question. Dans la doc du manifeste il y a:

type : (optionnel) type de paramètre parmis domain, path, user, app, boolean et password.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Et ça fait quoi "app"?

#### YEP 1.3 - Indiquer la licence associée au paquet | brouillon | AUTO | WORKING |
La licence du paquet est à indiquer dans un fichier `LICENSE` à la racine du paquet. Attention à ne pas confondre avec la licence de l'application qui va être installée dont l'acronyme est à renseigner dans le champ `license` du manifeste.

Les listes d'applications official.json et community.json n'acceptent que les paquets dont la licence est libre, de même pour la licence de l'application contenue. Certaines applications libres nécessitent des dépendances non-libres (exemple: mp3, drivers, etc.). Dans ce cas, il faut ajouter `+dep-non-free` à l'acronyme et si possible donner des précisions dans le README.md du paquet, l'intégration sera dans ce cas acceptée au cas par cas.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En fait je ne trouve pas que la mention de "dépendance" soit appropriée. Tu peux avoir une grosse appli avec plein de code libre et une toute petite partie non libre, ou l'inverse, un gros bouzin non libre qui appelle de tps en tps un bout de code libre. Y a t il une différence fondamentalement ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour moi ça change quand même l'"engagement" en terme de PI pour l'app.
En gros une app proprio avec des dépendances MIT ou LGPL, c'est une app purement propriétaire avec une entité qui a droit de vie ou de mort dessus.

Une app LGPL ou MIT avec des dépendances proprio, c'est bof, mais à priori il est possibe de remplacer les codes proprio voir même en faire un fork sous AGPLv3... Donc pour moi la proportion de code sous license libre me semble importante tout de même.

Bref pour ma part je fais une grande différence entre mediagoblin (qui doit pouvoir gérer aussi des format propriétaire) et youtube.

maniackcrudelis added a commit to YunoHost-Apps/nextcloud_ynh that referenced this pull request Nov 16, 2016
Pour respecter les règles actuelles du manifest.
Concernant la licence, voir YunoHost/doc#413
@maniackcrudelis
Copy link
Contributor

@julienmalik, j'ai corrigé les 2 coquilles dans le texte.
Peut-on valider cette PR? Je crois que nous sommes globalement d'accord.

@M5oul, et tout le groupe Com, êtes-vous d'accord pour que les modifications des YEP soient sous le giron du groupe Apps?

@zamentur
Copy link
Member Author

@opi du coup j'ai intégré l'utilisation de la liste d'accronyme du SPDX qui me semble plus complète.

@zamentur zamentur merged commit 81fe1b1 into master Jan 12, 2017
@M5oul M5oul deleted the enh-add-yep-1-3 branch January 19, 2017 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants