Bug 248407 – Review Request: perl-Term-ReadPassword - Asking the user for a password

Bug 248407 - Review Request: perl-Term-ReadPassword - Asking the user for a password

Summary:	Review Request: perl-Term-ReadPassword - Asking the user for a password

Status:	CLOSED NEXTRELEASE

Aliases:	None

Product:	Fedora
Classification:	Fedora
Component:	Package Review (Show other bugs)
Sub Component:
Version:	rawhide
Hardware:	All Linux

Priority	medium Severity medium
Target Milestone:	---
Target Release:	---
Assigned To:	Dominik 'Rathann' Mierzejewski
QA Contact:	Fedora Extras Quality Assurance
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2007-07-16 13:48 EDT by Steven Pritchard
Modified:	2007-11-30 17:12 EST (History)
CC List:	2 users (show)

See Also:
Fixed In Version:
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2007-07-25 12:51:53 EDT
Type:	---
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:

Flags:	dominik: fedora‑review+ wtogami: fedora‑cvs+

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description Steven Pritchard 2007-07-16 13:48:36 EDT

Spec URL: http://ftp.kspei.com/pub/steve/rpms/perl-Term-ReadPassword/perl-Term-ReadPassword.spec
SRPM URL: http://ftp.kspei.com/pub/steve/rpms/perl-Term-ReadPassword-0.07-1.src.rpm
Description:
This module lets you ask the user for a password in the traditional way,
from the keyboard, without echoing.

Comment 1 Dominik 'Rathann' Mierzejewski 2007-07-16 14:07:51 EDT

Great. I wanted to package it myself a long time ago. Taking the review.

Comment 2 Steven Pritchard 2007-07-23 18:24:21 EDT

Any luck with that review?

Comment 3 Dominik 'Rathann' Mierzejewski 2007-07-24 08:27:27 EDT

I'll do it tonight. Sorry for the delay.

Comment 4 Dominik 'Rathann' Mierzejewski 2007-07-24 17:07:40 EDT

MUST items:
 1. rpmlint is quiet
 2. package named properly
 3. spec filename matches package name
 4. meets packaging guidelines
 5. open source license
 6. license field matches actual license
 7. specfile written in AmE
 8. specfile is legible
 9. sources match upstream:
cc1a611fbb6dfd4d6e96e54434bb20b8  Term-ReadPassword-0.07.tar.gz
10. builds properly on devel/x86_64
11. no ExcludeArch necessary
12. proper BuildRequires
13. doesn't have locales
14. no shared libraries
15. not relocatable
16. owns all directories it creates
17. no duplicates in %files
18. proper file permissions
19. proper %clean section
20. consistent macro usage
21. code, not content
22. no large docs
23. %doc not necessary for runtime operation
24. no headers
25. no static libs
26. no pkgconfig files
27. no libraries with suffix
28. no devel subpackage
29. no .la files
30. no GUI
31. owns directories owned by other packages: perl-Term-ProgressBar,
perl-Term-UI -> we don't have any package owning %{perl_vendorlib}/Term, though,
so this is OK.
32. cleans buildroot in %install
33. filenames are valid UTF-8

SHOULD items:
 1. source package doesn't include any license file, please ask upstream to
include it
 2. no translations of summary/description available
 3. builds in mock/devel/x86_64
 4. noarch package
 5. software works as intended
 6. no scriptlets used
 7. no subpackages
 8. no pkgconfig files
 9. no file dependencies

APPROVED

Comment 5 Steven Pritchard 2007-07-24 18:53:06 EDT

New Package CVS Request
=======================
Package Name: perl-Term-ReadPassword
Short Description: Asking the user for a password
Owners: steve@silug.org
Branches: FC-6 F-7 EL-4 EL-5
InitialCC: fedora-perl-devel-list@redhat.com

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