Page 1 of 1

Patch to QxOrm.pri

PostPosted: Wed Feb 17, 2016 9:58 am
by mdw
Include support for all MSVC compilers:

#############################
# Compiler / Linker Options #
#############################

win32 {
CONFIG(debug, debug|release) {
} else {
DEFINES += NDEBUG
win32-msvc*: QMAKE_LFLAGS += /OPT:NOREF
} # CONFIG(debug, debug|release)
win32-g++: QMAKE_LFLAGS += -Wl,-export-all-symbols -Wl,-enable-auto-import
} # win32

Re: Patch to QxOrm.pri

PostPosted: Wed Feb 17, 2016 4:54 pm
by qxorm
Hello,

Thank you for the patch : I will test it (especially with older versions of qmake).
If everything works fine, I will add it to the next version of QxOrm library.

Re: Patch to QxOrm.pri

PostPosted: Fri Feb 19, 2016 2:08 pm
by qxorm
I tested a bit the /OPT:NOREF linker parameter :
- it is necessary for older compiler (like MSVC 2008) ;
- it is not necessary and increases output binary size for nothing with newer compiler (like MSVC 2012, so I suppose this is also true for MSVC 2013 and MSVC 2015).

So I will not apply your patch, this part of QxOrm.pri will remain :
Code: Select all
win32 {
CONFIG(debug, debug|release) {
} else {
DEFINES += NDEBUG
win32-msvc2005: QMAKE_LFLAGS += /OPT:NOREF
win32-msvc2008: QMAKE_LFLAGS += /OPT:NOREF
win32-msvc2010: QMAKE_LFLAGS += /OPT:NOREF
} # CONFIG(debug, debug|release)
win32-g++: QMAKE_LFLAGS += -Wl,-export-all-symbols -Wl,-enable-auto-import
} # win32

Re: Patch to QxOrm.pri

PostPosted: Mon Feb 22, 2016 8:10 am
by mdw
Ah okay, so this was intentional.