[MERGE] Merge Check EOL plugin into the core.

Alexander Belchenko bialix at ukr.net
Wed Mar 12 19:42:06 GMT 2008


John Arbash Meinel пишет:
> Jelmer Vernooij wrote:
> | Am Dienstag, den 11.03.2008, 15:32 +0200 schrieb Alexander Belchenko:
> |> 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.
> | No objections here code-wise, I'm just a bit worried that having this in
> | core implies that we're going to support this particular file format in
> | the future or at least would have to be able to upgrade from it. Is it
> | really necessary to have this in the core? I'd rather see it being a
> | separate plugin until actual line conversion support lands.
> |
> | Cheers,
> |
> | Jelmer
> 
> I also have my concerns about pulling it into core before it has cooked 
> a while
> as a plugin.

OK. I think the comment from James Westby is really too important (in another thread
about requiring a plugin's presence):

"Consider the case of someone using Alexander's plugin to control
line endings in their branch. If for some reason the plugin gets
deleted at some point it may be a while before they notice, and in
that time there may be code with the wrong line endings committed,"

It's enough for me to think that simplicity of installing/discoverable of plugin
may be better answer for question: how I can require to have some specific plugin
for some specific project?

I'm also leaning towards not including plugins into bzr.dev, I just thought it's
better to have such tool to help migrating new users, like IPython project.
Probably it's better to live with checkeol as separate plugin. And help to
Martin Albisetti on his work on APS. I think we are ready for something
like:

  easy-install <plugin-name>

If other people also think it's better not to merge this into the core, then bb:reject it.
The in next week I'll create separate launchpad project for bzr-checkeol and start to
provide installer for this plugin for Windows users.




More information about the bazaar mailing list