Re: [PATCH] VGA-softcursor.txt: Point at the correct directory of vgacon.c

[Randy Dunlap <randy.dunlap@oracle.com>]

On Mon, 25 Jun 2007 10:47:21 -0700 (PDT) Ioannis Barkas wrote:

> Fixes wrong, outdated path of vgacon.c
> 
> Hello we are Ioannis Barkas (risc4all@yahoo.com) and
> Nikos Barkas (doubl3nuts@yahoo.com).
> 
> VGA-softcursor.txt points to vgacon.c at a wrong
> directory so we fix it.
> 
> There is the patch(we do not attach it as many people
> suggest not to do so) from diff -u:

That didn't work so well.  Looks like yahoo mail is splitting
the patch lines at places that you didn't intend, so the
patch cannot be applied by 'patch':  'patch' says that the
patch is malformed.

You will need to continue to use attachments (yes, we prefer
inline patches instead of attachments) or you will need to
configure yahoo mail so that it doesn't muck up your inline
patches (I don't know if this is possible) or you will need
to use a different (and decent) mail client.  When you are
ready to try again, please send the patch to yourself first
and make sure that the received email patch can be applied
cleanly.


> --- /linux-2.6.21/Documentation/VGA-softcursor.txt
> 2007-04-26 04:08:32.000000000 -0400
> +++ /linux/Documentation/VGA-softcursor.txt	2007-06-25
> 02:21:23.000000000 -0400
> @@ -4,11 +4,11 @@
>     Linux now has some ability to manipulate cursor
> appearance. Normally, you
>  can set the size of hardware cursor (and also work
> around some ugly bugs in
>  those miserable Trident cards--see #define
> TRIDENT_GLITCH in drivers/video/
> -vgacon.c). You can now play a few new tricks:  you
> can make your cursor look
> -like a non-blinking red block, make it inverse
> background of the character it's
> -over or to highlight that character and still choose
> whether the original
> -hardware cursor should remain visible or not.  There
> may be other things I have
> -never thought of.
> +console/vgacon.c). You can now play a few new tricks:
>  you can make your
> +cursor look like a non-blinking red block, make it
> inverse background of the
> +character it's over or to highlight that character
> and still choose whether
> +the original hardware cursor should remain visible or
> not.  There may be other
> +things I have never thought of.
>  
>     The cursor appearance is controlled by a
> "<ESC>[?1;2;3c" escape sequence
>  where 1, 2 and 3 are parameters described below. If
> you omit any of them,


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***