It's just accepted in Testing project two days ago, not pushed to Trunk yet. We 
already had this discussion for one week, see attachment.

The function of the package was in meegotouchcp-connman for a long time. It's 
just a package split. And I already suggest Kevron looking into the possibility 
of making use of qt API in this package.

Peter



> -----Original Message-----
> From: [email protected] [mailto:[email protected]]
> Sent: Thursday, October 21, 2010 4:38 AM
> To: Zhu, Peter J; [email protected];
> [email protected]
> Cc: Van De Ven, Arjan; Saxena, Sunil; [email protected];
> [email protected]
> Subject: Re: [meego-packaging] [meego-commits] 8470: New package
> Trunk:Testing/connman-qt
> 
> Hi,
> 
> We should not integrate packages randomly like this. That is, this package
> seems to provide Qt API for apps. We need to run these type of changes via
> the architecture team & subsystem maintainers. This API will cause confusion
> for 3rd party developers.
> 
> Proper way to handle this is to as the Qt mobility people to include the
> functionality in there.
> 
> -sakari
> 
> 
> On 15/10/10 2:26 PM, "ext Zhu, Peter J" <[email protected]> wrote:
> 
> > Hi,
> >
> > What's this package for? Don't we have qtnetwork to utilize connman as API
> for
> > qt apps ?
> >
> > Peter
> >
> >> -----Original Message-----
> >> From: [email protected]
> >> [mailto:[email protected]] On Behalf Of Kevron Rees
> >> Sent: Friday, October 15, 2010 7:34 AM
> >> To: [email protected]
> >> Subject: [meego-commits] 8470: New package Trunk:Testing/connman-qt
> >>
> >> Hi,
> >> Adding new package connman-qt in project Trunk:Testing. Please review and
> >> accept ASAP.
> >>
> >> Justification for this new package:
> >>
> >> initial add of connman-qt, a subpackage of meegotouchcp-connman now as
> a
> >> new package
> >>
> >> Thank You,
> >> Kevron Rees
> >>
> >> [This message was auto-generated]
> >>
> >> ---
> >>
> >> Request #8470:
> >>
> >>   submit:   home:kevronx_m_rees/connman-qt(r11) ->
> >> Trunk:Testing/connman-qt
> >>
> >>
> >> Message:
> >>     initial add of connman-qt, a subpackage of meegotouchcp-connman
> now
> >> as a new package
> >>
> >> State:   new          2010-10-14T16:33:42 kevronx_m_rees
> >> Comment: None
> >>
> >>
> >> This is a NEW package in Trunk:Testing project.
> >> The files in the new package:
> >> connman-qt/
> >>   |__  Makefile
> >>   |__  connman-qt-0.0.1.tar.bz2
> >>   |__  connman-qt.changes
> >>   |__  connman-qt.spec
> >>   |__  connman-qt.yaml
> >>
> >> The content of the YAML file, connman-qt.yaml:
> >>
> ================================================================
> >> ===
> >> Name: connman-qt
> >> Summary: qt bindings for connman
> >> Version: 0.0.1
> >> Release: 1
> >> Group: System/GUI/Other
> >> License: Apache License
> >> URL: http://www.meego.com
> >> Sources:
> >>     - "%{name}-%{version}.tar.bz2"
> >> Description: |
> >>     This is a library for working with connman using Qt
> >>
> >> Requires:
> >>     - connman
> >> PkgBR:
> >>     - doxygen
> >>     - qt-devel-tools
> >> PkgConfigBR:
> >>     - QtCore >= 4.6.0
> >>     - QtDBus
> >>     - QtOpenGL
> >>     - QtGui
> >>     - dbus-1
> >> Configure: none
> >> Builder: make
> >> Files:
> >>     - "%{_libdir}/libconnman-qt.so.*"
> >> SubPackages:
> >>  - Name: devel
> >>    Summary: Development files for connman-qt
> >>    Group: Development/Libraries
> >>    Description: |
> >>     This package contains the files necessary to develop
> >>     applications using libconnman-qt
> >>    Files:
> >>     - "%{_usr}/include/connman-qt"
> >>     - "%{_usr}/lib/pkgconfig/connman-qt.pc"
> >>     - "%{_usr}/lib/connman-qt.pc"
> >>     - "%{_usr}/lib/libconnman-qt.prl"
> >>     - "%{_usr}/lib/libconnman-qt.so"
> >>
> >>
> ================================================================
> >> ===
> >>
> >>
> >> The content of the spec file, connman-qt.spec:
> >>
> ================================================================
> >> ===
> >> #
> >> # Do not Edit! Generated by:
> >> # spectacle version 0.17
> >> #
> >> # >> macros
> >> # << macros
> >>
> >> Name:       connman-qt
> >> Summary:    qt bindings for connman
> >> Version:    0.0.1
> >> Release:    1
> >> Group:      System/GUI/Other
> >> License:    Apache License
> >> URL:        http://www.meego.com
> >> Source0:    %{name}-%{version}.tar.bz2
> >> Source100:  connman-qt.yaml
> >> Requires:   connman
> >> Requires(post): /sbin/ldconfig
> >> Requires(postun): /sbin/ldconfig
> >> BuildRequires:  pkgconfig(QtCore) >= 4.6.0
> >> BuildRequires:  pkgconfig(QtDBus)
> >> BuildRequires:  pkgconfig(QtOpenGL)
> >> BuildRequires:  pkgconfig(QtGui)
> >> BuildRequires:  pkgconfig(dbus-1)
> >> BuildRequires:  doxygen
> >> BuildRequires:  qt-devel-tools
> >>
> >>
> >> %description
> >> This is a library for working with connman using Qt
> >>
> >>
> >>
> >> %package devel
> >> Summary:    Development files for connman-qt
> >> Group:      Development/Libraries
> >> Requires:   %{name} = %{version}-%{release}
> >>
> >> %description devel
> >> This package contains the files necessary to develop
> >> applications using libconnman-qt
> >>
> >>
> >>
> >> %prep
> >> %setup -q -n %{name}-%{version}
> >>
> >> # >> setup
> >> # << setup
> >>
> >> %build
> >> # >> build pre
> >> export PATH=$PATH:/usr/lib/qt4/bin
> >> qmake install_prefix=/usr
> >> # << build pre
> >>
> >>
> >> make %{?jobs:-j%jobs}
> >>
> >> # >> build post
> >> # << build post
> >> %install
> >> rm -rf %{buildroot}
> >> # >> install pre
> >> export INSTALL_ROOT=%{buildroot}
> >> # << install pre
> >> %make_install
> >>
> >> # >> install post
> >> # << install post
> >>
> >>
> >>
> >> %post -p /sbin/ldconfig
> >>
> >> %postun -p /sbin/ldconfig
> >>
> >>
> >>
> >>
> >>
> >> %files
> >> %defattr(-,root,root,-)
> >> %{_libdir}/libconnman-qt.so.*
> >> # >> files
> >> # << files
> >>
> >>
> >> %files devel
> >> %defattr(-,root,root,-)
> >> %{_usr}/include/connman-qt
> >> %{_usr}/lib/pkgconfig/connman-qt.pc
> >> %{_usr}/lib/connman-qt.pc
> >> %{_usr}/lib/libconnman-qt.prl
> >> %{_usr}/lib/libconnman-qt.so
> >> # >> files devel
> >> # << files devel
> >>
> >>
> >>
> ================================================================
> >> ===
> >>
> >> _______________________________________________
> >> Meego-commits mailing list
> >> [email protected]
> >> http://lists.meego.com/listinfo/meego-commits
> > _______________________________________________
> > MeeGo-packaging mailing list
> > [email protected]
> > http://lists.meego.com/listinfo/meego-packaging

--- Begin Message ---
>> -----Original Message-----
>> From: Kevron Rees [mailto:[email protected]]
>> Sent: Saturday, October 16, 2010 5:33 AM
>> To: Zhu, Peter J
>> Cc: [email protected]; [email protected];
>> [email protected]; [email protected]; Leibowitz, Michael;
>> Rupert, Kaitlin
>> Subject: RE: [meego-packaging] [meego-commits] 8470: New package
>> Trunk:Testing/connman-qt
>>
>> > I also found meegotouchcp-connman package creates a libconnman-qt
>> > subpackage to have similar function. Copy owners here.
>> >
>> > Seem we should unify our efforts on connman Qt.
>> >
>> > Peter
>> >
>>
>> connman-qt is from meegotouchcp-connman.  We split it out because it
>> will
>> be useful to other applications that may not want to be married to MTF
>> and
>> the Control Panel that is pulled in with meegotouchcp-connman.
>>
> Got it. Does meegotouchcp-connman has corresponding change in place so
> that no duplicate function is in MeeGo.
>

Work is ongoing to move meegotouchcp-connman to use the new connman-qt.

> BTW, from spec file, it doesn't use qt bearer management at all. Is there
> any chance to reuse some function provided by Qtnetwork?

I'll look into it deeper, but as of the present I don't see any pressing
need.  I really don't like the idea of mixing two diverse APIs together.

-Kevron


--- End Message ---
--- Begin Message ---
> I also found meegotouchcp-connman package creates a libconnman-qt
> subpackage to have similar function. Copy owners here.
>
> Seem we should unify our efforts on connman Qt.
>
> Peter
>

connman-qt is from meegotouchcp-connman.  We split it out because it will
be useful to other applications that may not want to be married to MTF and
the Control Panel that is pulled in with meegotouchcp-connman.

>> -----Original Message-----
>> From: [email protected] [mailto:[email protected]]
>> Sent: Saturday, October 16, 2010 2:25 AM
>> To: [email protected]; [email protected]
>> Cc: Zhu, Peter J; [email protected]
>> Subject: Re: [meego-packaging] [meego-commits] 8470: New package
>> Trunk:Testing/connman-qt
>>
>> On 16/10/10 4:03 AM, "ext Kevron Rees" <[email protected]>
>> wrote:
>>
>> > There is no API in bearer managment or in QtNetwork to list available
>> > connection technologies (3G, Wifi, Ethernet, GPS, etc), disable/enable
>> or
>> > connect to various technologies, etc.
>>
>> Using QtBearermanagement, you can list, connect and disconnect to
>> already
>> configured and known networks. But that's about it. You cannot configure
>> a
>> new connection.
>>
>>
>> >
>> > Regards,
>> > -Kevron
>> >
>> >> Hi,
>> >>
>> >> Peter talks about the connman plugin for bearer management:
>> >> http://doc.qt.nokia.com/qtmobility-1.0/bearer-management.html
>> >>
>> >> The plugin is available in Qt Network module since Qt 4.7.0.
>> >>
>> >> Cheers,
>> >>
>> >> Fathi
>> >> ________________________________________
>> >> From: [email protected]
>> >> [[email protected]] On Behalf Of ext Kevron Rees
>> >> [[email protected]]
>> >> Sent: Friday, October 15, 2010 7:35 PM
>> >> To: Zhu, Peter J
>> >> Cc: [email protected]; [email protected]
>> >> Subject: Re: [meego-packaging] [meego-commits] 8470: New package
>> >> Trunk:Testing/connman-qt
>> >>
>> >>> Hi,
>> >>>
>> >>> What's this package for? Don't we have qtnetwork to utilize connman
>> as
>> >>> API
>> >>> for qt apps ?
>> >>>
>> >>> Peter
>> >>>
>> >>
>> >> QtNetwork is a Qt module for working with things like http requests,
>> and
>> >> sockets.  It does not manage network connections at the device level.
>> >>
>> >> connman-qt provides an interface to qt apps for connecting to
>> networks at
>> >> the device level and then a developer can use QtNetwork to do things
>> after
>> >> a connection has been established.
>> >>
>> >> Hope that clears up what connman-qt is for.
>> >>
>> >>>> -----Original Message-----
>> >>>> From: [email protected]
>> >>>> [mailto:[email protected]] On Behalf Of Kevron Rees
>> >>>> Sent: Friday, October 15, 2010 7:34 AM
>> >>>> To: [email protected]
>> >>>> Subject: [meego-commits] 8470: New package Trunk:Testing/connman-qt
>> >>>>
>> >>>> Hi,
>> >>>> Adding new package connman-qt in project Trunk:Testing. Please
>> review
>> >>>> and
>> >>>> accept ASAP.
>> >>>>
>> >>>> Justification for this new package:
>> >>>>
>> >>>> initial add of connman-qt, a subpackage of meegotouchcp-connman now
>> as
>> >>>> a
>> >>>> new package
>> >>>>
>> >>>> Thank You,
>> >>>> Kevron Rees
>> >>>>
>> >>>> [This message was auto-generated]
>> >>>>
>> >>>> ---
>> >>>>
>> >>>> Request #8470:
>> >>>>
>> >>>>   submit:   home:kevronx_m_rees/connman-qt(r11) ->
>> >>>> Trunk:Testing/connman-qt
>> >>>>
>> >>>>
>> >>>> Message:
>> >>>>     initial add of connman-qt, a subpackage of meegotouchcp-connman
>> now
>> >>>> as a new package
>> >>>>
>> >>>> State:   new          2010-10-14T16:33:42 kevronx_m_rees
>> >>>> Comment: None
>> >>>>
>> >>>>
>> >>>> This is a NEW package in Trunk:Testing project.
>> >>>> The files in the new package:
>> >>>> connman-qt/
>> >>>>   |__  Makefile
>> >>>>   |__  connman-qt-0.0.1.tar.bz2
>> >>>>   |__  connman-qt.changes
>> >>>>   |__  connman-qt.spec
>> >>>>   |__  connman-qt.yaml
>> >>>>
>> >>>> The content of the YAML file, connman-qt.yaml:
>> >>>>
>> ================================================================
>> >>>> ===
>> >>>> Name: connman-qt
>> >>>> Summary: qt bindings for connman
>> >>>> Version: 0.0.1
>> >>>> Release: 1
>> >>>> Group: System/GUI/Other
>> >>>> License: Apache License
>> >>>> URL: http://www.meego.com
>> >>>> Sources:
>> >>>>     - "%{name}-%{version}.tar.bz2"
>> >>>> Description: |
>> >>>>     This is a library for working with connman using Qt
>> >>>>
>> >>>> Requires:
>> >>>>     - connman
>> >>>> PkgBR:
>> >>>>     - doxygen
>> >>>>     - qt-devel-tools
>> >>>> PkgConfigBR:
>> >>>>     - QtCore >= 4.6.0
>> >>>>     - QtDBus
>> >>>>     - QtOpenGL
>> >>>>     - QtGui
>> >>>>     - dbus-1
>> >>>> Configure: none
>> >>>> Builder: make
>> >>>> Files:
>> >>>>     - "%{_libdir}/libconnman-qt.so.*"
>> >>>> SubPackages:
>> >>>>  - Name: devel
>> >>>>    Summary: Development files for connman-qt
>> >>>>    Group: Development/Libraries
>> >>>>    Description: |
>> >>>>     This package contains the files necessary to develop
>> >>>>     applications using libconnman-qt
>> >>>>    Files:
>> >>>>     - "%{_usr}/include/connman-qt"
>> >>>>     - "%{_usr}/lib/pkgconfig/connman-qt.pc"
>> >>>>     - "%{_usr}/lib/connman-qt.pc"
>> >>>>     - "%{_usr}/lib/libconnman-qt.prl"
>> >>>>     - "%{_usr}/lib/libconnman-qt.so"
>> >>>>
>> >>>>
>> ================================================================
>> >>>> ===
>> >>>>
>> >>>>
>> >>>> The content of the spec file, connman-qt.spec:
>> >>>>
>> ================================================================
>> >>>> ===
>> >>>> #
>> >>>> # Do not Edit! Generated by:
>> >>>> # spectacle version 0.17
>> >>>> #
>> >>>> # >> macros
>> >>>> # << macros
>> >>>>
>> >>>> Name:       connman-qt
>> >>>> Summary:    qt bindings for connman
>> >>>> Version:    0.0.1
>> >>>> Release:    1
>> >>>> Group:      System/GUI/Other
>> >>>> License:    Apache License
>> >>>> URL:        http://www.meego.com
>> >>>> Source0:    %{name}-%{version}.tar.bz2
>> >>>> Source100:  connman-qt.yaml
>> >>>> Requires:   connman
>> >>>> Requires(post): /sbin/ldconfig
>> >>>> Requires(postun): /sbin/ldconfig
>> >>>> BuildRequires:  pkgconfig(QtCore) >= 4.6.0
>> >>>> BuildRequires:  pkgconfig(QtDBus)
>> >>>> BuildRequires:  pkgconfig(QtOpenGL)
>> >>>> BuildRequires:  pkgconfig(QtGui)
>> >>>> BuildRequires:  pkgconfig(dbus-1)
>> >>>> BuildRequires:  doxygen
>> >>>> BuildRequires:  qt-devel-tools
>> >>>>
>> >>>>
>> >>>> %description
>> >>>> This is a library for working with connman using Qt
>> >>>>
>> >>>>
>> >>>>
>> >>>> %package devel
>> >>>> Summary:    Development files for connman-qt
>> >>>> Group:      Development/Libraries
>> >>>> Requires:   %{name} = %{version}-%{release}
>> >>>>
>> >>>> %description devel
>> >>>> This package contains the files necessary to develop
>> >>>> applications using libconnman-qt
>> >>>>
>> >>>>
>> >>>>
>> >>>> %prep
>> >>>> %setup -q -n %{name}-%{version}
>> >>>>
>> >>>> # >> setup
>> >>>> # << setup
>> >>>>
>> >>>> %build
>> >>>> # >> build pre
>> >>>> export PATH=$PATH:/usr/lib/qt4/bin
>> >>>> qmake install_prefix=/usr
>> >>>> # << build pre
>> >>>>
>> >>>>
>> >>>> make %{?jobs:-j%jobs}
>> >>>>
>> >>>> # >> build post
>> >>>> # << build post
>> >>>> %install
>> >>>> rm -rf %{buildroot}
>> >>>> # >> install pre
>> >>>> export INSTALL_ROOT=%{buildroot}
>> >>>> # << install pre
>> >>>> %make_install
>> >>>>
>> >>>> # >> install post
>> >>>> # << install post
>> >>>>
>> >>>>
>> >>>>
>> >>>> %post -p /sbin/ldconfig
>> >>>>
>> >>>> %postun -p /sbin/ldconfig
>> >>>>
>> >>>>
>> >>>>
>> >>>>
>> >>>>
>> >>>> %files
>> >>>> %defattr(-,root,root,-)
>> >>>> %{_libdir}/libconnman-qt.so.*
>> >>>> # >> files
>> >>>> # << files
>> >>>>
>> >>>>
>> >>>> %files devel
>> >>>> %defattr(-,root,root,-)
>> >>>> %{_usr}/include/connman-qt
>> >>>> %{_usr}/lib/pkgconfig/connman-qt.pc
>> >>>> %{_usr}/lib/connman-qt.pc
>> >>>> %{_usr}/lib/libconnman-qt.prl
>> >>>> %{_usr}/lib/libconnman-qt.so
>> >>>> # >> files devel
>> >>>> # << files devel
>> >>>>
>> >>>>
>> >>>>
>> ================================================================
>> >>>> ===
>> >>>>
>> >>>> _______________________________________________
>> >>>> Meego-commits mailing list
>> >>>> [email protected]
>> >>>> http://lists.meego.com/listinfo/meego-commits
>> >>>
>> >>
>> >> _______________________________________________
>> >> MeeGo-packaging mailing list
>> >> [email protected]
>> >> http://lists.meego.com/listinfo/meego-packaging
>> >>
>> >
>
>


--- End Message ---
_______________________________________________
MeeGo-packaging mailing list
[email protected]
http://lists.meego.com/listinfo/meego-packaging

Reply via email to