MantisBT - CDash
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0012368CDash(No Category)public2011-07-28 12:312013-08-30 05:49
ReporterSean McBride 
Assigned ToJulien Jomier 
PriorityhighSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version1.8 
Target VersionFixed in Version2.2 
Summary0012368: CDash's build results html should use monospace font for 'standard error' output
DescriptionTake a look at:

http://www.cdash.org/CDash/viewBuildError.php?buildid=1373522 [^]

in the 'standard error' output, we see:

/Users/builder/kitware/ITK/Modules/Numerics/FEM/include/itkFEMImageMetricLoad.hxx:167:7: error: 'itk::fem::FEMP<itk::fem::Element>' is not a pointer
      dynamic_cast<Element *>( ( *elt ) )->GetIntegrationPointAndWeight(i, ip, w, m_NumberOfIntegrationPoints);
      ^ ~~~~~~~~

the clang compiler outputs nice error messages, but assumes you are using a monospace font. Those ~ characters are meant to underline the "*elt", which is not what one sees with CDash 1.8.3's rendering. See attached screenshot.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filespng Screen Shot 2011-07-28 at 12.29.26.png (223,534) 2011-07-28 12:31
https://public.kitware.com/Bug/file/3987/Screen%20Shot%202011-07-28%20at%2012.29.26.png
png

jpg Win64-Safari.5.1.5.jpg (382,885) 2012-04-04 12:16
https://public.kitware.com/Bug/file/4286/Win64-Safari.5.1.5.jpg
Issue History
Date ModifiedUsernameFieldChange
2011-07-28 12:31Sean McBrideNew Issue
2011-07-28 12:31Sean McBrideFile Added: Screen Shot 2011-07-28 at 12.29.26.png
2011-07-28 13:42David ColeAssigned To => David Cole
2011-07-28 13:42David ColeStatusnew => assigned
2011-07-28 13:43David ColeNote Added: 0027098
2011-07-28 17:04Sean McBrideNote Added: 0027101
2012-01-17 03:30Julien JomierAssigned ToDavid Cole => Julien Jomier
2012-01-17 04:55Julien JomierNote Added: 0028327
2012-01-17 04:55Julien JomierStatusassigned => resolved
2012-01-17 04:55Julien JomierFixed in Version => 2.0
2012-01-17 04:55Julien JomierResolutionopen => fixed
2012-03-29 11:53Sean McBrideNote Added: 0028997
2012-03-29 11:53Sean McBrideStatusresolved => feedback
2012-03-29 11:53Sean McBrideResolutionfixed => reopened
2012-04-01 11:15Julien JomierNote Added: 0029028
2012-04-02 10:07Sean McBrideNote Added: 0029036
2012-04-02 10:07Sean McBrideStatusfeedback => assigned
2012-04-02 10:14David ColeNote Added: 0029055
2012-04-02 10:19Sean McBrideNote Added: 0029056
2012-04-04 09:08Julien JomierNote Added: 0029076
2012-04-04 11:08Sean McBrideNote Added: 0029077
2012-04-04 12:16Julien JomierFile Added: Win64-Safari.5.1.5.jpg
2012-04-04 12:16Julien JomierNote Added: 0029078
2012-04-04 13:14Sean McBrideNote Added: 0029079
2013-03-01 11:08Sean McBrideNote Added: 0032426
2013-08-29 13:06Sean McBrideNote Added: 0033762
2013-08-30 05:49Julien JomierNote Added: 0033763
2013-08-30 05:49Julien JomierStatusassigned => resolved
2013-08-30 05:49Julien JomierFixed in Version2.0 => 2.2
2013-08-30 05:49Julien JomierResolutionreopened => fixed

Notes
(0027098)
David Cole   
2011-07-28 13:43   
Good idea. (I thought it was that way already actually.......)

Should be fairly easy to address...
(0027101)
Sean McBride   
2011-07-28 17:04   
Great! Looking forward to ITK's dashboard being updated...
(0028327)
Julien Jomier   
2012-01-17 04:55   
Just tested with CDash 2.0 and it seems to show spaces correctly. Otherwise feel free to reopen.
(0028997)
Sean McBride   
2012-03-29 11:53   
Doesn't seem to be fixed... See for example:

http://open.cdash.org/viewBuildError.php?buildid=2133734 [^]
(0029028)
Julien Jomier   
2012-04-01 11:15   
Can you point to an existing build?
(0029036)
Sean McBride   
2012-04-02 10:07   
hmmm, why did that link go dead? Anyway, from today's VTK:

http://open.cdash.org/viewBuildError.php?type=1&buildid=2146444 [^]
(0029055)
David Cole   
2012-04-02 10:14   
By the way, this is a problem when viewed from today's up-to-date Safari on a Mac or an iPad. (On my Mac, Lion 10.7.3, Safari 5.1.5. On my iPad, iOS 5.1.)
(0029056)
Sean McBride   
2012-04-02 10:19   
Indeed, it renders correctly for me in FireFox, but not Safari.
(0029076)
Julien Jomier   
2012-04-04 09:08   
It works for me with Safari 5.1.5 on Windows...
(0029077)
Sean McBride   
2012-04-04 11:08   
Julien, can you attach a screenshot? It's sounding like a Safari bug... I can file a bug with Apple, but your screenshot would be a convincing argument I think!
(0029078)
Julien Jomier   
2012-04-04 12:16   
I just uploaded the screenshot. Thanks!
(0029079)
Sean McBride   
2012-04-04 13:14   
I filed rdar://11183482 with Apple.
(0032426)
Sean McBride   
2013-03-01 11:08   
I finally heard back from Apple yesterday:

---------------------
This is a follow-up regarding regarding Bug ID# 11183482.

Engineering has determined that this issue behaves as intended based on the following information:

On OS X, textareas use a proportional font by default.

The author of the website can specify a monospace font for the text areas if they want to.

If this is still an issue, or you have questions regarding the resolution of this issue, please update your bug report with that information.

We are now closing this bug report.
---------------------
(0033762)
Sean McBride   
2013-08-29 13:06   
Ping. :)

I'm no web expert, but Apple seems to be saying that Safari's behaviour is correct, do you concur?

Even if not, can you specify a monospace font as they suggest?

I should also add that gcc 4.8 now does as clang does, and outputs ASCII art arrows that expect monospaced fonts, see -fdiagnostics-show-caret here:
http://gcc.gnu.org/gcc-4.8/changes.html [^]

This bug is easily my #1 issue with cdash.
(0033763)
Julien Jomier   
2013-08-30 05:49   
That was a tricky one. Basically for textarea we need to specify the font-family to be one of the monospace font. By default Safari on Mac (not Windows) doesn't use monopsace font by default.

textarea {
  font-size: 14px;
  font-family: Courier New, Consolas,Monaco,Lucida Console,Liberation Mono,DejaVu Sans Mono,Bitstream Vera Sans Mono, monospace;
}