Opened 16 years ago

Closed 16 years ago

Last modified 15 years ago

#1214 closed bug (fixed)

Python not loading in Windows QGIS (build from source)

Reported by: smizuno Owned by: nobody
Priority: minor: annoyance Milestone:
Component: Build/Install Version: Trunk
Keywords: windows python Cc:
Must Fix for Release: No Platform: Windows
Platform Version: Awaiting user input: no

Description

For Windows platform, when I build from source the file libqgispython.dll is not loaded by QGIS, therefore there is no Python support.

I have found in src/app/qgisapp.cpp in line 3820 (file is r9003)

QString pythonlibName("qgispython");

If I make the string "libqgispython" then the .dll is loaded and Python support is available.

With Linux the name doesn't seem to matter.

So, for Windows the name should be "libqgispython" or the file should be built as qgispython.dll

Change History (5)

comment:1 by jef, 16 years ago

Resolution: fixed
Status: newclosed

MinGW issue fixed in r9030

comment:2 by smizuno, 16 years ago

Resolution: fixed
Status: closedreopened

I am reopening this ticket because r9030 builds the dll as liblibqgispython.dll, which won't be loaded unless it is renamed to qgispython.dll.

I neglected to mention that it was MinGW that I was building with, but you picked up on it. Thanks.

I believe the the file should be referenced from qgisapp.cpp by the name "libqgispython" rather than "qgispython". This is what I have been doing when I build for Windows and Mandriva Linux.

I don't have have any experience with MSVC building or Macs, so I'm not sure whether the name of the library and how it is referenced has any problems if the name is libqgispython in those situations.

in reply to:  2 ; comment:3 by jef, 16 years ago

Replying to smizuno:

I am reopening this ticket because r9030 builds the dll as liblibqgispython.dll, which won't be loaded unless it is renamed to qgispython.dll.

ouch.

I neglected to mention that it was MinGW that I was building with, but you picked up on it. Thanks.

r9044 hopefully fixes the problem correctly...

in reply to:  3 comment:4 by smizuno, 16 years ago

Resolution: fixed
Status: reopenedclosed

Replying to jef:

r9044 hopefully fixes the problem correctly...

Yes, the problem is fixed. Thank you.

I am closing the ticket.

comment:5 by (none), 15 years ago

Milestone: Version 1.0.0

Milestone Version 1.0.0 deleted

Note: See TracTickets for help on using tickets.