Hi Hendrik, > >> > > so I think it is time to release openobex-1.6 during some time next > >> > > week. What are the pending patches? > >> > > >> > You can pull the "updates" branch from > >> > git://gitorious.org/openobex/mainline.git > >> > > >> > When Alex patches are applied, I need an additional round to further > >> > adapt > >> > to it. The patches are already there, just not uploaded, yet. > >> > >> If you pull from "usb" branch instead, you also get Alex' patches. > >> I already changes version and SONAME but with CMake, I do not follow the > >> libtool naming scheme, thus using .so.1.6.0 instead of .so.2.0.0. > >> This has no > >> practical implications, as both have the .so.2 link. > > > > as I said, I decided to 2.0.0 and also pushed that change out now. > > Yes, I know. I also acknoledge that this is a work-around for libtool > that I do not follow in the CMakeLists.txt files. Unless you say that > the release version is not openobex-1.6 but openobex-2.0. Is that the > case?
I don't understand it. We have to bump the soname. I don't care what version you call the package/library, but for Unix/Linux linking purpose we have to have a different soname. > > I also started fixing out screwed up declarations overload. Hope that > > works now as expected. > > Let me test those changes for the Win32 build before release, please! > It might needs other changes. I will tell you when I'm done with this. Sure. Let me know when you have verified it. > > It fixes all built warnings except these two: > > > > /bin/sh ../libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. > > -I.. -I../include -fvisibility=hidden -Wall -O2 > > -D_FORTIFY_SOURCE=2 -Werror -Wextra -Wno-unused-parameter > > -Wno-missing-field-initializers -Wdeclaration-after-statement > > -Wmissing-declarations -Wredundant-decls -Wcast-align > > -D_FORTIFY_SOURCE=2 -MT libopenobex_la-obex.lo -MD -MP -MF > > .deps/libopenobex_la-obex.Tpo -c -o libopenobex_la-obex.lo `test -f > > 'obex.c' || echo './'`obex.c > > gcc -DHAVE_CONFIG_H -I. -I.. -I../include -fvisibility=hidden -Wall > > -O2 -D_FORTIFY_SOURCE=2 -Werror -Wextra -Wno-unused-parameter > > -Wno-missing-field-initializers -Wdeclaration-after-statement > > -Wmissing-declarations -Wredundant-decls -Wcast-align > > -D_FORTIFY_SOURCE=2 -MT libopenobex_la-obex.lo -MD -MP -MF > > .deps/libopenobex_la-obex.Tpo -c obex.c -fPIC -DPIC -o > > .libs/libopenobex_la-obex.o > > cc1: warnings being treated as errors > > obex.c:1180: error: no previous declaration for ‘BtOBEX_ServerRegister’ > > obex.c:1207: error: no previous declaration for ‘BtOBEX_TransportConnect’ > > > > So how do we get rid of these two? I wanna build it with -Werror now. > > in lib/obex.c: > #include "bluez_compat.h" > > before > #include <openobex/obex.h> That is not it. Seems like Bluetooth support is wrongly enabled or the ifdef around SOL_RFCOMM somewhere is messed up. > Additionally, you _MUST_ make sure that include/ is searched by the > preprocessor before /usr/include. Else you might get strange build > problems when openobex is already installed. I know. Will ensure that this works. Regards Marcel ------------------------------------------------------------------------------ Register Now for Creativity and Technology (CaT), June 3rd, NYC. CaT is a gathering of tech-side developers & brand creativity professionals. Meet the minds behind Google Creative Lab, Visual Complexity, Processing, & iPhoneDevCamp as they present alongside digital heavyweights like Barbarian Group, R/GA, & Big Spaceship. http://p.sf.net/sfu/creativitycat-com _______________________________________________ Openobex-users mailing list Openobex-users@lists.sourceforge.net http://lists.sourceforge.net/lists/listinfo/openobex-users
