MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0015206CMakeCMakepublic2014-10-14 02:282015-04-06 09:07
ReporterLorenz Witte 
Assigned ToBrad King 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSWindowsOS Version
Product VersionCMake 3.0.2 
Target VersionCMake 3.2Fixed in VersionCMake 3.2 
Summary0015206: add_jar() in UseJava.cmake uses wrong classpath seperator when cross-compiling under Windows
DescriptionThe condition to use ";" as classpath separator includes a check for the switch "WIN32" which is a target switch. When cross-compiling for a non-windows target, this switch is not present and the separator defaults to ":".

I think it should rather check for "CMAKE_HOST_WIN32" instead.
Steps To ReproduceCall add_jar() during a cross-compilation on a Windows host.
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch 0001-fixed-classpath-separator-on-WIN32-cross-compilation.patch (891) 2014-10-14 02:28
https://public.kitware.com/Bug/file/5274/0001-fixed-classpath-separator-on-WIN32-cross-compilation.patch
Issue History
Date ModifiedUsernameFieldChange
2014-10-14 02:28Lorenz WitteNew Issue
2014-10-14 02:28Lorenz WitteFile Added: 0001-fixed-classpath-separator-on-WIN32-cross-compilation.patch
2014-10-14 10:02Brad KingNote Added: 0037030
2014-10-14 10:03Brad KingAssigned To => Brad King
2014-10-14 10:03Brad KingStatusnew => resolved
2014-10-14 10:03Brad KingResolutionopen => fixed
2014-10-14 10:03Brad KingFixed in Version => CMake 3.2
2014-10-14 10:03Brad KingTarget Version => CMake 3.2
2015-04-06 09:07Robert MaynardNote Added: 0038423
2015-04-06 09:07Robert MaynardStatusresolved => closed

Notes
(0037030)
Brad King   
2014-10-14 10:02   
Applied, thanks:

 UseJava: Fix classpath separator on WIN32 cross compilation
 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7a373d5e [^]
(0038423)
Robert Maynard   
2015-04-06 09:07   
Closing resolved issues that have not been updated in more than 4 months.