[slime-devel] Re: [slime-cvs] CVS update: slime/HACKING

Helmut Eller <heller@common-lisp.net> writes:
+If you would like to send us improvements you can create a patch with +C-x v = in the buffer or with 'diff -c'. You can also include a +ChangeLog entry describing your change.
Hey, I can't read 'diff -c'! But I'm willing to learn if you can cite some virtues :-)

Luke Gorrie <lgorrie@common-lisp.net> writes:
Helmut Eller <heller@common-lisp.net> writes:
+If you would like to send us improvements you can create a patch with +C-x v = in the buffer or with 'diff -c'. You can also include a +ChangeLog entry describing your change.
Hey, I can't read 'diff -c'! But I'm willing to learn if you can cite some virtues :-)
It's 'Context diff' and is very similar to 'Unified diff', ie 'diff -u'. Instead of having separate old and new sections they're just annotated by '+' and '-' in the first column. I find them much easier to read. 'james -- James A. Crippen <james at unlambda.com> Lambda Unlimited 61.2204N, 149.8964W Recursion 'R' Us Anchorage, Alaska, USA, Earth Y = \f.(\x.f(xx))(\x.f(xx))

Luke Gorrie <lgorrie@common-lisp.net> writes:
Helmut Eller <heller@common-lisp.net> writes:
+If you would like to send us improvements you can create a patch with +C-x v = in the buffer or with 'diff -c'. You can also include a +ChangeLog entry describing your change.
Hey, I can't read 'diff -c'! But I'm willing to learn if you can cite some virtues :-)
I changed it to -u. I too think -c is harder to read. Initially I made a reference to the Emacs manual and they use -c, probably because context diffs have a better chance to apply cleanly. I said -c for consistency. But then I realized that their requirements aren't quite appropriate for our simple project and removed the link. Helmut.

Helmut Eller <e9626484@stud3.tuwien.ac.at> writes:
Luke Gorrie <lgorrie@common-lisp.net> writes:
Helmut Eller <heller@common-lisp.net> writes:
+If you would like to send us improvements you can create a patch with +C-x v = in the buffer or with 'diff -c'. You can also include a +ChangeLog entry describing your change.
Hey, I can't read 'diff -c'! But I'm willing to learn if you can cite some virtues :-)
I changed it to -u.
I too think -c is harder to read. Initially I made a reference to the Emacs manual and they use -c, probably because context diffs have a better chance to apply cleanly. I said -c for consistency. But then I realized that their requirements aren't quite appropriate for our simple project and removed the link.
I think they also use -c because it is POSIX supported but -u is a GNU extension. At least, I recall the last time I used Solaris that it didn't have -u in its diff, but it did have -c, and presumed then that -c was a POSIX specified option. 'james -- James A. Crippen <james at unlambda.com> Lambda Unlimited 61.2204N, 149.8964W Recursion 'R' Us Anchorage, Alaska, USA, Earth Y = \f.(\x.f(xx))(\x.f(xx))

"James" == James A Crippen <james@unlambda.com> writes: James> Helmut Eller <e9626484@stud3.tuwien.ac.at> writes: Luke Gorrie <lgorrie@common-lisp.net> writes:
Helmut Eller <heller@common-lisp.net> writes:
+If you would like to send us improvements you can create a patch with +C-x v = in the buffer or with 'diff -c'. You can also include a +ChangeLog entry describing your change.
Hey, I can't read 'diff -c'! But I'm willing to learn if you can cite some virtues :-)
I changed it to -u.
I too think -c is harder to read. Initially I made a reference to the Emacs manual and they use -c, probably because context diffs have a better chance to apply cleanly. I said -c for consistency. But then I realized that their requirements aren't quite appropriate for our simple project and removed the link.
James> I think they also use -c because it is POSIX supported but -u is James> a GNU extension. At least, I recall the last time I used Solaris James> that it didn't have -u in its diff, but it did have -c, and James> presumed then that -c was a POSIX specified option. Also, in case someone didn't know, for XEmacs/Emacs users, M-x diff-context->unified saves the day. --J.

Helmut Eller <e9626484@stud3.tuwien.ac.at> writes:
I too think -c is harder to read. Initially I made a reference to the Emacs manual and they use -c, probably because context diffs have a better chance to apply cleanly.
Actually, I think it's completely RMS's preference. He doesn't like to read unified diffs. -- Alan Shutko <ats@acm.org> - I am the rocks. I just posted it, I don't think I can explain it.
participants (5)
-
Alan Shutko
-
Helmut Eller
-
james@unlambda.com
-
Jan Rychter
-
Luke Gorrie