[Kget] GSOC -- Review
Matthias Fuchs
mat69 at gmx.net
Sat Aug 15 17:02:34 CEST 2009
Hi,
My name is Matthias Fuchs and I'm the GSOC-student who worked on KGet (just to
be sure that you know who I am :) ). Now that GSOC is close to its end it
would be fantastic if you could review my code.
So turn your "-pedantic"-mode on and look for errors, design mistakes, coding
style stuff etc. that way the code should be hopefully in a real good state
once it enters trunk.
You can get the code here [1] or simply follow the instruction (copied from a
mail I sent to Urs Wolfer):
git clone git://github.com/mfuchs/kget-gsoc.git && cd kget-gsoc
git checkout -b GSOC/master && git pull origin GSOC/master
now to get a diff you could type
git diff master > fullDiff.diff
to update type, you should do that from while to while, to see if I commited
changes to github:
git pull origin GSOC/master
to see all commits type:
gitk
that way you could also look at only the last commits.
What is the structure of my repo?
I tried to have as few commits as possible, so I put together what belongs
together, that does not mean though that each commit would compile on its own.
Additionally the commits with upper letters like "METALINKCREATOR: ..." mean
that they are only temporary (to avoid rebasing --> that would be a pain for
you if you update), as soon as I commit all that to trunk they will be merged
with the correct commits, in this case with the one that initially adds the
MetalinkCreator.
I hope you understood my ramblings, please add comments, flamings or whatever
you please. :)
Cheers!
matthias
[1] http://github.com/mfuchs/kget-gsoc/tree/GSOC/master
PS.: Yes, I plan to stick to KGet-Developement, so your comments would not be
useless. ;)
More information about the Kget
mailing list