Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 190873 - Review Request: gnome-ppp - A GNOME 2 WvDial frontend
Summary: Review Request: gnome-ppp - A GNOME 2 WvDial frontend
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michael A. Peters
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-05-05 20:21 UTC by Christoph Wickert
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-06 23:58:05 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Christoph Wickert 2006-05-05 20:21:27 UTC
Spec URL: 
http://home.arcor.de/christoph.wickert/fedora/extras/review/SPECS/gnome-ppp.spec
SRPM URL: 
http://home.arcor.de/christoph.wickert/fedora/extras/review/SRPMS/gnome-ppp-0.3.23-1.fc5.src.rpm
Description: 
GNOME PPP is a modem dialup tool designed to integrate nicely into GNOME 2 
Desktop Environment. It is very similar to GPPP dialup utility created for 
GNOME 1 and KDE equivalent, KPPP. Since it is a graphical frontend for the 
excellent WvDial too, very little knowledge is needed to setup a dialup 
connection using GNOME PPP. It features ease of use, HIG dialogs, auto 
detection of your modem and connection monitoring.

Comments:
builds fine in mock
rpmlint is happy

Comment 1 Michael A. Peters 2006-05-06 06:02:00 UTC
Good:

* md5sum matches upstream:
ec2e20fc713a01c953d759bea3df8618  gnome-ppp-0.3.23.tar.bz2
* Package appropriately named (matches upstream)
* spec file matches src.rpm name
* Package meets package guidelines
* Appropriate License (GPL), matches COPYING file, in %doc
* Spec file written in legible American English
* Succesfully builds in FC5 i386 mock
* locales handled properly
* no shared libs to fuss over
* not relocatable
* owns every directory it creates
* no duplicate files. Not even any triplicate files.
* %defattr properly used, proper file permissions
* Proper %clean
* consistent use of $RPM_BUILD_ROOT
* doc package not necessary
* Package functions w/o docs
* No devel package to fuss over
* Includes proper desktop file - uses desktop-file-install
* Scriptlets are sane

-=-

I no longer have a modem for it to configure, so I have not tested this package.
But that is a *should* and not a *must*

-=-

Output of rpmlint on mock built fc5 i386 packages:

[mpeters@jerusalem result]$ ls *.rpm
gnome-ppp-0.3.23-1.fc5.i386.rpm  gnome-ppp-debuginfo-0.3.23-1.fc5.i386.rpm
gnome-ppp-0.3.23-1.fc5.src.rpm
[mpeters@jerusalem result]$ rpmlint *.rpm
[mpeters@jerusalem result]$ echo $?
0
[mpeters@jerusalem result]$ 

APPROVED

Comment 2 Michael A. Peters 2006-05-06 06:12:32 UTC
Note on the test thing - I did try the app. No segfault.
It correctly creates a ~/.wvdial.conf

w/o a modem for it to detect though, that's as far as I could get.

Comment 3 Christoph Wickert 2006-05-06 22:53:24 UTC
Thanks for the review and responding so quickly, Michael.

(In reply to comment #1)
> 
> I no longer have a modem for it to configure, so I have not tested this package.
> But that is a *should* and not a *must*
> 
IMO it should be a MUST and to be honest I haven't thought about that before. I
don't have a modem atm ether, so I did not test this package. I rolled this
package for my sister who is a complete newby. She manged to setup a connection
and is happy with the package, so I guess it must be foolproof. ;) But I'm not
sure if modem detection and works properly, so I asked a friend of mine to test
this.

I'm not going to build or release this, before I receive feedback (because I'm
not interested in maintaining a broken piece of software).

Comment 4 Michael A. Peters 2006-05-06 23:15:15 UTC
It will use /dev/modem by default - I *think* udev will create /dev/modem
symlink if hardware is present, so modem detection would only be necessary if
fedora itself did not pick it up (or more than one is present).

Even if detection fails, /dev/ttySN manually configured should work.

Comment 5 Christoph Wickert 2006-05-06 23:58:05 UTC
Right, gnome-ppp is just calling wvdialconf anyway, so this doesn't really
matter. Imported into CVS, build for devel.

Closing.


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