Skip to content
This repository has been archived by the owner on May 17, 2022. It is now read-only.

Add C glib builders #154

Closed
kszucs opened this issue Aug 9, 2019 · 3 comments · Fixed by #172
Closed

Add C glib builders #154

kszucs opened this issue Aug 9, 2019 · 3 comments · Fixed by #172

Comments

@kszucs
Copy link
Contributor

kszucs commented Aug 9, 2019

Steps (these could be documented in projects/arrow/README.md):

  1. Add a new docker image with the required dependencies added, based on the cpp one.
  2. Build the image(s) via ursabot docker -n c_glib build (assuming that the image is named as c_glib).
  3. Add a builder based on the CppTest builder with c_glib specific steps and properties.
  4. Add it to arrow's master.cfg
  5. Configure a scheduler with the appropiate file change paths (cpp/* and c_glib/*)
  6. Add an entry for the builder in the .travis.yml to test it automatically.

@kou would you like to pick up this issue?

@kou
Copy link
Collaborator

kou commented Aug 9, 2019

OK.
I'll work on this.

@kszucs
Copy link
Contributor Author

kszucs commented Aug 9, 2019

Let me know if you've stuck with anything!

kou added a commit that referenced this issue Sep 2, 2019
@kou
Copy link
Collaborator

kou commented Sep 2, 2019

I've created #172.
I used c-glib than c_glib for image name because we already use - in cpp-benchmark.

kszucs pushed a commit that referenced this issue Sep 8, 2019
* Add support for C GLib

Fix #154

* flake8

* trigger c_glib builders on C++ changes as well
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants