-
Notifications
You must be signed in to change notification settings - Fork 79
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
Fix test errors #136
Fix test errors #136
Conversation
Signed-off-by: Hajime Tazaki <thehajime@gmail.com>
Signed-off-by: Hajime Tazaki <thehajime@gmail.com>
I sent you an invite so that you can merge your fixes. If you're not sure how to deal with something, file an issue, assign it to the maintainer, an in case the maintainer is nowhere to be found in a few days, disable the package. |
The compiler is too old on Travis. I will remove the two packages ( I will
put back them later ).
Thanks
On Wed, 22 Mar 2017 at 12:55, Antti Kantee ***@***.***> wrote:
I sent you an invite so that you can merge your fixes.
If you're not sure how to deal with something, file an issue, assign it to
the maintainer, an in case the maintainer is nowhere to be found in a few
days, disable the package.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#136 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFyMixbcKOApN2InLj1OoN3UuV1TpWf5ks5roQwygaJpZM4Mjz1n>
.
--
Sent from iPad
|
On 22/03/17 11:59, kvart2006 wrote:
The compiler is too old on Travis. I will remove the two packages ( I will
put back them later ).
What is "too old"? Can you document the minimum required compiler in
the README? If it's something that supports -std=c++17, it needs to
explicitly stated as a requirement.
Also, please don't remove the packages, just disable them from the
automated tests, pending fixes to the Travis env.
|
Ok, I will disable them.
I am also thinking about a system that works like Travis but for
Rumpkernel.
Travis is using Docker containers and Ubuntu 12.04. A bit redondant in this
context.
That platform could be more Xen oriented etc.
On Wed, 22 Mar 2017 at 13:03, Antti Kantee ***@***.***> wrote:
On 22/03/17 11:59, kvart2006 wrote:
> The compiler is too old on Travis. I will remove the two packages ( I
will
> put back them later ).
What is "too old"? Can you document the minimum required compiler in
the README? If it's something that supports -std=c++17, it needs to
explicitly stated as a requirement.
Also, please don't remove the packages, just disable them from the
automated tests, pending fixes to the Travis env.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#136 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFyMiznnskLLgPbvoGqQtKlfxTXtewifks5roQ4YgaJpZM4Mjz1n>
.
--
Sent from iPad
|
@kvart2006 for your information, travis can ignore test errors by marking |
@anttikantee thanks. I will merge this PR soon. |
Two fixes for rumprun-package test on travis-ci. - update URL for apache2 related tarballs. - install genisoimage package
Two fixes for rumprun-package test on travis-ci.
The test still fails with packages (servus and zerobuf), which I'm not sure how to fix it.