Re: [PATCH 1/24] Documentation: Move variables from config.txt to separate file

Thomas Rast wrote:

 Documentation/config-vars.txt | 1691 ++++++++++++++++++++++++++++++++++++++++
 Documentation/config.txt      | 1693 +----------------------------------------

Scary.  Does -B -M -C -C -C make it any better?

On Mon, Jul 26, 2010 at 19:38, Jonathan Nieder [off-list ref] wrote:

Thomas Rast wrote:

quoted

 Documentation/config-vars.txt | 1691 ++++++++++++++++++++++++++++++++++++++++
 Documentation/config.txt      | 1693 +----------------------------------------

Scary.  Does -B -M -C -C -C make it any better?

It makes it even scarier:

 Documentation/{config.txt => config-vars.txt} |  223 ++--
 Documentation/config.txt                      | 1923 ++-----------------------

Heya,

On Mon, Jul 26, 2010 at 15:18, Ævar Arnfjörð Bjarmason [off-list ref] wrote:

On Mon, Jul 26, 2010 at 19:38, Jonathan Nieder [off-list ref] wrote:

quoted

Scary.  Does -B -M -C -C -C make it any better?

It makes it even scarier:

So, should we somehow allow the user to tell us that yes, really, we
want to treat these two files as a move for the purpose of generating
a patch diff?

-- 
Cheers,

Sverre Rabbelier

Ævar Arnfjörð Bjarmason [off-list ref] writes:

On Mon, Jul 26, 2010 at 19:38, Jonathan Nieder [off-list ref] wrote:

quoted

Thomas Rast wrote:

quoted

 Documentation/config-vars.txt | 1691 ++++++++++++++++++++++++++++++++++++++++
 Documentation/config.txt      | 1693 +----------------------------------------

Scary.  Does -B -M -C -C -C make it any better?

It makes it even scarier:

 Documentation/{config.txt => config-vars.txt} |  223 ++--
 Documentation/config.txt                      | 1923 ++-----------------------

Is it scary?

If bulk of config.txt has been moved to config-vars.txt, but either lines
the moved to config-vars.txt have further edits or more lines have been
added to the file, while the remainder of config.txt had some other
changes, the above two is more or less exactly what I would expect to see.