[Pkg-scicomp-devel] [Pkg-scicomp-commits] [SCM] paraview branch, master, updated. 8c06dcb12df5d5fcdd30df2231e2fa58172ca9fa

[Rafael Laboissiere]

* Christophe Prud'homme <prudhomm at debian.org> [2009-05-21 05:27]:

> tags 516689 pending
> tags 528640 pending
> tags 529632 pending
> thanks
> 
> The following commit has been merged in the master branch:
> commit 8c06dcb12df5d5fcdd30df2231e2fa58172ca9fa
> Author: Christophe Prud'homme <prudhomm at debian.org>
> Date:   Thu May 21 07:26:11 2009 +0200
> 
>     fixed a few bugs in paraview(build with Qt4.5 and conflict with vtkpython)

Congratulations, you made your first successful git push!

That said, I would advise our team members to break down their commits
into smaller pieces. I see that in the commit 8c06dcb above, you lumped
together several unrelated changes and bug fixes.  This lack of fine
granularity makes it harder to track down problems later.

Git is actually great for addressing this issue, thanks to the command
"git add -i".  You can find a good introduction to this feature in a blog
post by Ryan Tomayko [1]. Essentially, this command allows you to
selectively add individual files *_and_* hunks of the diff patch to the
index.  You can even edit a patch to remove lines that appear together in
a hunk.

[1] http://tomayko.com/writings/the-thing-about-git

Happy hacking,

-- 
Rafael