Bug 427738 - Review Request: dropbear - SSH2 server and client
Summary: Review Request: dropbear - SSH2 server and client
Keywords:
Status: CLOSED WORKSFORME
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-01-07 11:04 UTC by Lennert Buytenhek
Modified: 2014-02-20 13:44 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-03-11 02:24:28 UTC
Type: ---
Embargoed:
tcallawa: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Lennert Buytenhek 2008-01-07 11:04:29 UTC
Spec URL: http://www.wantstofly.org/~buytenh/fedora-packages/dropbear.spec

SRPM URL: http://www.wantstofly.org/~buytenh/fedora-packages/dropbear-0.50-1.src.rpm

Description:
Dropbear is a relatively small SSH 2 server and client.  Dropbear
is particularly useful for "embedded"-type Linux (or other Unix)
systems, such as wireless routers.

This is my first package.  I am looking for a sponsor.

Comment 1 Tom "spot" Callaway 2008-01-07 15:34:39 UTC
Taking this one.

Comment 2 Tom "spot" Callaway 2008-01-07 15:57:27 UTC
Only problems that I see:

- Please put the full URL path to the Sources (where possible).

- dropbear.src: W: strange-permission dropbear.init 0755

Should be easy enough to fix, just make sure the perms are -x on that file
before making the SRPM.

- dropbear.x86_64: W: file-not-utf8 /usr/share/doc/dropbear-0.50/CHANGES

Just add these lines in %setup:

# convert to UTF-8
iconv -f iso-8859-1 -t utf-8 -o CHANGES{.utf8,}
mv CHANGES{.utf8,}

- dropbear.x86_64: W: service-default-enabled /etc/rc.d/init.d/dropbear
  dropbear.x86_64: W: no-reload-entry /etc/rc.d/init.d/dropbear

Do you intend for this service to be enabled by default? Also, since the init
file is custom, please add the reload function. It can be as simple as calling
"stop, start".

Comment 3 Lennert Buytenhek 2008-01-10 16:07:21 UTC
Thank you for the review.  I have uploaded 0.50-2 (which incorporates
all your review comments) to:

http://www.wantstofly.org/~buytenh/fedora-packages/dropbear-0.50-2.src.rpm

I hope I've fixed everything in this version.

Comment 4 Tom "spot" Callaway 2008-01-14 19:26:27 UTC
Good:

- rpmlint checks return:

dropbear.src:81: W: macro-in-%changelog setup
dropbear.src: W: mixed-use-of-spaces-and-tabs (spaces: line 82, tab: line 1)

Both of these are trivial, but you really shouldn't use macros in %changelog,
because rpm will try to evaluate them. Safe to ignore.

- package meets naming guidelines
- package meets packaging guidelines
- license (MIT) OK, text in %doc, matches source 
- spec file legible, in am. english
- source matches upstream (97f31bbe1d36f156ec895de97ecc1d400da5f0e5)
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file

APPROVED. Go ahead and request sponsorship, as described here:
http://fedoraproject.org/wiki/PackageMaintainers/Join#head-a601c13b0950a89568deafa65f505b4b58ee869b

Comment 5 Lennert Buytenhek 2008-01-14 21:17:48 UTC
New Package CVS Request
=======================
Package Name: dropbear
Short Description: SSH2 server and client
Owners: buytenh
Branches: F-8
InitialCC:
Cvsextras Commits: yes

Comment 6 Kevin Fenzi 2008-01-14 21:49:40 UTC
cvs done.

Comment 7 Lubomir Kundrak 2008-03-08 10:19:15 UTC
I would like to see this in EPEL.
Lennert: I'd be happy if you could maintain it; would you?

Comment 8 Lennert Buytenhek 2008-03-08 10:33:13 UTC
Package Change Request
======================
Package Name: dropbear
New Branches: EL-4 EL-5

Comment 9 Kevin Fenzi 2008-03-08 19:34:43 UTC
cvs done.

Comment 10 Christopher Meng 2014-02-20 06:52:04 UTC
Package Change Request
======================
Package Name: dropbear
New Branches: epel7
Owners: buytenh cicku itamarjp

Comment 11 Gwyn Ciesla 2014-02-20 13:44:56 UTC
Git done (by process-git-requests).


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