MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0010068CMakeModulespublic2009-12-18 19:002011-05-02 14:45
ReporterPeter Collingbourne 
Assigned ToMarcus D. Hanwell 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake-2-8 
Target VersionFixed in VersionCMake 2.8.3 
Summary0010068: Use Python naming convention for extension modules created using PYTHON_ADD_MODULE
DescriptionIt would be nice if Python extension modules declared using PYTHON_ADD_MODULE and built as a shared object were named using the correct naming conventions (i.e. no lib prefix, .pyd suffix on Windows) so that Python can find them automatically. I am attaching a patch which modifies PYTHON_ADD_MODULE to set an appropriate PREFIX and SUFFIX based on the platform.

This has been tested on Windows and Linux, I am unaware of the conventions on other platforms.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch 0002-Use-Python-naming-convention-for-extension-modules-c.patch (1,213) 2009-12-18 19:01
https://public.kitware.com/Bug/file/2731/0002-Use-Python-naming-convention-for-extension-modules-c.patch
Issue History
Date ModifiedUsernameFieldChange
2009-12-18 19:00Peter CollingbourneNew Issue
2009-12-18 19:01Peter CollingbourneFile Added: 0002-Use-Python-naming-convention-for-extension-modules-c.patch
2010-12-14 17:08David ColeAssigned To => Marcus D. Hanwell
2010-12-14 17:08David ColeStatusnew => assigned
2011-01-10 13:12Marcus D. HanwellNote Added: 0024540
2011-01-10 13:12Marcus D. HanwellStatusassigned => resolved
2011-01-10 13:12Marcus D. HanwellFixed in Version => CMake 2.8.3
2011-01-10 13:12Marcus D. HanwellResolutionopen => fixed
2011-05-02 14:45David ColeNote Added: 0026328
2011-05-02 14:45David ColeStatusresolved => closed

Notes
(0024540)
Marcus D. Hanwell   
2011-01-10 13:12   
This was fixed in CMake 2.8.3.
(0026328)
David Cole   
2011-05-02 14:45   
Closing resolved issues that have not been updated in more than 3 months.