Bug 1343734 (liboobs) - Review Request: liboobs - Wrapping library to the System Tools Backends
Summary: Review Request: liboobs - Wrapping library to the System Tools Backends
Keywords:
Status: CLOSED NOTABUG
Alias: liboobs
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: mulhern
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: system-tools-backends
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-06-07 19:06 UTC by Raphael Groner
Modified: 2016-07-06 18:53 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-07-06 18:53:01 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Raphael Groner 2016-06-07 19:06:47 UTC
Spec URL: https://raphgro.fedorapeople.org/review/qt/lx/liboobs.spec
SRPM URL: https://raphgro.fedorapeople.org/review/qt/lx/liboobs-3.0.0-1.fc24.src.rpm
Description: Wrapping library to the System Tools Backends
Liboobs is a wrapping library to the System Tools Backends,
it will provide easy to access GObjects to system configuration
details, like users, groups and network interfaces, it will handle
sessions with the backend and data consistency, too.
Fedora Account System Username: raphgro

Rawhide builds:
https://copr.fedorainfracloud.org/coprs/raphgro/review/build/332378/

Comment 1 Raphael Groner 2016-06-07 19:44:32 UTC
Rawhide builds:
https://copr.fedorainfracloud.org/coprs/raphgro/review/build/332385/

Comment 2 mulhern 2016-06-09 18:24:50 UTC
It's unfortunate that the upstream developers couldn't resist a lame sexist joke.

Does FPC have any sort of policy about this kind of thing?

Comment 3 Raphael Groner 2016-06-09 18:41:47 UTC
libsexy and libsexymm are not forbidden, either. What's your point? Can you make a better suggestion for the package name? There's a guideline to use the upstream project name.

Comment 5 Raphael Groner 2016-06-10 12:55:39 UTC
Hi mulhern,
thanks again for your interest. This lib depends on stb, could you do the review there, too and first? Otherwise, we need to keep with copr builds cause of the broken dependency till packages reach into testing.

Comment 6 Rex Dieter 2016-06-10 13:14:59 UTC
If you want a distro *policy* decision, FESCo is probably the better body to query, FPC is more about technical-only packaging issues.

Comment 7 mulhern 2016-06-15 19:56:40 UTC
https://fedorahosted.org/fesco/ticket/1587

Comment 8 Raphael Groner 2016-06-16 09:16:15 UTC
(In reply to mulhern from comment #7)
> https://fedorahosted.org/fesco/ticket/1587

Commented.

Comment 9 mulhern 2016-06-27 12:43:05 UTC
Associated FPC ticket: https://fedorahosted.org/fpc/ticket/633

Comment 10 mulhern 2016-06-27 13:18:00 UTC
So the first question is, is it worth packaging?

Last commit was in 2011, what makes it suddenly so desirable for Fedora in 2016?

Two apparently reasonable PRs were opened in 2014 and completely ignored by the
maintainers (eventually closed by the person who opened them). That's not usually a good sign. The fixes are small, but those are usually the easiest to merge if you are paying attention at all.

system-tools-backends also stopped in 2011, but its history since then is more obscure.

Comment 11 Raphael Groner 2016-06-27 13:39:47 UTC
Both dependencies are needed for lxqt-admin, bug #1343738.
LXQt development is very active.

Comment 12 mulhern 2016-06-27 16:23:23 UTC
_Very_ active. They seem to be in process of removing liboobs dependency due to the fact that it is not maintained.

See: 446891a14e5810c8fe395031008d406005b52dc7 and current README.

Comment 13 Raphael Groner 2016-06-27 16:44:15 UTC
(In reply to mulhern from comment #12)
> _Very_ active. They seem to be in process of removing liboobs dependency due
> to the fact that it is not maintained.
> 
> See: 446891a14e5810c8fe395031008d406005b52dc7 and current README.

Thanks for the hint. I overlooked that and 'll apply the patch or use latest git snapshot.

Comment 14 mulhern 2016-07-06 13:32:41 UTC
I'm assuming this and https://bugzilla.redhat.com/show_bug.cgi?id=1343733 can be closed, since the packages are not needed by lxqt-admin.

On the other hand, https://bugzilla.redhat.com/show_bug.cgi?id=1343608 is still open. It is _very_ simple, pure Python, and should be essentially a rubber stamp. Do you think you can do it?

Comment 15 Raphael Groner 2016-07-06 18:53:01 UTC
Agreed. Closing.


Note You need to log in before you can comment on or make changes to this bug.