[ltt-dev] [UST PATCH] Get Online targets checker

Matthew Khouzam matthew.khouzam at ericsson.com
Mon Mar 28 16:20:26 EDT 2011


Duly noted. I added the other stuff

"Now passes checkpactch.pl and has no obvious style problems and is ready for submission."
as a way of saying I ran the tests. Wasn't sure where to put it without breaking the git import.


On 11-03-28 03:25 PM, Alexandre Montplaisir wrote:
> On 11-03-28 01:12 PM, Matthew Khouzam wrote:
>> Added a check in get online pids to only send the pids that are
>> currently online.
>> Now passes checkpactch.pl and
>> has no obvious style problems and is ready for submission.
>>
>> Signed-off-by: Matthew Khouzam<matthew.khouzam at ericsson.com>
>> ---
> I have to nitpick here (we can't let you go can we ;) )
>
> The first line of git messages should be at the present tense ("Add a
> check...") and not go over 50 characters.
> The second line should ALWAYS be empty.
> If you use a real editor to generate the commits (like vim, not the one
> in Eclipse for example...) it will warn you of those things.
>
>
> Handy link:
> http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html
>




More information about the lttng-dev mailing list