|
Rockbox mail archiveSubject: Re: Re: uchida: r25233 - trunk/apps/pluginsRe: Re: uchida: r25233 - trunk/apps/plugins
From: Yoshihisa Uchida <y_uchida-lj_at_infoseek.jp>
Date: Thu, 18 Mar 2010 20:55:41 +0900 Hi Thomas, I think that I commit each patch under normal conditions. But in this case, these patches are created by me and these are patches to the text viewer plugin. Then these were commited once. (it is efficient.) When committing, I wrote more details, sorry. Sincerely yours, Yoshihisa Uchida y_uchida-lj_at_infoseek.jp > ----- original message ----- > send: "Thomas Martitz" <thomas.martitz_at_student.htw-berlin.de> > receive: "rockbox-dev_at_cool.haxx.se" <rockbox-dev_at_cool.haxx.se> > date: 10/03/18 03:10 > subject: Re: uchida: r25233 - trunk/apps/plugins > > Am 17.03.2010 15:22, schrieb Frank Gevaerts: > > On Wed, Mar 17, 2010 at 12:17:56PM +0100, mailer_at_svn.rockbox.org wrote: > > > >> Log Message: > >> text viewer plugin applies patches FS#8445, FS#9546, FS#9853, FS#9855, FS#9892, FS#9893, FS#9898, FS#9902, and FS#9990. > >> > > I think commit messages should have a description of the commit, not > > just a reference to the tracker. > > > > Frank > > > > > > Also, if the patches are already separate, they should also be committed > separately IMO (with a description for each as gevaerts mentioned). > Received on 2010-03-18 Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy |