[80511] branches/gsoc11-statistics

[Ryan Schmidt]

On Jul 13, 2011, at 23:48, derek at macports.org wrote:

> Revision: 80511
>          http://trac.macports.org/changeset/80511
> Author:   derek at macports.org
> Date:     2011-07-13 21:48:08 -0700 (Wed, 13 Jul 2011)
> Log Message:
> -----------
> Initial commit of statistics collection Ruby on Rails app.

This is a lot of files. I would have expected to see many more smaller commits instead of one huge one. Commit early and commit often, so that individual logical changes appear as individual commits and can be reviewed on their own merits, and errors can be spotted sooner. I'll guess nobody is going to review this large commit, so errors might be overlooked.


> - Seed database with all ports and categories (seeds.rb) - data comes from mpwa script

This file is enormous. The commit mail was over 7MB, mostly due to this file, which as you say contains a database of all ports and categories. Naturally, such a database will quickly be out of date. Why does it need to be committed at all? Isn't it dynamically generated somehow?