MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0009050CMakeCMakepublic2009-05-19 07:262013-03-04 08:38
ReporterDavid Cole 
Assigned ToDavid Cole 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in VersionCMake-2-8 
Summary0009050: SystemInformation reported incorrectly with dashboards run from crontab entry on Mac OSX
DescriptionIn SystemInformation.cxx, we should use the full path names (a la find_program) for the binaries executed to gather the information. The PATH is different when running from a crontab entry...
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filesdiff SystemInformation.cxx.diff (56,291) 2009-05-19 11:32
https://public.kitware.com/Bug/file/2261/SystemInformation.cxx.diff
diff SystemInformation-1.cxx.diff (10,747) 2009-05-19 12:13
https://public.kitware.com/Bug/file/2262/SystemInformation-1.cxx.diff
Issue History
Date ModifiedUsernameFieldChange
2009-05-19 07:26David ColeNew Issue
2009-05-19 07:26David ColeStatusnew => assigned
2009-05-19 07:26David ColeAssigned To => David Cole
2009-05-19 11:32Mike JacksonFile Added: SystemInformation.cxx.diff
2009-05-19 11:34Mike JacksonNote Added: 0016530
2009-05-19 11:44Bill HoffmanNote Added: 0016531
2009-05-19 11:54Mike JacksonNote Added: 0016532
2009-05-19 12:13Mike JacksonFile Added: SystemInformation-1.cxx.diff
2009-05-19 12:14Mike JacksonNote Added: 0016533
2012-08-13 15:06David ColeStatusassigned => new
2012-08-13 15:06David ColeNote Added: 0030593
2012-08-13 15:07David ColeStatusnew => backlog
2012-08-13 15:22David ColeAssigned ToDavid Cole =>
2012-09-09 08:11Rolf Eike BeerNote Added: 0030964
2012-09-09 16:48David ColeNote Added: 0030972
2012-09-09 16:48David ColeStatusbacklog => resolved
2012-09-09 16:48David ColeFixed in Version => CMake-2-8
2012-09-09 16:48David ColeResolutionopen => fixed
2012-09-09 16:48David ColeAssigned To => David Cole
2013-03-04 08:38Robert MaynardNote Added: 0032442
2013-03-04 08:38Robert MaynardStatusresolved => closed

Notes
(0016530)
Mike Jackson   
2009-05-19 11:34   
I rewrote most of the method that gets the information on OS X using the supplied system calls that are included with OS X. Seems to work fine on 10.4 and 10.5 machines. There may be an issue with a 10.3 machine but I don't have any way of testing it to find out.

Please test the supplied patch and let me know if there are any issues.
(0016531)
Bill Hoffman   
2009-05-19 11:44   
The patch seems to change lots of lines that are not part of the patch. Seems like a bunch of white space changes, do you think you could fix that and have a patch that only touches the code that was actually changed?
(0016532)
Mike Jackson   
2009-05-19 11:54   
Huh, I thought I fixed all that. I'll try again.
(0016533)
Mike Jackson   
2009-05-19 12:14   
New Patch Attached.
(0030593)
David Cole   
2012-08-13 15:06   
Sending old, not-recently-updated issues to the backlog.

(The age of the bug alone means that nobody is actively working on it...)

If an issue you care about is sent to the backlog when you feel it should have been addressed in a different manner, please bring it up on the CMake mailing list for discussion. Sign up for the mailing list here, if you're not already on it: http://www.cmake.org/mailman/listinfo/cmake [^]

It's easy to re-activate a bug here if you can find a CMake developer who has the bandwidth to take it on, and ferry a fix through to our 'next' branch for dashboard testing.
(0030964)
Rolf Eike Beer   
2012-09-09 08:11   
It looks like this has been merged in 5634ca02f4457a8c95ba43a78cf968894e0d8ae5 which is part of 2.8.0 and all following versions.
(0030972)
David Cole   
2012-09-09 16:48   
The patch does appear to have been applied: it's in 'master' and 'next', and I trust Eike's evaluation that it's been in there since 2.8.0

Marking resolved as 'fixed'

(There are still some calls in the SystemInformation.cxx file to RunProcess that take program names without full paths. I guess if nobody's complaining about the remaining problems, then there aren't really any problems remaining...)

Thanks, Eike, for pointing this out.
(0032442)
Robert Maynard   
2013-03-04 08:38   
Closing resolved issues that have not been updated in more than 4 months.