public inbox for [email protected]  
help / color / mirror / Atom feed
From: Andreas 'ads' Scherbaum <[email protected]>
To: [email protected]
Subject: Re: Remove "unsupported version" warning for Greenplum
Date: Fri, 22 Jan 2016 11:12:58 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <CA+OCxowjQ=bovYE-WzZfXri+iFp0kmqcx8GrXDPBhXjLyJN5Vg@mail.gmail.com>
References: <[email protected]>
	<CA+OCxoxZQzjb5OMRGJp+WiYLhEpvGqer-QLLhutRivea575KvA@mail.gmail.com>
	<[email protected]>
	<CA+OCxozqNONmJBPAf0QCaQCLhAidrWGrb3eBvveb8t7388L3=g@mail.gmail.com>
	<[email protected]>
	<CABUevExjUYGAfHdPbc_P=iFjrdKwy8_7gX5S431wFke0WxY+Gg@mail.gmail.com>
	<CA+OCxozfkB-c9p8jzQxr=dPwXr11HeVi5ErVWFZkNTRLbsd8Mw@mail.gmail.com>
	<CABUevEzC1Hc80dxhbCV9-j9vmM5xchH6xidsYeK_4aa7SVHrCA@mail.gmail.com>
	<CA+OCxowjQ=bovYE-WzZfXri+iFp0kmqcx8GrXDPBhXjLyJN5Vg@mail.gmail.com>
List-Unsubscribe:  <mailto:[email protected]?body=unsub%20pgadmin-hackers>

On 22.01.2016 11:04, Dave Page wrote:
> On Fri, Jan 22, 2016 at 9:57 AM, Magnus Hagander <[email protected]> wrote:
>>
>>
>> On Fri, Jan 22, 2016 at 10:43 AM, Dave Page <[email protected]> wrote:
>>>
>>> On Thu, Jan 21, 2016 at 10:54 PM, Magnus Hagander <[email protected]>
>>> wrote:
>>>>
>>>>
>>>> On Thu, Jan 21, 2016 at 11:35 PM, Andreas 'ads' Scherbaum
>>>> <[email protected]> wrote:
>>>>>
>>>>> On 21.01.2016 10:31, Dave Page wrote:
>>>>>>
>>>>>> On Tue, Jan 19, 2016 at 10:54 PM, Andreas 'ads' Scherbaum
>>>>>> <[email protected]> wrote:
>>>>>>>
>>>>>>> On 19.01.2016 16:03, Dave Page wrote:
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> Your patch won't apply again. I have no idea why - I'm trying to do
>>>>>>>> it
>>>>>>>> on my Mac, which is a *nix under the hood (they don't use Mac line
>>>>>>>> endings any more - that was the old Mac OS 9 and earlier from a
>>>>>>>> decade
>>>>>>>> or so ago iirc). How are you creating them? The normal way is to do
>>>>>>>> something like:
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> And how does the attached work? Fresh clone again, only difference is
>>>>>>> a
>>>>>>> warning (not an error) for whitespaces removed.
>>>>>>
>>>>>>
>>>>>> Still doesn't apply. I tried on the following systems:
>>>>>>
>>>>>> Mac OS X 10.11.1 - git version 2.5.4 (Apple Git-61)
>>>>>> Windows 7 Enterprise SP1 - git version 1.8.1.msysgit.1
>>>>>> CentOS release 6.7 (Final) - git version 1.7.1
>>>>>>
>>>>>> I'm fairly convinced at this stage that there's something funky on
>>>>>> your system. Perhaps we should take a look next week when we're both
>>>>>> in Brussels?
>>>>>>
>>>>>
>>>>> After debugging back and forth with Magnus, it looks like that Google
>>>>> Mail
>>>>> is fooling you. Your downloaded file has a different line ending, and
>>>>> your
>>>>> file is 3269 bytes, where the original file is 3210 bytes. That's 59
>>>>> additional line breaks.
>>>>>
>>>>> 4fa0990a1020e425fe95b99ea9f186de  gp-warning2.diff
>>>>>
>>>>> The file you download from the archive:
>>>>>
>>>>> http://www.postgresql.org/message-id/[email protected]
>>>>>
>>>>> is also correct.
>>>
>>> Well that's weird. But why is it only happening with your patches? I
>>> apply patches from others constantly without issues.
>>
>>
>> What MIME types do you get those in typically? Could be that gmail is
>> reacting to it being text/x-patch and not application/x-patch or something?
>
> text/x-patch or application/octet-stream seem to be fine from others.
> The one thing I noticed is the majority of patches I've received from
> others tend to be base64 encoded, whilst Ads' is not.
>
> I'd be interested to try a patch that's been zipped before attachment.

Yes, I decided to gzip them next time, before sending them.

Honestly I have no idea why only my patches. There is nothing unusual in 
my Thunderbird, pretty much a default installation. And downloading my 
patch from Thunderbird or the PG Archive results in the correct file. So 
it must be a Google thing.

Ok, now that this problem is solved, I will move the versions into the 
header files and send you a new version soon.


Thanks,

-- 
				Andreas 'ads' Scherbaum
German PostgreSQL User Group
European PostgreSQL User Group - Board of Directors
Volunteer Regional Contact, Germany - PostgreSQL Project


-- 
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers



view thread (16+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected]
  Subject: Re: Remove "unsupported version" warning for Greenplum
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox