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

log stderr of make to a file and use tail to show last 1000 lines #601

Merged
merged 1 commit into from
Jun 9, 2018

Conversation

eine
Copy link
Collaborator

@eine eine commented Jun 8, 2018

When GHDL is built with GCC backend, and the target version of GCC is not the same as the version of the GCC used in the build, the process logs so heavily that the 4MB in travis-ci is reached. This results in the build being broken (see last two lines in https://api.travis-ci.com/v3/job/127969859/log.txt). Moreover, if all the make output is saved to a file, the build fails because no output is received in 10 minutes and travis thinks that it is stalled.

This PR logs stderr to a file (not stdout). When make is done, the last 1000 lines of the file are shown. This allows the build to finish successfully.

@eine eine mentioned this pull request Jun 8, 2018
@Paebbels Paebbels self-requested a review June 9, 2018 05:44
@Paebbels Paebbels self-assigned this Jun 9, 2018
@Paebbels Paebbels added CI: Travis-CI Continuous Integration issues related to the Travis-CI platform (Linux and MacOS-based CI) Backend: GCC GHDL using GCC toolchain for code generation labels Jun 9, 2018
@Paebbels Paebbels added this to the v0.36 milestone Jun 9, 2018
@Paebbels Paebbels merged commit d69ff3e into ghdl:master Jun 9, 2018
@eine eine deleted the fix-make-use-tail branch June 9, 2018 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend: GCC GHDL using GCC toolchain for code generation CI: Travis-CI Continuous Integration issues related to the Travis-CI platform (Linux and MacOS-based CI)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants