MantisBT - CMake
View Issue Details
0010766CMakeCMakepublic2010-05-26 22:322011-01-12 07:31
Yaakov (Cygwin Ports) 
Brad King 
normalminoralways
closedfixed 
CMake-2-8 
CMake 2.8.4CMake 2.8.4 
0010766: Utilities/Release/Cygwin: fix for libncurses10
With the switch to upstream ncurses "ABI 6", Cygwin's ncurses has YA ABI bump and is now libncurses10. However, the regex used in Utilities/Release/Cygwin/CMakeLists.txt to determine which libncurses is being used does not handle multiple-digit ABIs.

libncurses8 was the first version to be built with libtool and therefore contains a hyphen (cygncurses-8.dll). It was first introduced in 2004[1], so it should be sufficiently old to rely on. Furthermore, libncurses7 has a serious flaw in that it completely breaks if rebased.

Therefore IMO the easiest solution is to only look at the hyphened versions and change the regex accordingly. Patch attached.

[1] http://cygwin.com/ml/cygwin/2004-10/msg00674.html [^]
No tags attached.
patch 2.8.1-cygwin-libncurses.patch (730) 2010-05-26 22:32
https://public.kitware.com/Bug/file/3147/2.8.1-cygwin-libncurses.patch
Issue History
2010-05-26 22:32Yaakov (Cygwin Ports)New Issue
2010-05-26 22:32Yaakov (Cygwin Ports)File Added: 2.8.1-cygwin-libncurses.patch
2010-12-14 18:40David ColeAssigned To => Brad King
2010-12-14 18:40David ColeStatusnew => assigned
2010-12-15 09:01Brad KingNote Added: 0024143
2010-12-15 09:01Brad KingStatusassigned => closed
2010-12-15 09:01Brad KingResolutionopen => fixed
2011-01-12 07:31David ColeFixed in Version => CMake 2.8.4
2011-01-12 07:31David ColeTarget Version => CMake 2.8.4

Notes
(0024143)
Brad King   
2010-12-15 09:01   
Applied, thanks:
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=0f5c711f [^]