Re: [PATCH] Add support for commit attributes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2014-04-10 6:25 GMT+02:00 Duy Nguyen <pclouds@xxxxxxxxx>:
> On Thu, Apr 10, 2014 at 2:38 AM, Diego Lago
> <diego.lago.gonzalez@xxxxxxxxx> wrote:
>> Commit attributes are custom commit extra headers the user can
>> add to the commit object.
>>
>> The motivation for this patch is that in my company we have a custom
>> continuous integration software that uses a custom formatted commit
>> message (currently in YALM format) to show several information into
>> our CI server front-end.
>>
>> But this YALM-based commit message pollutes the commit object not being
>> human readable, so a good form of achieve the YALM's behaviour (without
>> using YALM nor any other structured language) is to add custom attributes
>> to the commit object itself.
>>
>> For example, in our CI server we show the risk of the change (that can
>> be low, medium or high); we, as said before, add this information by putting
>> YALM code inside the commit message, but the problem is that this message
>> is not human readable.
>
> If the problem is polluting human eyes, wouldn't it be better to make
> git-log to filter it out? For example, we could tell git that all
> fields (in the message body) that start with X- are "rubbish", so
> instead of showing "X-something: base64 stuff...", it shows
> "X-something: <filtered out>" instead? At least people will see that
> this commit carries human-unreadable stuff.
> --
> Duy

Writing this data into the message, the user is forced to write it in
the correct format (I think is better to write key=value pairs as an
option instead of writing as message lines with spaces in key, between
key and equal sign and value, and other mistakes). And is simpler to
parse these attributes than the message itself.

And, what if the log message is seen from the command line instead of
our CI front-end? Why the CLI user (for example) should see
information that does not need or does not want to see?

Commit attributes are extra information, not the main information,
hence this patch.

PD: Sorry for the previous message not in plain text.

-- 
Diego Lago González
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]