Skip to content
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

Easy Installation method is out of date #30

Closed
dwsilk opened this issue Jun 16, 2017 · 19 comments
Closed

Easy Installation method is out of date #30

dwsilk opened this issue Jun 16, 2017 · 19 comments

Comments

@dwsilk
Copy link
Member

dwsilk commented Jun 16, 2017

The Easy Installation method documented in the README installs 1.1.1, latest version is 1.2.0. Not sure if these instructions should be removed, or pgxn should be updated?

@strk
Copy link
Contributor

strk commented Aug 21, 2017

I guess we want to upload a new version.
@palmerj I registered a PGXN account strk if you want to delegate this part.

@strk
Copy link
Contributor

strk commented Sep 20, 2017

Current plan is that instructions need be updated to drop PGXN and add apt-get instructions to fetch from public PPA repo. \cc @imincik

@strk
Copy link
Contributor

strk commented Sep 21, 2017

@imincik I'm thinking that PGXN support more distribution than the LINZ PPA, as it seems to be building the package locally. So it might still be worth keeping, as long as we can handle to maintain it.

What I understand is that each extension has a single maintainer on PGXN so the only way to get this moved would be for @palmerj to delegate maintainance (to an "abstract" user?)

@imincik
Copy link
Contributor

imincik commented Sep 21, 2017

@imincik I'm thinking that PGXN support more distribution than the LINZ PPA

I am not sure if we have enough force to maintain and support PGXN packages.

@strk
Copy link
Contributor

strk commented Sep 25, 2017

Then I'd drop the Easy Installation method section.
Please confirm you're fine with it: #55 and #56

@imincik
Copy link
Contributor

imincik commented Sep 25, 2017

Today we discussed with @strk whether we should keep packages on PGXN or not.

Pros:

  • support for other OS (Windows, Mac)

Cons:

  • we don't have resources, test and dev environments and maybe even skills to support PostgreSQL on other OS than Linux
  • for Debian/Ubuntu systems, installation even from self built DEB is better option

@palmerj , @dwsilk , @strk any thoughts ?

@dwsilk
Copy link
Member Author

dwsilk commented Sep 25, 2017

Dropping support for PGXN doesn't bother me.

@palmerj
Copy link
Contributor

palmerj commented Sep 25, 2017

I more in favour of keeping in on PGXN, but agree the LINZ benefits are zero right now. Two pro reasons:

  • access for other LINUX distros
  • awareness that the extension exists for others PG users by putting it in a centralised index and website. I think the SEO for finding the extension by google is not enough.

I don't think the resources to publish are that great. On release just need to run one archive command and then upload to http://manager.pgxn.org.

My 2cents

@imincik
Copy link
Contributor

imincik commented Sep 26, 2017

@palmerj ,

My biggest concerns are that we are probably not able to test and support PGXN packages on other platforms right now.

For publishing to PGXN, we need access to your PGXN account or find some other workaround.

@palmerj
Copy link
Contributor

palmerj commented Sep 26, 2017

My biggest concerns are that we are probably not able to test and support PGXN packages on other platforms right now.

My experience is that support is little for *unix and auto builds and report from failure are done by the PXN people. Maybe just have a policy of saying sorry we can't help to Windows users? Of course your call.

@strk
Copy link
Contributor

strk commented Sep 26, 2017 via email

@imincik
Copy link
Contributor

imincik commented Oct 3, 2017

My current opinion is that we definitely should not drop PGXN until we have proper Debian packages support at least for all supported Ubuntu LTS >= 14.04 (16.04 is missing now) and all supported PostgreSQL versions from apt.postgresql.org (this is missing as well because of Launchpad limits, we need to change whole DEB building pipeline to build packages for multiple PostgreSQL versions) in our Debian repository.

So keep PGXN for now.

@strk
Copy link
Contributor

strk commented Oct 8, 2017

It looks like we don't need a publish the code for PGXN to grab it:
https://lists.osgeo.org/pipermail/postgis-devel/2017-October/026621.html

The only difference from publishing on PGXN.net is we need full URL and a known tag (if we want a non-development version)

@palmerj
Copy link
Contributor

palmerj commented Oct 8, 2017

Ok thanks interesting and good for testing the meta.json file. However one of the main reasons in my mind is to add the extension to PGXN for good google SEO. Type "table versioning postgresql" into google and see what the top hit is :)

I wonder if it's possible to configure the PGXN account to point to a github branch rather than having to upload a release tarball.

@strk
Copy link
Contributor

strk commented Oct 8, 2017 via email

@strk
Copy link
Contributor

strk commented Oct 9, 2017

Now, in order to be able to fetch from repository we'd need a non-generated META.json, while we currently generate it at make time. If we plan to instruct users to PGXN directly from github, we should fix that too.

@strk
Copy link
Contributor

strk commented Oct 9, 2017

I was wrong with the above, pgxnclient 1.2.1 can install from github repo and no META.json just fine, but older versions cannot install from http at all, looks like: https://travis-ci.org/linz/linz-lds-bde-schema/jobs/285601161#L731-L733

@strk
Copy link
Contributor

strk commented Oct 9, 2017

I'm taking it back, I've no evidence of old versions not working. Adding http based pgxn install in travis configuration for linz-lds-bde-schema here: linz/linz-lds-bde-schema#49

@strk
Copy link
Contributor

strk commented Nov 3, 2017

table_version 1.3.1 is now available on PGXN, so closin this. https://pgxn.org/dist/table_version/

@strk strk closed this as completed Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants