search

mate-desktop / mate-notification-daemon

Daemon to display passive pop-up notifications
https://mate-desktop.org
GNU General Public License v2.0
30 stars 26 forks source link

initial travis-CI support #142

Closed raveit65 closed 5 years ago

raveit65 commented 5 years ago

@monsta Are debian and ubuntu build requires the same? Which debian version provide docker images? Than i will add debian to config. I forked m-n-d an did a successful test. https://github.com/raveit65/mate-notification-daemon/pull/3

raveit65 commented 5 years ago

No luck with adding debian. https://travis-ci.org/raveit65/mate-notification-daemon/builds with that commit https://github.com/raveit65/mate-notification-daemon/commit/e8f344069c1b2895106dd22858579e261ad7df37 Looks like the distro name is wrong. DISTRO="debian:stretch"

raveit65 commented 5 years ago

Hmm, looks like debian needs to be add to docker-build script. https://github.com/mate-desktop/mate-dev-scripts/blob/9a6099ca324454028ea6767f1f196dd130065a0d/travis/docker-build#L127

yetist commented 5 years ago

Yes, if use debian the cmd is not setup.

raveit65 commented 5 years ago

Ok, than i will merge PR like it is and we need to add debian to docker-build script first.

raveit65 commented 5 years ago

\o/ it works. https://travis-ci.org/mate-desktop/mate-notification-daemon/builds/484804313