Skip to content

Fixing issue on Unix sockets in Native Binding - #179

Closed
soletan wants to merge 10 commits into
brianc:masterfrom
soletan:master
Closed

Fixing issue on Unix sockets in Native Binding#179
soletan wants to merge 10 commits into
brianc:masterfrom
soletan:master

Conversation

@soletan

@soletan soletan commented Aug 24, 2012

Copy link
Copy Markdown
Contributor

Binding natively connections to Unix sockets failed due to DNS lookups applied on pathname to Unix socket's folder.

Binding natively connections to Unix sockets failed due to DNS lookups applied on pathname to Unix socket's folder.
@travisbot

Copy link
Copy Markdown

This pull request passes (merged 6e0922a into ba733f5).

@travisbot

Copy link
Copy Markdown

This pull request fails (merged eaa2ba1 into ba733f5).

@travisbot

Copy link
Copy Markdown

This pull request passes (merged 663a965 into ba733f5).

@travisbot

Copy link
Copy Markdown

This pull request fails (merged 66a63ad into ba733f5).

@travisbot

Copy link
Copy Markdown

This pull request fails (merged 6b95d75 into ba733f5).

@travisbot

Copy link
Copy Markdown

This pull request fails (merged 53f5036 into ba733f5).

@travisbot

Copy link
Copy Markdown

This pull request fails (merged c921287 into ba733f5).

@travisbot

Copy link
Copy Markdown

This pull request fails (merged 216dc41 into ba733f5).

@travisbot

Copy link
Copy Markdown

This pull request fails (merged 62132ac into ba733f5).

@brianc

brianc commented Aug 25, 2012

Copy link
Copy Markdown
Owner

hmm...would it be possible to squash your commits down into a single commit that passes the tests? Are you able to run the tests locally? If you need help running them I'll be glad to do what I can to get you up and running.

@soletan

soletan commented Aug 25, 2012

Copy link
Copy Markdown
Contributor Author

Sorry, for that, but all those commits aren't actually related to my
initial pull request regarding fixture of Unix socket support in
natively binding with pg.

I tried to reduce code on parsing responses then and obviously this
fixture is incomplete, yet. I'll have a look at it ... regarding
squashing commits I'm not much familiar with git, so any help would be
appreciated.

fixing issue on "this" properly passed into parseX-methods processing response messages depending on message type
@travisbot

Copy link
Copy Markdown

This pull request passes (merged 596f518 into ba733f5).

@soletan

soletan commented Aug 25, 2012

Copy link
Copy Markdown
Contributor Author

Sorry for committing to my fork w/o properly testing and while pull
request was in effect. Now the tests have passed.

@soletan

soletan commented Aug 25, 2012

Copy link
Copy Markdown
Contributor Author

BTW: Now I know my fault and won't do any further edits on master branch ever again. ;)

@brianc

brianc commented Aug 31, 2012

Copy link
Copy Markdown
Owner

No worries - travis was going crazy. I didn't know pull requests worked like that either.

I assume you're ready for me to test your patch & merge it now? 😄

If you could, just make a new branch with only the unix socket fixes related and then do a pull request off of that branch?

@brianc

brianc commented Sep 11, 2012

Copy link
Copy Markdown
Owner

The crazy train left this pull request's station long ago. I want to fix this. Circle back to me when you get the branching thang figured out.

@brianc brianc closed this Sep 11, 2012
@soletan

soletan commented Sep 14, 2012

Copy link
Copy Markdown
Contributor Author

I tried to figure out how to branch in git and on using github and
didn't quite succeed. Nevertheless the regarding patch is as easy as
making all that branching stuff oversized at all. So let's try doing it
the ancient way. The diff is available here:

soletan@6e0922a

I repeated my efforts to checkout your project using c9.io, branching
there, modifying the file according to that diff there, committing to
whatever repo and still don't know what to do then. push requests aren't
succeeding.

Best Regards,
Thomas Urban

toxA IT-Dienstleistungen - Thomas Urban
Walter-Friedländer-Straße 2
D-10249 Berlin

Design + Programmierung = DoppelPack
http://www.doppelpack-berlin.de

Am 11.09.2012 03:25, schrieb Brian C:

The crazy train left this pull request's station long ago. I want to fix
this. Circle back to me when you get the branching thang figured out.


Reply to this email directly or view it on GitHub
#179 (comment).

@matinm

matinm commented Jan 18, 2013

Copy link
Copy Markdown

@brianc , any updates here? Seems UNIX sockets still don't work with pg's native bindings.

@booo

booo commented Jan 18, 2013

Copy link
Copy Markdown
Contributor

I think we need a test for this... @matinm could you think of adding a test case?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants