[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Help to help: bugs squash.



Hi Oded,

I've been meaning to reply to your commits basically since I saw them, cause 
they made me very happy :-) (It's also true, that I (think) I have seen some 
issues with them, but mostly I'm happy about your work and unhappy that I 
didnt find time to reply yet, because I was busy with other stuff...) 

First they simply make me happy, because it's always great to see more good 
people doing good work on Debian Edu and because I hope I will see more of 
them :))

I'm now writing this, because I fear you have seen my other mails on the list 
today and that you might be a bit disappointed to not really have received 
feedback about them. I know I would be curious for feedback if I where 
you :-) And I do think your contributions and giving you feedback is 
important.

And now it's the middle of the night and the gathering is almost over for me, 
I will traveling home tomorrow morning and already commited to other stuff 
which I plan to do during my travel time. So it will (very probably) take 
some more time until I can properly comment on them. But maybe someone else 
will jump in, I'm just a single developer.. ;)

Anyway...

On Dienstag, 20. Januar 2009, Oded Naveh wrote:
> 	O.K. I gather it's safe to check in with 'svn commit'.
> Else, stop me now, so I don't "shoot someone's leg" (or higher;).

No, I dont plan to stop you at all :)

As said, I have had some "issues" and/or questions when reading your commit 
diffs but nothing were I felt being shot at ;-) Quite the opposite. (!)

One issue, and thats probably the main thing I saw, is that I thought your 
patches were too big (especially the 2nd but also the 1st). Better split the 
patch into small logical units, as it's much easier to review and understand. 
And always provide a meaningful debian/changelog entry :) (If I remember 
correctly, you have done that (good changelog), but I wont check this now, 
this mail has gotten way longer than I initially thought anyway.)

Secondly, your 2nd patch was maybe too intrusive _at this moment in time_, 
that is, so close before the Lenny release. (Even though the Edu lenny 
release very likely will be later than the Debian lenny release. But we 
should also stabilize our packages and not introduce intrusive new things.)
But thats my opinion also, and maybe just a misunderstanding on my side of 
your work ;-)

Last but not least, you cannot really shot us in the leg ;-) If the commit is 
really unsuitable, we can always revert it :-D 
_Maybe_ you want to that with your 2nd patch - and then commit the same 
changes again in several commits?!!?

(And definitly: if the patch has to be that long to fix stuff we need to fix 
(now), so be it! :-)

> 	Regarding Bugzila either its messaging mechanism is broken again, or
> its messages containing attachments gets blocked at this side.
> I attached a proposed patch to bug 1316 [1], yet nothing showed up here.
> [1] http://bugs.skolelinux.no/cgi-bin/bugzilla/show_bug.cgi?id=1316

I see two attachments from you there now. So all is good here?


Oh, and "btw", I read your idea on debian-boot the other day, to solve the 
problem with the too long profile question by making it even longer - this 
sounds like a really elegant hack!!


> Thanks again.
> Odd.

Thank you!


As this mail has gotten very long, let me summarize it, those 2 extra lines 
wont hurt: please keep up the good work, but maybe make smaller commits.

I will send another mail while looking at the patches. This mail was just 
written out of memory :-)


regards,
	Holger

Attachment: signature.asc
Description: This is a digitally signed message part.


Reply to: