Code review comment for lp://qastaging/~peco/ubuntu/maverick/sessioninstaller/newfix-for-793396

Revision history for this message
Luke Yelavich (themuso) wrote :

Thanks for your work. It appears your branch has 2 changelog entries. Could you plesae merge the entries into 1? Please also target the fix to maverick-proposed, should you feel its worthwhile pushing a fix to maverick updates.

I'd also advise the version to be 0.20+bzr115-0ubuntu1.1, as this makes sure the version doesn't conflict with any later package versions of sessioninstaller in other Ubuntu releases.

Thanks again.

review: Needs Fixing

« Back to merge proposal