Patch because

You find a bug using QxOrm library and you know how to fix it : submit a patch on this forum

Patch because

Postby jduran » Fri May 18, 2012 1:31 pm

Hello,
Since boost version 1.4.0, the libraries are not including the -mt suffix to match system conventions. http://stackoverflow.com/questions/1768943/building-boost-on-linux-library-names. So, I post a patch to change the name of serialization library:

Code: Select all
--- QxOrm.pri.prev      2012-05-18 15:27:54.217021032 +0200
+++ QxOrm.pri   2012-05-18 15:29:09.247962796 +0200
@@ -51,8 +51,8 @@
 unix {^M
 QX_BOOST_INCLUDE_PATH = $$quote(/usr/include)^M
 QX_BOOST_LIB_PATH = $$quote(/usr/lib)^M
-QX_BOOST_LIB_SERIALIZATION_DEBUG = "boost_serialization-mt-d"^M
-QX_BOOST_LIB_SERIALIZATION_RELEASE = "boost_serialization-mt"^M
+QX_BOOST_LIB_SERIALIZATION_DEBUG = "boost_serialization-d"^M
+QX_BOOST_LIB_SERIALIZATION_RELEASE = "boost_serialization"^M
 } # unix^M
 ^M
 ##############################^M
@@ -111,3 +111,4 @@
 } else {^M
 LIBS += -l$${QX_BOOST_LIB_SERIALIZATION_RELEASE}^M
 } # CONFIG(debug, debug|release)^M
^M


Joaquim Duran
jduran
 

Re: Patch because

Postby QxOrm admin » Mon May 21, 2012 9:06 am

Hi,

I post a patch to change the name of serialization library

Thanx very much for your patch !
I think I will wait a little bit to change the default name of boost serialization library (I'm working with boost 1.38 on Linux)...
QxOrm admin
 

Re: Patch because

Postby jduran » Mon May 21, 2012 10:16 am

Hi,

It is possible to define a Qmake variable to set the name of the library from parameters? like BOOST_SERIAL_LIB....

Joaquim Duran
jduran
 

Re: Patch because

Postby QxOrm admin » Mon May 21, 2012 11:12 am

It is possible to define a Qmake variable to set the name of the library from parameters ?

Why do you want to do it ?
Just open QxOrm.pri file and change boost library path and names...
QxOrm admin
 

Re: Patch because

Postby jduran » Thu May 24, 2012 10:37 am

Why do you want to do it ?


You get flexibility only passing parameters to qmake and it is not needed to edit files. For example:

Code: Select all
qmake PREFIX=/usr/local/qxorm QMAKE_INCDIR+=/usr/local/boost_1.49/boost QMAKE_LIBDIR+=/usr/local/boost_1.49/lib BOOST_SERIAL_LIB=boost_serialization BOOST_SERIAL_LIB_DEBUG=boost_serialization
jduran
 

Re: Patch because

Postby QxOrm admin » Fri May 25, 2012 7:31 am

Ok, it will be done for next version QxOrm 1.2.5...
I think I will prefix all variables with QX_, for example : QX_PREFIX, etc...
QxOrm admin
 

Re: Patch because

Postby qxorm » Tue Dec 11, 2012 1:13 pm

Done, you can test it with the following BETA version :
http://www.qxorm.com/version/QxOrm_1.2.5_BETA_04.zip
qxorm
Site Admin
 
Posts: 481
Joined: Mon Apr 12, 2010 7:45 am


Return to QxOrm - Submit a patch

Who is online

Users browsing this forum: No registered users and 2 guests

cron