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

Added option 'encoding' to songbook files #69

Merged
merged 2 commits into from
Nov 25, 2014
Merged

Added option 'encoding' to songbook files #69

merged 2 commits into from
Nov 25, 2014

Conversation

paternal
Copy link
Contributor

Closes #62.

@paternal
Copy link
Contributor Author

C'est une solution à #62, en bien plus simple : si une option encoding est présente dans un ficher .sb, alors tous les fichiers sont ouverts en utilisant cet encodage. Sinon, on fait comme avant : on essaye de deviner.

@Luthaf
Copy link
Contributor

Luthaf commented Nov 23, 2014

Ça m'a l'air plutôt bien, et ça devrait résoudre le problème pour les cas simples.

Il faudra juste ajouter de la doc sur ce point.

@paternal
Copy link
Contributor Author

Il faudra juste ajouter de la doc sur ce point.

Effectivement : patacrep/patacrep-doc#7

Ça m'a l'air plutôt bien, et ça devrait résoudre le problème pour les cas simples.

Ça fonctionne bien pour patadata : patacrep/patadata#15


Je propose d'accepter ce patch, et ne ne plus toucher à l'encodage tant qu'on a pas d'autres problèmes : l'autre solution proposée en #62 me paraît tout de même pas anecdotique à mettre en place, et intrusive dans les fichiers à gérer. Du coup, tant que le problème ne se présente pas, je pense qu'on peut considérer que c'est résolu. Je propose donc de fermer #62, ce ticket (#69), patacrep/patadata#15, et patacrep/patacrep-doc#7 (une fois documenté).

@Luthaf
Copy link
Contributor

Luthaf commented Nov 25, 2014

Ça me va.

Luthaf added a commit that referenced this pull request Nov 25, 2014
Added option 'encoding' to songbook files
@Luthaf Luthaf merged commit 4205520 into python3 Nov 25, 2014
@Luthaf Luthaf deleted the encoding branch November 25, 2014 11:16
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.

None yet

2 participants