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

Deal with color output from GCC #192

Open
troeger opened this issue Nov 30, 2017 · 4 comments
Open

Deal with color output from GCC #192

troeger opened this issue Nov 30, 2017 · 4 comments
Labels

Comments

@troeger
Copy link
Owner

troeger commented Nov 30, 2017

Some GCC version produce coloured output on errors. This currently messes up the rendering in the frontend with funny characters.

Switching it of by default (see https://gcc.gnu.org/onlinedocs/gcc-5.2.0/gcc/Language-Independent-Options.html) only works if the compiler is modern enough.

The options seem to be:

  • Detect the GCC version on executor configuration and set the compiler call defaults accordingly.
  • Allow adding compiler options in executor.cfg and let the admins deal with it.
  • Allow adding compiler options in the Job interface and let the tutors deal with it.
  • Parse color ASCII code in output and render it properly in the frontend.
@troeger troeger added this to the Next release milestone Nov 30, 2017
@troeger
Copy link
Owner Author

troeger commented Dec 18, 2017

@tzwenn @thehappyhippo @werner-matthias

Any opinions ?

@troeger troeger removed this from the Next release milestone Dec 18, 2017
@thehappyhippo
Copy link
Collaborator

Add an option for environment variables in the executor.cfg to handle GCC colors (GCC_COLORS) and other stuff. The validator has to handle with output etc. Submission result output format should be specified in sent result (text/html/markdown etc.) to enable rendering in the web ui. The validator could set a environment variable for rendering format for the executor, or add a tag in the first line of output?

@troeger
Copy link
Owner Author

troeger commented Jan 10, 2018

@werner-matthias votes for rendering them in the "opensubmit-exec test" and web page outputs.

@cforler
Copy link

cforler commented Oct 11, 2018

Easy idea. Just pipe the output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants