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

Improve buildrpms.sh for nightly builds #5333

Merged
merged 3 commits into from
Oct 16, 2017
Merged

Improve buildrpms.sh for nightly builds #5333

merged 3 commits into from
Oct 16, 2017

Conversation

daniviga
Copy link
Contributor

@daniviga daniviga commented Oct 9, 2017

cc @m-kuhn

Description

Improve a bit the buildrpms.sh. Now with the -u flag it is possible to generate packages with a version based on the git commit id (short). Priority of the package is managed via Epoch (latest build is always considered newer when in 'unstable' mode).

This allows nightly builds to be created on copr. The workflow is:

  • GIT is cloned and master checked out
  • SRPM is generated via buildrpms.sh -u -s
  • SRPM is uploaded to COPR via copr-cli
  • TODO: installation is tested in an ephemeral LXD container

CI build is available here: https://ci.services.vigano.me/#/builders/1/builds/6
Results will be (hopefully) published here: https://copr.fedorainfracloud.org/coprs/dani/qgis-testing/build/621301/

Checklist

Reviewing is a process done by project maintainers, mostly on a volunteer basis. We try to keep the overhead as small as possible and appreciate if you help us to do so by completing the following items. Feel free to ask in a comment if you have troubles with any of them.

  • Commit messages are descriptive and explain the rationale for changes
  • Commits which fix bugs include fixes #11111 in the commit message next to the description
  • Commits which add new features are tagged with [FEATURE] in the commit message
  • Commits which change the UI or existing user workflows are tagged with [needs-docs] in the commit message and containt sufficient information in the commit message to be documented
  • I have read the QGIS Coding Standards and this PR complies with them
  • This PR passes all existing unit tests (test results will be reported by travis-ci after opening this PR)
  • New unit tests have been added for core changes
  • I have run the scripts/prepare-commit.sh script before each commit

@daniviga daniviga changed the title Better rpm Improve buildrpms.sh for nightly builds Oct 9, 2017
@m-kuhn m-kuhn merged commit afb2a6a into qgis:master Oct 16, 2017
@m-kuhn
Copy link
Member

m-kuhn commented Oct 16, 2017

Great work @daniviga
Once the copr autobuilds are in place, can we add instructions for these to qgis homepage? And would it be possible / would you be willing to have this on a qgis Fedora account?

@daniviga
Copy link
Contributor Author

Once the copr autobuilds are in place, can we add instructions for these to qgis homepage? And would it be possible / would you be willing to have this on a qgis Fedora account?

Hi @m-kuhn, sorry for the late response, I didn't have much time to work on personal projects.
I almost completed the work to have a CD/CI (at least, checking that the package can be installed) via buildbot + copr. Now I'm going to setup a daily or weekly builds for it. Unfortunately COPR has been very slow in these last days with a long looooong build queue, so that slowed down a bit the work.

I would be very happy to have instruction on how to use these builds on the the QGIS website. About using a fedora account I will try, hopefully this weekend, to get in touch with the Fedora maintainers to understand how to proceed.

@daniviga daniviga deleted the better-rpm branch March 12, 2018 05:49
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.

None yet

2 participants