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? 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
