[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#18850: smerge-mode: use diff-check-labels
From: |
Ivan Shmakov |
Subject: |
bug#18850: smerge-mode: use diff-check-labels |
Date: |
Tue, 28 Oct 2014 09:19:36 +0000 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
>>>>> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>>> That looks OK, yes. Tho you don't need to change -L to --label.
>> The point is that, strictly speaking, (diff-check-labels) only
>> checks for the --label option proper, not for its aliases.
> Of course, part of the question, then is: are there diff commands
> that handle one of the two but not the other. If there are, then we
> should use the one that's more often supported, and if there aren't
> then it doesn't matter.
I have no reason to insist on doing it either way. However, it
still feels inconsistent to check for a specific option, and
then use its alias.
Then, however, there’s the following in diffutils/NEWS:
> User-visible changes in version 2.8:
[…]
> * The following diff options are still accepted, but are no longer
> documented.
> They may be withdrawn in future releases.
> -h (omit; it has no effect)
> -H (use --speed-large-files instead)
> -L (use --label instead)
(Version 2.8 was apparently released March, 2002.)
> So far, smerge-mode hasn't even bothered to check, so maybe the check
> isn't that important any more (IIRC it was added for Solaris's diff,
> tho I'd suspect that current derivatives of Solaris woujld accept
> either -L or --label or both).
Again, I have no reason to argue for retaining the check,
especially given that none of my systems seem to have diff(1)
versions other than the one from Diffutils, as of 3.0, or a
later version.
Also to note:
> User-visible changes in version 2.0:
[…]
> * diff options renamed:
> --label renamed from --file-label
Now, given that 2.7 – the oldest version available from
http://ftp.gnu.org/gnu/diffutils/ – dates back to 1994…
--
FSF associate member #7257 http://boycottsystemd.org/ … 3013 B6A0 230E 334A
- bug#18824: split diff-check-labels off diff-no-select, Ivan Shmakov, 2014/10/25
- bug#18824: split diff-check-labels off diff-no-select, Stefan Monnier, 2014/10/25
- bug#18850: smerge-mode: use diff-check-labels, Ivan Shmakov, 2014/10/27
- bug#18850: smerge-mode: use diff-check-labels, Ivan Shmakov, 2014/10/27
- bug#18850: smerge-mode: use diff-check-labels, Stefan Monnier, 2014/10/27
- bug#18850: smerge-mode: use diff-check-labels, Ivan Shmakov, 2014/10/27
- bug#18850: smerge-mode: use diff-check-labels, Stefan Monnier, 2014/10/27
- bug#18850: smerge-mode: use diff-check-labels,
Ivan Shmakov <=
- bug#18850: smerge-mode: use diff-check-labels, Stefan Monnier, 2014/10/28