Bug 612671

Summary: Review Request: nodm - A display manager automatically starting an X session
Product: [Fedora] Fedora Reporter: Tom Atkinson <tom_atkinson>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: cwickert, fedora-package-review, michel, notting, pahan
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-06-15 16:01:26 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 201449    

Description Tom Atkinson 2010-07-08 14:23:29 EDT
Spec URL: http://dl.iondrip.net/nodm.spec
SRPM URL: http://dl.iondrip.net/nodm-0.6-2.fc13.src.rpm

Description: An automatic display manager which automatically starts an X session at system boot. It is meant for devices like smartphones, but can be used on a regular computer as well, if the security implications are acceptable.

This is my first package, and I am seeking a sponsor.

This is a re-review since the package has been orphaned and was last updated more than 3 months ago. The original review request is bug #559559. The spec and srpm linked to above are unmodified.
Comment 1 Michel Alexandre Salim 2010-07-11 08:41:49 EDT
Hi Tom,

Could you take a look at some of these and do the pre-review a couple of them? Provide the links to the bugs you pre-review and I'll then check your work, finish the review, and if everything is satisfactory, approve this request.

I normally sponsor packagers if during the review process they show they understand the packaging guidelines, but since in this case the package is being taken over unchanged, I'm afraid I need another way of gauging this understanding :) Plus this helps take a stab at the review backlog.


Comment 2 Tom Atkinson 2010-07-24 11:13:15 EDT
Hi Michel

I have done a pre-review of tidyp at #617524

Comment 3 Tom Atkinson 2010-07-24 12:04:18 EDT
Also reviewed poezio at bug #617405
Comment 4 Michel Alexandre Salim 2010-07-27 04:21:05 EDT
Thanks, I'll take a look later today.
Comment 5 Christoph Wickert 2010-07-29 19:44:08 EDT
Hi Tom, from a formal point of view, your review in bug #617405 is fine, however poezio is packaged badly. Just going through the checklist is not enough. I will review the package soon, then you see what I mean and can learn from it.
Comment 6 Jason Tibbitts 2010-12-01 21:07:29 EST
Is anything happening here?