[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Xisco Faulí  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #14 from Xisco Faulí  ---
Fito, why has it been moved back to NEW without giving any explanation?
Putting it back to RESOLVED FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Adolfo Jayme  changed:

   What|Removed |Added

 Status|RESOLVED|NEW
 Resolution|FIXED   |---

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Adolfo Jayme  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

--- Comment #13 from Commit Notification 
 ---
Arkadiy Illarionov committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=f465685aecf508be987ae199e77667af896a5e8f

Resolves: tdf#112343 remove last dbus-glib mentions

It will be available in 6.2.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

--- Comment #12 from Commit Notification 
 ---
Arkadiy Illarionov committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=02addb3618f026acd72ece76fc726b9473cc5940

tdf#112343 Change dependency from dbus-glib to dbus

It will be available in 6.2.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:6.1.0|target:6.1.0 target:6.2.0

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

--- Comment #11 from Arkadiy Illarionov  ---
Hi,
I've made patch for this issue: https://gerrit.libreoffice.org/#/c/57157/
Can anybody review it?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

--- Comment #10 from Arkadiy Illarionov  ---
(In reply to Samuel Mehrbrodt (CIB) from comment #9)
> (In reply to Arkadiy Illarionov from comment #8)
> > As I understand, ScreenSaverInhibitor was last user of dbus-glib and check
> > for DBus support
> > (https://opengrok.libreoffice.org/xref/core/configure.ac#9955) can be
> > changed from 
> > PKG_CHECK_MODULES(DBUS, dbus-glib-1 >= 0.70)
> > to 
> > PKG_CHECK_MODULES(DBUS, dbus-1 >= 0.60)
> > 
> > However some modules, e.g. sd/source/ui/remotecontrol/BluetoothServer.cxx,
> > use dbus and glib, so proposed change breaks build of these modules.
> > 
> > I believe fixing this is more or less simple for somebody familiar with
> > build system.
> 
> Can you port the BluetoothServer.cxx module also over?

I'm not sure I can do this. At least I couldn't find DBus -> GDBus API
comparison similar to https://developer.gnome.org/gio/stable/ch35s02.html.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

--- Comment #9 from Samuel Mehrbrodt (CIB)  ---
(In reply to Arkadiy Illarionov from comment #8)
> As I understand, ScreenSaverInhibitor was last user of dbus-glib and check
> for DBus support
> (https://opengrok.libreoffice.org/xref/core/configure.ac#9955) can be
> changed from 
> PKG_CHECK_MODULES(DBUS, dbus-glib-1 >= 0.70)
> to 
> PKG_CHECK_MODULES(DBUS, dbus-1 >= 0.60)
> 
> However some modules, e.g. sd/source/ui/remotecontrol/BluetoothServer.cxx,
> use dbus and glib, so proposed change breaks build of these modules.
> 
> I believe fixing this is more or less simple for somebody familiar with
> build system.

Can you port the BluetoothServer.cxx module also over?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

--- Comment #8 from Arkadiy Illarionov  ---
As I understand, ScreenSaverInhibitor was last user of dbus-glib and check for
DBus support (https://opengrok.libreoffice.org/xref/core/configure.ac#9955) can
be changed from 
PKG_CHECK_MODULES(DBUS, dbus-glib-1 >= 0.70)
to 
PKG_CHECK_MODULES(DBUS, dbus-1 >= 0.60)

However some modules, e.g. sd/source/ui/remotecontrol/BluetoothServer.cxx, use
dbus and glib, so proposed change breaks build of these modules.

I believe fixing this is more or less simple for somebody familiar with build
system.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:6.1.0

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

--- Comment #7 from Commit Notification 
 ---
Arkadiy Illarionov committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=fc3e31ff8eaff7385e200f82972c4e5eb9055723

tdf#112343 Port ScreenSaverInhibitor to GDBus

It will be available in 6.1.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Samuel Mehrbrodt (CIB)  changed:

   What|Removed |Added

   Hardware|x86-64 (AMD64)  |All

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

--- Comment #6 from Samuel Mehrbrodt (CIB)  ---
(In reply to Arkadiy Illarionov from comment #5)
> Hi,
> I'd like to work on this bug.
> Is there any tests for DBus in source tree? How can I test my changes?

I'm not aware of any tests. You need to find a way to trigger the code path.
If you need help with that, you might better ask in IRC.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

--- Comment #5 from Arkadiy Illarionov  ---
Hi,
I'd like to work on this bug.
Is there any tests for DBus in source tree? How can I test my changes?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Samuel Mehrbrodt (CIB)  changed:

   What|Removed |Added

   Keywords|needsDevAdvice  |
 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

--- Comment #4 from Samuel Mehrbrodt (CIB)  ---
Yes, existing usages of dbus-glib should be ported over to gdbus.
There is also a UNO Api wrapper for that, which should be used. See
https://cgit.freedesktop.org/libreoffice/core/commit/?id=91d361604781f92640c9727add1e08cde854037e

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Xisco Faulí  changed:

   What|Removed |Added

 CC||s.mehrbr...@gmail.com,
   ||xiscofa...@libreoffice.org

--- Comment #3 from Xisco Faulí  ---
(In reply to Buovjaga from comment #2)
> It seems GDBus is used in some places:
> https://cgit.freedesktop.org/libreoffice/core/commit/
> ?id=2a593ff092c628eff906c4df9b4da993a3973ec3

@Samuel, do you have any insight here?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Buovjaga  changed:

   What|Removed |Added

   Keywords|needsDevEval|needsDevAdvice
 CC||todven...@suomi24.fi

--- Comment #2 from Buovjaga  ---
It seems GDBus is used in some places:
https://cgit.freedesktop.org/libreoffice/core/commit/?id=2a593ff092c628eff906c4df9b4da993a3973ec3

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112343] Consider porting to gdbus.

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=112343

Jean-Baptiste Faure  changed:

   What|Removed |Added

   Keywords||needsDevEval
 CC||jbfa...@libreoffice.org

--- Comment #1 from Jean-Baptiste Faure  ---
Set needDevEval keyword.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs