Re: [aur-dev] some patches to gendummydata (take 3)
On Thu, Apr 07, 2011 at 03:21:48PM +0200, Lukas Fleischer wrote: On Wed, Apr 06, 2011 at 07:27:59PM -0700, elij wrote: On Wed, Apr 6, 2011 at 7:23 PM, elij elij...@gmail.com wrote: Updated with changes Lukas requested. - split logging and mysql removal into separate patches - update logging patch with uppercase variable convention - drop tabs - spaces patch - clarified commit messages hmm. looks like I missed a single line extraction in my rebase -i for splitting the mysql and logging patches. +CATEGORIES_COUNT = 17 # the number of categories from aur-schema I can fix and resend if desired. Nah, 's alright. I'll keep that in mind and fix it when applying your patches. Apart from that, your patches look fine to me now. I'll push them as soon as the gettext/Transifex transition is done. Pushed, including the CATEGORIES_COUNT fix. I also changed the random number range used in genCategory() in the MySQL dependency patch to generate 1-based IDs instead of 0-based ones. Otherwise it would generate wrong IDs and foreign key constraints on the CategoryID column would fail when importing dummy data into the database.
Re: [aur-dev] some patches to gendummydata (take 3)
On Wed, Apr 06, 2011 at 07:27:59PM -0700, elij wrote: On Wed, Apr 6, 2011 at 7:23 PM, elij elij...@gmail.com wrote: Updated with changes Lukas requested. - split logging and mysql removal into separate patches - update logging patch with uppercase variable convention - drop tabs - spaces patch - clarified commit messages hmm. looks like I missed a single line extraction in my rebase -i for splitting the mysql and logging patches. +CATEGORIES_COUNT = 17 # the number of categories from aur-schema I can fix and resend if desired. Nah, 's alright. I'll keep that in mind and fix it when applying your patches. Apart from that, your patches look fine to me now. I'll push them as soon as the gettext/Transifex transition is done.
Re: [aur-dev] some patches to gendummydata (take 3)
On Wed, Apr 6, 2011 at 7:23 PM, elij elij...@gmail.com wrote: Updated with changes Lukas requested. - split logging and mysql removal into separate patches - update logging patch with uppercase variable convention - drop tabs - spaces patch - clarified commit messages hmm. looks like I missed a single line extraction in my rebase -i for splitting the mysql and logging patches. +CATEGORIES_COUNT = 17 # the number of categories from aur-schema I can fix and resend if desired.