You are here -> Home » Forums » Discussions » Magellan

Magellan

Corrections et améliorations

Author Message
danman
# le 3/08/2010 à 14h48
Heureux d'être là
Group : Member

Bonjour,

Ce matin même, j'ai commencé à corriger quelques erreurs dans logram DE, principalement dans magellan.

j'ai pour le moment : - Corrigé le makefile principal, qui demandait a etre root pour compiler (il faisait l'installation en meme temps ...) et donc pour nettoyer aussi. - Corrigé le plugin ghetto de panache, mauvaise include pour phonon. - Corrigé le code de magellan pour le rendre compilable (et démarrable).

Actuellement, j'ai droit à des erreurs de signaux, ce qui fait que le navigateur ne fait rien, mais je suis sur la bonne voie :) .

voila, l'adresse du clone est sur gitorious : http://gitorious.org/~danman/logram/corrections_magellan , si vous avez des avis sur le code, ou des requetes ;).

EDIT : A leobuntu et leo2sulivan : est-ce que je suis autorisé à modifier des parties ? par exemple la toolbar, qui est "salement" codé, au sens où je dois faire une forward declaration pour que ça marche ?

merci.

Editing

  • le 3/08/2010 à 17h03 by danman : demande à leobuntu et leo2sulivan
Leo`
# le 3/08/2010 à 18h24
And it's a hard rain a gonna-fall
Avatar
Group : Codeur

Salut danman,

dans la mesure où je ne m'occupe plus du tout de Magellan, vas-y, te gène pas si tu veux améliorer ou changed quoi que ce soit. J'ai bien que conscience que j'avais (assez) mal codé ça à l'époque... et puis de toute façon, c'est pas sous GPL pour rien ;)

Good luck, si tu as une question sur le code, n'hésite pas, même ça fait des mois que j'y ai pas touché :D

:)

danman
# le 3/08/2010 à 18h41
Heureux d'être là
Group : Member

Non non, j'ai deja quadrillé tout le code ;) . merci pour l'autorisation, je suppose que c'est pareil pour leo2sulivan alors.

et puis si je demande, c'est juste pour éviter de me faire crier dessus pour avoir refait un truc entier alors qu'il y avait deja un truc existant :p .

Editing

  • le 3/08/2010 à 18h42 by danman : edition
steckdenis
# le 3/08/2010 à 19h19
Ça marche !
Avatar
Group : Administrateur

C'est pleinement ton droit de refaire tout ce que tu veux, c'est libre :D . Et s'ils te crient dessus, et que ce n'est pas justifié (ne me faites pas dire ce que je n'ai pas dit :-° ), alors tu peux les ignorer.

Editing

  • le 3/08/2010 à 19h21 by steckdenis : http://gitorious.org/~danman/logram/corrections_magellan/commit/8212666a285a0a8b4c2f4c593ae9392c07684f45 : supprime les Makefile générés par QMake, ils sont dépendants de la personne qui compile.
  • le 3/08/2010 à 19h21 by steckdenis : http://gitorious.org/~danman/logram/corrections_magellan/commit/8212666a285a0a8b4c2f4c593ae9392c07684f45 : le -march=i686 du CFLAGS du .pro ne doit pas être là. -mtune=generic implicite, inutile.
  • le 3/08/2010 à 19h23 by steckdenis : http://gitorious.org/~danman/logram/corrections_magellan/commit/030609f90e61d2ef8b3911d2b630b8f46bc6f5ae : llibs_all requis car lbase dépend de llibs installé. Corrigé dans Setup par CMake ;-) .

KDE le fait depuis 10 ans.

danman
# le 3/08/2010 à 21h45
Heureux d'être là
Group : Member

pour le llibs_all, on ne peux pas tout simplement ajouter un répertoire d'include/de lib ? parce que compiler en root, bof bof :S .

ou alors dire a l'utilisateur qu'il doit installer llibs avant de continuer par exemple.

et sinon, pourquoi le cmake est pas dans le git ? :D

pour les makefiles, ils ont du se glisser par erreur :-° (sauf dans llbase, car c'est pour lancer le reste)

oki pour le reste.

merci des infos :D

edit: Clean ne devrait pas supprimer les makefile aussi ?

Editing

  • le 3/08/2010 à 21h46 by danman : makefile
steckdenis
# le 3/08/2010 à 21h54
Ça marche !
Avatar
Group : Administrateur

Les .h ne suffisent pas, le lieu a également besoin de savoir où se trouvent les libs. Tu peux t'amuser à tuner les paramètres -I et -L du compilo (CFLAGS += -Ichemin ; LIBS += -Lchemin dans les .pro).

CMake est comme QMake, mais en bien plus évolué. Ça prend pas mal de temps de passer de l'un à l'autre, même si ça a des avantages (pourcentage de progression, compilation de plusieurs projets en une commande facilitée, etc). CMake.org te donnera plus d'infos, mais c'est une tâche assez énorme ^^ .

Make clean appelle normalement le "make distclean" de QMake. Si qmake ne supprime pas les makefiles, c'est de sa faute :) .

Voilà. Bon courage, et merci ! Gitorious sert enfin, et c'est agréable de voir un développeur actif qui committe ce qu'il fait :) .

KDE le fait depuis 10 ans.

danman
# le 3/08/2010 à 22h15
Heureux d'être là
Group : Member

De rien, mais je connais Cmake ;).

@Leobuntu : finalement j'ai bien une question, qu'est est la différence entre tes slots open, openUrl, et load o_O

danman
# le 4/08/2010 à 1h00
Heureux d'être là
Group : Member

voila, les pages webs se chargent, mais la QTabBar n'est pas fonctionnelle(edit : très facile a activer ;)) et je n'ai pas mis les titres/icones des sites.

je pars demain pour une semaine, mais je devrais pouvoir travailler dessus ;) .

edit : un ptit screen de ce que ça donne actuellement (c'est pas grand chose, mais ça avance :) ) :

Magellan

(c'est en lien car ça pese 400 Ko quand meme, donc pour les petites connexions ...).

leobuntu : pour la LAddressBar, tu mettais le style a la barre entière, je l'ai mis juste a la lineedit et ça fonctionne ;)

je push bientot.

Editing

  • le 4/08/2010 à 1h14 by danman : qtabbar facile a activer
  • le 4/08/2010 à 11h12 by danman : avancée de la barre.
Leo`
# le 5/08/2010 à 19h26
And it's a hard rain a gonna-fall
Avatar
Group : Codeur

Salut danman, désolé de pas avoir pu poster plus tôt, je voyage un peu, j'ai pas trop le temps :

en ce qui concerne les slots, je peux pas vraiment te dire vu que j'ai absolument plus le code en tête (ça remonte à 9 mois... ) mais en gros, je dirais - de mémoire - que open appelait une dialog box pour demander à l'user d'ouvrir un fichier (.html), openUrl lui demandait d'entrer une url, probablement en mettant le focus et selectAll() sur la barre d'adresse, comme Firefox, et le dernier est appelé pour commencer le chargement de la page une fois que le bouton "go" à été cliqué. Je regarde ce weekend si j'ai le temps et je te confirme ça.

pour ce qui est de la LAdressBar, de mémoire, le code concernant les style sheets était assez... moche, et pour cause, ça devait être la première fois que je les utilisait. Si tu veux optimiser ça, c'est très bien. Celà dit, si tu as le temps, je te suggère de plutôt implémenter ça sous forme de paintEvent (je te renvoie vers la doc si tu connais pas... en gros une fonction virtual protected que tu peux réimplémenter et qui gère l'affichage, le "dessin" du widget), ce sera un peu plus galère pour l'affichage du texte/etc, mais t'auras beaucoup plus de possibilités (si tu veux faire un gradient ou quoi par exemple, je sais pas...) et ce sera plus clean qu'une style sheet.

Voila, hésite pas à répondre si t'as d'autres questions

à plus :ninja:

Editing

  • le 5/08/2010 à 19h30 by Leo` : quand je te demande un retour à la ligne tu le met, espèce de **** de site :D

:)

danman
# le 6/08/2010 à 13h46
Heureux d'être là
Group : Member

C'est pas la peine ;) j'ai deja enlevé et recréé des slots.

Pour la styleSheet, ça fonctionne (maintenant) donc pour le moment je vais laisser comme ça, ya des choses plus importantes a faire :) (mais oui je connais).

je vais bientot commit, est-ce que quelqu'un sait comment changer l'auteur d'un commit ? j'avais oublié de configurer git sur ce portable, et donc c'est le nom de mon frere, et git commit --amend ne le change pas :S .

danman
# le 6/08/2010 à 14h12
Heureux d'être là
Group : Member

Voila pushé, tant pis pour l'auteur.

j'attend vos commentaires :)

edit: j'ai oublie de commit des fichiers, donc faut attendre que je récupère le portable de mon frere (chose difficile, etant un bon joueur, mais je vais y arriver :ninja: )

Editing

  • le 8/08/2010 à 10h33 by danman : edition
  • le 9/08/2010 à 19h24 by danman : erreur