Patch for dbus-c++ win32 porting
Ralf Habacker
ralf.habacker at freenet.de
Mon May 31 22:24:46 CEST 2010
Am 30.05.2010 00:59, schrieb Mat:
> Hi
>
> I add cmake support for dbus-c++ in these days. It can now use
> original git source
> and build binary/library without autotools.
>
> Would you like to review the patches or give me some advice/suggestion?
>
Many thanks, workes good expect two little issues in the cmake stuff
1. I was forced to add CMAKE_INSTALL_PATH base pathes to the find_...
statements
2. in the link_directories statement DBUS_LIBS has to be removed because
it contains not only pathes
I added these files to emerge with two little fixes to
http://websvn.kde.org/trunk/kdesupport/emerge/portage/win32libs-sources/dbus-c%2B%2B-src/
I suggest to get in contact with dbus-c++ maintainer to get these
patches upstream.
Regards
Ralf
> sincerely, Mat.
>
>
> On Thu, May 20, 2010 at 12:53 AM, Mat<matlinuxer2 at gmail.com> wrote:
>
>> On Tue, May 18, 2010 at 4:00 PM, Ralf Habacker<ralf.habacker at freenet.de> wrote:
>>
>>> Mat schrieb:
>>>
>>>> Hi All,
>>>> I use dbus-c++ library in my programs and found that it's great for
>>>> c++ based projects.
>>>>
>>>> I tried to port dbus-c++ to win32, and have some progress these days.
>>>> The libdbusc++ can now work on my winxp :-)
>>>> ( for some simple examples, not yet try some real program, like
>>>> inkboard,...etc )
>>>>
>>>> Patches are as attachment:
>>>> dbus-c++-src-commit.patch is for import into emerge dir.
>>>>
>>>>
>>> +++ portage/win32libs-sources/dbus-cpp-src/dbus-cpp-src-0.5.0.py
>>> + self.targets[ver] =
>>> 'http://people.debian.org.tw/~mat/kde-windows/libdbus-c++-%s.tar.gz' % ver
>>>
>>> Is this source the same as from
>>> https://sourceforge.net/projects/dbus-cplusplus/ ?
>>>
>> Yes. I just updated it from dbus-cplusplus git. It's the same source
>> to dbus-cpluscplus
>>
>>
>>> Because this package is mingw only it should print a related hint and
>>> break on msvc
>>>
>> Good, I just added it.
>>
>>
>>> +++ portage/testing/pthreads-win32/pthreads-win32-2.7.0.py
>>> added to emerge
>>>
>>> +++ portage/testing/pkg-config/pkg-config-0.23-3.py
>>> added fixes to emerge
>>>
>>> +++ portage/testing/glib/glib-2.24.0-2.py
>>> added fixes to emerge
>>>
>>>
>>>> dbus-1.pc is for c:\kderoot\lib\pkgconfig\
>>>>
>>>>
>>> this should also be in the patch
>>>
>> Where do you suggestion to attach the patch? ( dbus-src/ pkg-config/
>> dbus-cpp-src )
>>
>>
>>>> works_for_win32.patch is a patch for dbus-c++-src[1] source code
>>>>
>>>>
>>> is this patch included in the above mentioned link ? It should not
>>>
>>> If patches are not in the archive file it should be added to def
>>> setTargets( self ): by
>>>
>>> self.patchToApply[v] = ( 'xxxx.patch',<number of stripped
>>> leading components from file names> )
>>>
>>> where v is the build target.
>>>
>> Good, I have updated to use this method in new file. Thanks for your guidance
>>
>>
>>>
>>>
>>> ++ bin/utils.py (working copy)
>>> @@ -913,7 +913,7 @@
>>>
>>> def applyPatch(sourceDir, file, patchLevel='0'):
>>> """apply single patch"""
>>> - cmd = "patch -d %s -p%s< %s" % ( sourceDir, patchLevel, file )
>>> + cmd = "patch -d %s -p%s --binary< %s" % ( sourceDir, patchLevel,
>>> file )
>>>
>>> for what is this good ?
>>>
>> Ah, sorry for forget to drop the dirty hack. Please ignore it.
>>
>>
>>>
>>>> Could you help to review these works and give me some advice to push
>>>> the works go furthur?
>>>>
>>>>
>>> I suggest to push the patches to the dbus-c++ project directly and to
>>> use a git source from the original because otherwise there is ongoing
>>> maintenance of the portage package required.
>>>
>> Great.
>> I found that kde-windows's emerge provide git source packing methods,
>> it's quite convenient. Thank you!
>>
>>
>> Now, I encountered some problem.
>> For dbus-c++ is autotools based source, so it need automake,
>> autoconf,... programs to gerenerate/update the Makefiles and
>> "configure" scripts, which blockes the packaging process.
>>
>> I tried to add autotools into emerge ( in one of attachments ), and
>> try to compile it. However it does not work.
>>
>> Do you have any suggestion about this situation, or would you like to
>> give some advice or direction how to solve this problem?
>>
>> Finally, thank you again for your review and advice :-)
>>
>> sincerely, Mat.
>>
>>
>>> Regards
>>> Ralf
>>>
>>>
>>>
>>> _______________________________________________
>>> Kde-windows mailing list
>>> Kde-windows at kde.org
>>> https://mail.kde.org/mailman/listinfo/kde-windows
>>>
>>>
>>
>>
>>
>> _______________________________________________
>> Kde-windows mailing list
>> Kde-windows at kde.org
>> https://mail.kde.org/mailman/listinfo/kde-windows
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-windows/attachments/20100531/89c49b57/attachment.htm
More information about the Kde-windows
mailing list