New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Troubleshooting issue with travis #134

Closed
wants to merge 4 commits into
base: svn-2.3
from

Conversation

Projects
None yet
2 participants
@robe2
Member

robe2 commented Aug 10, 2017

No description provided.

robe2 added some commits Aug 10, 2017

@strk

This comment has been minimized.

Show comment
Hide comment
@strk

strk Aug 10, 2017

Member

There's your problem:

drwxr-x--- 33 travis travis 4096 Aug 10 08:27 travis

Basically, the PostgreSQL server user (postgres?) cannot access travis's home.
Try adding this step:

chmod 755 ~

I would not upgrade PostgreSQL in the same PR, better keep focused :)

Member

strk commented Aug 10, 2017

There's your problem:

drwxr-x--- 33 travis travis 4096 Aug 10 08:27 travis

Basically, the PostgreSQL server user (postgres?) cannot access travis's home.
Try adding this step:

chmod 755 ~

I would not upgrade PostgreSQL in the same PR, better keep focused :)

@strk

This comment has been minimized.

Show comment
Hide comment
@strk

strk Aug 10, 2017

Member

ERROR: could not access file "/home/travis/build/postgis/postgis/regress/00-regress-install/lib/postgis-2.3": Permission denied

You changed permissions too late, they are needed before make check (which is the pre-install check, thus needing to access travis directory).

Also, just changing /home/travis permission should be fine, no need to descend (as you're trying..)

Member

strk commented Aug 10, 2017

ERROR: could not access file "/home/travis/build/postgis/postgis/regress/00-regress-install/lib/postgis-2.3": Permission denied

You changed permissions too late, they are needed before make check (which is the pre-install check, thus needing to access travis directory).

Also, just changing /home/travis permission should be fine, no need to descend (as you're trying..)

@robe2

This comment has been minimized.

Show comment
Hide comment
@robe2

robe2 Aug 10, 2017

Member

@strk is it going to cause a problem when we mirror that I just inline edited and the patch I created is a branch of this repo. I guess we can test if you are not sure or I can delete this patch branch and pull request before I commit.

Member

robe2 commented Aug 10, 2017

@strk is it going to cause a problem when we mirror that I just inline edited and the patch I created is a branch of this repo. I guess we can test if you are not sure or I can delete this patch branch and pull request before I commit.

strk pushed a commit that referenced this pull request Aug 10, 2017

Fix travis build by changing permissions of home folder
upgrade to test on 9.5
References #3803 for PostGIS 2.3.4
Closes #134

git-svn-id: http://svn.osgeo.org/postgis/branches/2.3@15547 b70326c6-7e19-0410-871a-916f4a2858ee
@robe2

This comment has been minimized.

Show comment
Hide comment
@robe2

robe2 Aug 10, 2017

Member

Fixed at svn r3803.

Member

robe2 commented Aug 10, 2017

Fixed at svn r3803.

@robe2 robe2 closed this Aug 10, 2017

@strk

This comment has been minimized.

Show comment
Hide comment
@strk

strk Aug 10, 2017

Member

it looks like it didn't create a problem, but please delete the branch now that you don't need it anymore

Member

strk commented Aug 10, 2017

it looks like it didn't create a problem, but please delete the branch now that you don't need it anymore

@robe2 robe2 deleted the robe2-patch-1 branch Aug 10, 2017

pramsey pushed a commit to pramsey/postgis-gh that referenced this pull request Oct 16, 2017

Fix travis build by changing permissions of home folder
upgrade to test on 9.5
References #3803 for PostGIS 2.3.4
Closes postgis/postgis#134

git-svn-id: https://svn.osgeo.org/postgis/branches/2.3@15547 b70326c6-7e19-0410-871a-916f4a2858ee
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment