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

Produce artifacts from CI with Mapnik XML #3012

Open
pnorman opened this issue Jan 10, 2018 · 9 comments
Open

Produce artifacts from CI with Mapnik XML #3012

pnorman opened this issue Jan 10, 2018 · 9 comments
Labels
Milestone

Comments

@pnorman
Copy link
Collaborator

pnorman commented Jan 10, 2018

cc @lonvia

With the increasing problems of getting npm and carto, some have asked about having artifacts generated from CI with the Mapnik XML

@kocio-pl
Copy link
Collaborator

Could you be more verbose, please? I don't get what's this ticket about and what kind of problems are you mentioning.

@pnorman
Copy link
Collaborator Author

pnorman commented Jan 10, 2018

Could you be more verbose, please? I don't get what's this ticket about and what kind of problems are you mentioning.

Requiring a npm install is a reasonably high barrier if you only want to use the style, not develop it. If all someone needs is the XML, they should be able to get that as an artifact.

@kocio-pl kocio-pl added the code label Jan 10, 2018
@kocio-pl kocio-pl added this to the New features milestone Jan 10, 2018
@lonvia
Copy link

lonvia commented Jan 10, 2018

I was the one asking about artifacts, so let me elaborate. npm does not ship as a package with Debian anymore. The manual installation instructions for npm ask you to pipe a script from the internets into a sudo bash, which is a no-no for obvious reasons. In the end I settled for nvm to set up a current nodejs and npm without requiring root. It's still quite a bit of work, so precompiled XML for those of us who just want to render would be awesome.

I've looked a bit into what Travis can do. Unfortunately, it does not host artifacts itself like appveyor does. So you need to provide some hosting. The most promising might be the uploading to Github releases. Having xml for the releases would be more than enough for non-devs.

@kocio-pl
Copy link
Collaborator

Any ideas about how it should be done? Probably some simple script to be launched before every release or just a one liner in RELEASES.md?

@HolgerJeromin
Copy link
Contributor

Another way would be letting travis checkin the results to an repository. Not kidding

https://github.com/open62541/open62541/blob/master/tools/travis/travis_push_release.sh
produces https://open62541.org/releases/

@kocio-pl
Copy link
Collaborator

Sounds great to me. Anybody willing to prepare the code?

@HolgerJeromin
Copy link
Contributor

Its kind of hacking github for serving files without the need of the full history.
The downside are the massive checkins which floods the history of the repository. Thats why it seems a good idea to use another repository and reset it from time to time:
open62541/open62541#108

@pnorman
Copy link
Collaborator Author

pnorman commented Jan 10, 2018

👎 on having travis push to a git repo.

We could either have Travis push to github releases, another artifact location, or look at something like circleci which has artifact support.

@matkoniecz
Copy link
Contributor

matkoniecz commented Jan 10, 2018

I am against giving push access to external entities.

Separate repo would be acceptable for me or something similar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants