Bug 987395 - Review Request: gwebsockets - GLib based websockets server
Summary: Review Request: gwebsockets - GLib based websockets server
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Christopher Meng
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-23 10:44 UTC by Peter Robinson
Modified: 2013-08-23 09:50 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-08-23 09:50:26 UTC
Type: Bug
Embargoed:
i: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Peter Robinson 2013-07-23 10:44:22 UTC
SPEC: http://pbrobinson.fedorapeople.org/gwebsockets.spec
SRPM: http://pbrobinson.fedorapeople.org/gwebsockets-0.2-1.fc19.src.rpm

Description: 
A websocket server written in python. It uses GIO for network 
communication and hence it easily integrates with the GLib mainloop.

koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=5643557

Comment 1 Christopher Meng 2013-07-24 01:12:38 UTC
Ah...

Can you tell me why not python-gwebsockets?

Thanks.

Comment 2 Peter Robinson 2013-07-24 05:43:56 UTC
> Can you tell me why not python-gwebsockets?

Because of the upstream package name

Comment 3 Christopher Meng 2013-07-25 01:55:02 UTC
No test. Test requires autobahn module, which is not packaged.

I can't find a reason why not running a test.

Comment 4 Peter Robinson 2013-08-04 08:57:17 UTC
I'm unsure what info you need from me?

Comment 5 Christopher Meng 2013-08-04 09:43:12 UTC
(In reply to Peter Robinson from comment #4)
> I'm unsure what info you need from me?

Ok. 

Question: Is there a reason to disable tests a.k.a. %check section?

Comment 6 Peter Robinson 2013-08-04 23:20:09 UTC
(In reply to Christopher Meng from comment #3)
> No test. Test requires autobahn module, which is not packaged.
> 
> I can't find a reason why not running a test.

You answer the question yourself. autobahn isn't packaged

Comment 7 Christopher Meng 2013-08-05 02:51:41 UTC
OK.

I intend to add depends of that review request later, but unfortunately that module has bundled library.

So approved.

When autobahn is ready, please try adding check to see if it can works.

Thanks.

Comment 8 Peter Robinson 2013-08-05 07:09:58 UTC
> So approved.

Thanks

> When autobahn is ready, please try adding check to see if it can works.

Do you happen to have a BZ # for the review request so I can track it?

New Package GIT Request
=======================
Package Name: gwebsockets
Short Description: GLib based websockets server
Owners: pbrobinson erikos
Branches: F-19 F-18
InitialCC:

Comment 9 Christopher Meng 2013-08-05 07:12:20 UTC
(In reply to Peter Robinson from comment #8)

Later when upstream decide to remove bundled code.

It seems that a bundled pbkdf2 py is in the autobahn.

I will notify you ASAP, don't worry.

Comment 10 Gwyn Ciesla 2013-08-05 12:23:48 UTC
Git done (by process-git-requests).


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