Re: [PATCH] doc/ntsec.xml: Fix typo

2018-02-21 Thread Brian Inglis
On 2018-02-21 14:37, Corinna Vinschen wrote:
> On Feb 21 14:20, Brian Inglis wrote:
>> On 2018-02-21 14:05, Corinna Vinschen wrote:
>>> Hi David,
>>>
>>> On Feb 21 18:09, David Macek wrote:
 ---
  winsup/doc/ntsec.xml | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/winsup/doc/ntsec.xml b/winsup/doc/ntsec.xml
 index df1d54930..03293591b 100644
 --- a/winsup/doc/ntsec.xml
 +++ b/winsup/doc/ntsec.xml
 @@ -914,7 +914,7 @@ This is not valid:
  
  
 -Apart from this restriction, the reminder of the line can have as
 +Apart from this restriction, the remainder of the line can have as
  many spaces and TABs as you like.
  
 -- 
 2.16.2.windows.1
>>>
>>> The patch is malformed.  It claims to contain 7 lines (6 lines context,
>>> one line changed), but actually it has only 4 lines context.  Please
>>> check your git settings.
>>
>> "It's an een-justice!" -- Calimero
>> Check your mail client blank line squishing settings; I see:
> 
> I'm using mutt which is usually sticks to the original.  I checked again
> and when looking into the message with vi I see this:
> 
>   
> =20=20
>   
> [...]
>   
> =20=20
> 
> Why are the spaces decodes as =20 in these two lines?
> 
> In theory the decode-copy command should retain the empty lines
> and it did so before.  Why not now?!?

I see exactly the same in TB view source, but the message appears as expected.
I can see why you strenuously request git format-patch/send-email attachments 
;^>

-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada


Re: [PATCH] doc/ntsec.xml: Fix typo

2018-02-21 Thread David Macek

On 2018-02-21 14:05, Corinna Vinschen wrote:

The patch is malformed.  It claims to contain 7 lines (6 lines context,
one line changed), but actually it has only 4 lines context.  Please
check your git settings.


On 21. 2. 2018 22:56, Brian Inglis wrote:

I can see why you strenuously request git format-patch/send-email attachments 
;^>


I did use `git format-patch` to make that message (then sent using TB).
I guess I'll have to try something else next time.

--
David Macek



smime.p7s
Description: S/MIME Cryptographic Signature


[PATCH] doc/ntsec.xml: Fix typo

2018-02-21 Thread David Macek

---
 winsup/doc/ntsec.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/winsup/doc/ntsec.xml b/winsup/doc/ntsec.xml
index df1d54930..03293591b 100644
--- a/winsup/doc/ntsec.xml
+++ b/winsup/doc/ntsec.xml
@@ -914,7 +914,7 @@ This is not valid:
 
 
 

-Apart from this restriction, the reminder of the line can have as
+Apart from this restriction, the remainder of the line can have as
 many spaces and TABs as you like.
 
 
--

2.16.2.windows.1


--
David Macek



smime.p7s
Description: S/MIME Cryptographic Signature


Re: [PATCH] doc/ntsec.xml: Fix typo

2018-02-21 Thread Corinna Vinschen
Hi David,

On Feb 21 18:09, David Macek wrote:
> ---
>  winsup/doc/ntsec.xml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/winsup/doc/ntsec.xml b/winsup/doc/ntsec.xml
> index df1d54930..03293591b 100644
> --- a/winsup/doc/ntsec.xml
> +++ b/winsup/doc/ntsec.xml
> @@ -914,7 +914,7 @@ This is not valid:
>  
>  
> -Apart from this restriction, the reminder of the line can have as
> +Apart from this restriction, the remainder of the line can have as
>  many spaces and TABs as you like.
>  
> -- 
> 2.16.2.windows.1

The patch is malformed.  It claims to contain 7 lines (6 lines context,
one line changed), but actually it has only 4 lines context.  Please
check your git settings.


Thanks,
Corinna

-- 
Corinna Vinschen  Please, send mails regarding Cygwin to
Cygwin Maintainer cygwin AT cygwin DOT com
Red Hat


signature.asc
Description: PGP signature


Re: [PATCH] doc/ntsec.xml: Fix typo

2018-02-21 Thread Brian Inglis
On 2018-02-21 14:05, Corinna Vinschen wrote:
> Hi David,
> 
> On Feb 21 18:09, David Macek wrote:
>> ---
>>  winsup/doc/ntsec.xml | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/winsup/doc/ntsec.xml b/winsup/doc/ntsec.xml
>> index df1d54930..03293591b 100644
>> --- a/winsup/doc/ntsec.xml
>> +++ b/winsup/doc/ntsec.xml
>> @@ -914,7 +914,7 @@ This is not valid:
>>  
>>  
>> -Apart from this restriction, the reminder of the line can have as
>> +Apart from this restriction, the remainder of the line can have as
>>  many spaces and TABs as you like.
>>  
>> -- 
>> 2.16.2.windows.1
> 
> The patch is malformed.  It claims to contain 7 lines (6 lines context,
> one line changed), but actually it has only 4 lines context.  Please
> check your git settings.

"It's an een-justice!" -- Calimero
Check your mail client blank line squishing settings; I see:

> diff --git a/winsup/doc/ntsec.xml b/winsup/doc/ntsec.xml
> index df1d54930..03293591b 100644
> --- a/winsup/doc/ntsec.xml
> +++ b/winsup/doc/ntsec.xml
> @@ -914,7 +914,7 @@ This is not valid:
>  
>  
>  
> -Apart from this restriction, the reminder of the line can have as
> +Apart from this restriction, the remainder of the line can have as
>  many spaces and TABs as you like.
>  
>  

-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada


Re: [PATCH] doc/ntsec.xml: Fix typo

2018-02-21 Thread Corinna Vinschen
On Feb 21 14:20, Brian Inglis wrote:
> On 2018-02-21 14:05, Corinna Vinschen wrote:
> > Hi David,
> > 
> > On Feb 21 18:09, David Macek wrote:
> >> ---
> >>  winsup/doc/ntsec.xml | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/winsup/doc/ntsec.xml b/winsup/doc/ntsec.xml
> >> index df1d54930..03293591b 100644
> >> --- a/winsup/doc/ntsec.xml
> >> +++ b/winsup/doc/ntsec.xml
> >> @@ -914,7 +914,7 @@ This is not valid:
> >>  
> >>  
> >> -Apart from this restriction, the reminder of the line can have as
> >> +Apart from this restriction, the remainder of the line can have as
> >>  many spaces and TABs as you like.
> >>  
> >> -- 
> >> 2.16.2.windows.1
> > 
> > The patch is malformed.  It claims to contain 7 lines (6 lines context,
> > one line changed), but actually it has only 4 lines context.  Please
> > check your git settings.
> 
> "It's an een-justice!" -- Calimero
> Check your mail client blank line squishing settings; I see:

I'm using mutt which is usually sticks to the original.  I checked again
and when looking into the message with vi I see this:

  
=20=20
  
[...]
  
=20=20

Why are the spaces decodes as =20 in these two lines?

In theory the decode-copy command should retain the empty lines
and it did so before.  Why not now?!?


Corinna

-- 
Corinna Vinschen  Please, send mails regarding Cygwin to
Cygwin Maintainer cygwin AT cygwin DOT com
Red Hat


signature.asc
Description: PGP signature


Re: [PATCH] doc/ntsec.xml: Fix typo

2018-02-21 Thread Corinna Vinschen
On Feb 21 22:37, Corinna Vinschen wrote:
> On Feb 21 14:20, Brian Inglis wrote:
> > On 2018-02-21 14:05, Corinna Vinschen wrote:
> > > Hi David,
> > > 
> > > On Feb 21 18:09, David Macek wrote:
> > >> ---
> > >>  winsup/doc/ntsec.xml | 2 +-
> > >>  1 file changed, 1 insertion(+), 1 deletion(-)
> > >>
> > >> diff --git a/winsup/doc/ntsec.xml b/winsup/doc/ntsec.xml
> > >> index df1d54930..03293591b 100644
> > >> --- a/winsup/doc/ntsec.xml
> > >> +++ b/winsup/doc/ntsec.xml
> > >> @@ -914,7 +914,7 @@ This is not valid:
> > >>  
> > >>  
> > >> -Apart from this restriction, the reminder of the line can have as
> > >> +Apart from this restriction, the remainder of the line can have as
> > >>  many spaces and TABs as you like.
> > >>  
> > >> -- 
> > >> 2.16.2.windows.1
> > > 
> > > The patch is malformed.  It claims to contain 7 lines (6 lines context,
> > > one line changed), but actually it has only 4 lines context.  Please
> > > check your git settings.
> > 
> > "It's an een-justice!" -- Calimero
> > Check your mail client blank line squishing settings; I see:
> 
> I'm using mutt which is usually sticks to the original.  I checked again
> and when looking into the message with vi I see this:
> 
>   
> =20=20
>   
> [...]
>   
> =20=20
> 
> Why are the spaces decodes as =20 in these two lines?
> 
> In theory the decode-copy command should retain the empty lines
> and it did so before.  Why not now?!?

Anyway, patch pushed.


Thanks,
Corinna

-- 
Corinna Vinschen  Please, send mails regarding Cygwin to
Cygwin Maintainer cygwin AT cygwin DOT com
Red Hat


signature.asc
Description: PGP signature