[MERGE] Merge Check EOL plugin into the core.
Eugene Wee
eugenew at starhub.net.sg
Wed Mar 12 04:25:51 GMT 2008
Hi James,
Alexander noted not long after the announcement of this plugin:
> Different people have different views of this problem.
> It's actually intermediate solution, and I hope something will be done
> in real line-endings support.
In fact, the name of the configuration file was originally .checkeol, I
made the suggestion that Alexander should consider a rename to .bzreol
precisely because it might be more flexible in the event that a
line-ending conversion plugin was implemented later.
At this stage, it seems to me that since the configuration file
specifies the required line-endings for given file extensions, it is
just a matter of automatically converting for the user instead of
aborting the commit and informing the user.
However, Martin Pool has suggested that the configuration file format be
changed. I am not sure if his suggested file format is better though, as
it states the required line-ending for each file extension, whereas the
current file format states file extensions for each line-ending, but
does not look like an ini file so could be less confusing (if it is
confusing at all).
Regards,
Eugene Wee
James Westby wrote:
> On Tue, 2008-03-11 at 15:32 +0200, Alexander Belchenko wrote:
>> From help topic:
>>
>> The 'checkeol' plugin installs special pre-commit hook to Bazaar VCS.
>> This hook allows you to check content of committed files for kind
>> of line-endings. If your project has strict policy about style of
>> source files you could automate EOL check with such plugin.
>> If some of committed files will have inappropriate line-endings
>> commit will be aborted.
>>
>> Here is the patch to merge this plugin into the core.
>
> Hi Alexander,
>
> Thanks for submitting this, I think it would make a valuable addition
> to the core.
>
> The code looks pretty sane to me, so I don't have any comments on that.
>
> My only comment would be, does including this prejudice any work that
> may happen in the future to support line endings in a more flexible way?
> For instance, could the .bzreol file specify all that is wanted for
> that?
>
> I don't think this should preclude the plugin being merged, I just
> wondered if you had considered the issue.
>
> Thanks,
>
> James
>
>
>
More information about the bazaar
mailing list