YunoHost-Apps / lutim_ynh

Lutim package for YunoHost
https://lut.im
GNU General Public License v3.0
5 stars 7 forks source link

Merge des 2 versions de lutim #1

Closed maniackcrudelis closed 9 years ago

maniackcrudelis commented 9 years ago

J'ouvre une issue pour voir comment merger nos travaux respectif. Si cela te convient.

maniackcrudelis commented 9 years ago

Bon, pour le moment, je suis confronté à des nouvelles erreurs avec carton, qui me sort soudainement des erreurs de dépendances sur d'autres modules. Il est d'humeur taquine aujourd'hui.

Mais je note déjà quelques différences:

Pour ce que j'ai vu rapidement, car je me prend la tête avec carton...

matlink commented 9 years ago

Je n'ai suivi que ce tuto : http://framacloud.org/cultiver-son-jardin/installation-de-lutim/ Il est peut-être plus sage d'embarquer les sources en effet, à voir. As tu une adresse xmpp que ce soit plus simple pour discuter de cela ?

maniackcrudelis commented 9 years ago

Oui, xxx Mais je suis pas très présent car j'ai malheureusement pas beaucoup de temps. Je garderais toutefois mon client jabber actif quand je serais sur lutim.

Je veux bien que tu fasses un essai de mon package pour voir si tout les problèmes que je rencontre aujourd'hui sont à incomber à ma VM ou pas.

matlink commented 9 years ago

Le problème est que je n'ai pas très envie de supprimer mon install' sur mon serv de prod. Je vais tester sur mon rpi.

maniackcrudelis commented 9 years ago

En coup de vent, avec un VM à la bonne heure (ça va m'agacer ça!), le setup se passe bien. J'avais un guillemet simple mal placé dans le fichier de conf. Il est intéressant d'ailleurs de savoir que le service ne démarre pas pour ce genre d'erreur. Mais sans log, car le fichier init.d filtre sa sortie. Il faut faire sauter la redirection d'erreur dans la section start pour voir les problèmes de démarrage.

matlink commented 9 years ago

Oui, il peut être intéressant de mettre la sortie standard du service lutim dans un fichier de log (/var/log/lutim/lutim.conf) par exemple, afin d'aider le debuggage. Ainsi, lutim pourra être géré au niveau de l'interface web également (on pourra y consulter les logs, start/stop, enable/disable le service).

maniackcrudelis commented 9 years ago

J'avais envisagé de passer lutim en service dans l'interface ynh, mais j'avais mis l'idée de côté (par flemme sans doute, je ne me sert jamais de cette interface). Toutefois, il est vrai que Mme Michu se sert rarement de ssh, donc il serait bon de le passer par ynh. Et de rétablir un log par la même occasion. En revanche, j'ignore où on passe le log dans le fichier init.

maniackcrudelis commented 9 years ago

Je décortique un peu ton script pour voir les points à merger.

La vérif du port, https://github.com/matlink/lutim_ynh/blob/master/scripts/install#L12. Bon... j'allais dire qu'il fallait éviter pour ne pas ouvrir le port sur le firewall, mais en fait non, c'est encore une autre commande. Donc c'est plutôt une bonne idée!

https://github.com/matlink/lutim_ynh/blob/master/scripts/install#L33 C'est une bonne idée également de vérifier le dossier cible, je ne le fais jamais, mais c'est plutôt bon de vérifier.

https://github.com/matlink/lutim_ynh/blob/master/scripts/install#L69 Très judicieux, c'est à garder!

Par contre je suis surpris que tu n'es pas de problème avec les droits du service. https://github.com/ldidry/lutim/issues/50 Pour ma part, je suis forcé de patcher le script init, sinon il créé ses fichiers en root et hypnotoad ne peux plus les lire!

https://github.com/matlink/lutim_ynh/blob/master/scripts/install#L92 C'est quoi? Je ne connais pas cette commande ynh.

https://github.com/matlink/lutim_ynh/blob/master/scripts/install#L95 Je créé une issue pour ça, c'est le point où je bloque pour le moment...

maniackcrudelis commented 9 years ago

Que dirais-tu d'un kanboard pour organiser le travail restant à faire? Il y a un package pour ynh qui marche très bien.

(Dernier spam pour cette nuit, promis)

matlink commented 9 years ago

Salut !

matlink commented 9 years ago

L'installation des paquets est vraiment très très lente sur ma machine virtuelle Virtualbox. Pour la date est à l'heure, je vois pas le problème :/

maniackcrudelis commented 9 years ago

Brièvement (je suis sur mon téléphone) Le problème de la date est un problème qui se pose avec les instantanés sans les additions client. Au démarrage d'un instantané enregistré à son état d'exécution (pas de reboot propre) la date est celle de la dernière exécution. Il faut passer en ligne de commande sur l'hôte pour réinitialiser l'horloge du bios de la vm.

maniackcrudelis commented 9 years ago
matlink commented 9 years ago

Non pas de problèmes de droits pour les images. Pour l'accès aux utilisateurs, de base yunohost accepte tous les utilisateurs du système donc ça pose pas de soucis. On peut retirer la commande je pense. T'as le lien du kanboard ?

maniackcrudelis commented 9 years ago

Je t'ai envoyé le lien sur jabber avec login et mdp à changer hier soir. Si tu ne les retrouve pas je te renverrai tout ça