On 17 November 2014 06:21, Anders Roxell <[email protected]> wrote:
> On 17 November 2014 12:17, Taras Kondratiuk <[email protected]> > wrote: > > On 11/17/2014 12:00 PM, Maxim Uvarov wrote: > >> > >> On 11/17/2014 12:01 PM, Taras Kondratiuk wrote: > >>> > >>> On 11/15/2014 12:01 PM, Maxim Uvarov wrote: > >>>> > >>>> Merged! > >>> > >>> > >>> Guys, what about commit message? How one should understand what was the > >>> reason for this patch by looking into git log? > >>> > >> > >> yes, agree maybe we need some more clear message for that. But I think > >> short description says what was done. > >> > >>> Also it would be nice to have 'linux-generic:' prefix in the subject. > >>> So browsing via git log one could easily determine what was touched by > >>> the patch. > >>> > >> > >> No prefix by default means it's for linux-generic. > > > > > > That is not clear for a new user. > > Right prefixes make browsing through git log much more pleasant. > > I agree with you Taras! > > > Prefixes can be hierarchical: > > > > - "linux-generic: crypto: ..." or "platform: crypto:" > > I like "platform: crypto:" > > > - "cunit: crypto: ..." or "validation: crypto:" > > we should stick to "cunit: crypto:" until we change the directory to > validation right? > Yes, in progress. > > Cheers, > Anders > > > - "example: ipsec: ..." > > > > > > _______________________________________________ > > lng-odp mailing list > > [email protected] > > http://lists.linaro.org/mailman/listinfo/lng-odp > > _______________________________________________ > lng-odp mailing list > [email protected] > http://lists.linaro.org/mailman/listinfo/lng-odp > -- *Mike Holmes* Linaro Sr Technical Manager LNG - ODP
_______________________________________________ lng-odp mailing list [email protected] http://lists.linaro.org/mailman/listinfo/lng-odp
