Bug 226143 - Merge Review: mingetty
Merge Review: mingetty
Status: CLOSED ERRATA
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Gwyn Ciesla
Fedora Package Reviews List
: Reopened
Depends On:
Blocks: F9MergeReviewTarget
  Show dependency treegraph
 
Reported: 2007-01-31 14:40 EST by Nobody's working on this, feel free to take it
Modified: 2008-01-25 10:34 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-01-25 10:34:46 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
limburgher: fedora‑review+


Attachments (Terms of Use)
Patch for all current issues, save the URL tag. (651 bytes, patch)
2007-12-26 16:08 EST, Gwyn Ciesla
no flags Details | Diff

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 14:40:59 EST
Fedora Merge Review: mingetty

http://cvs.fedora.redhat.com/viewcvs/devel/mingetty/
Initial Owner: laroche@redhat.com
Comment 1 Gwyn Ciesla 2007-12-26 15:24:47 EST
rpmlint output on SRPM:
mingetty.src: W: summary-ended-with-dot A compact getty program for virtual
consoles only.
Summary ends with a dot.

mingetty.src: W: invalid-license GPL
The value of the License tag was not recognized.  Known values are:
"Adobe", "AFL", "AGPLv1", "AGPLv3", "AMPAS BSD", "ARL", "ASL 1.0", "ASL 1.0+",
"ASL 1.1", "ASL 1.1+", "ASL 2.0", "ASL 2.0+", "APSL 2.0", "APSL 2.0+",
"Artistic 2.0", "Artistic clarified", "BitTorrent", "Boost", "BSD", "BSD with
advertising", "CeCILL", "CDDL", "CPL", "Condor", "Copyright only", "Cryptix",
"Crystal Stacker", "EPL", "eCos", "EFL 2.0", "EFL 2.0+", "EU Datagrid", "FTL",
"Giftware", "Glide", "gnuplot", "GPL+", "GPL+ or Artistic", "GPLv2+ or
Artistic", "GPLv2", "GPLv2 with exceptions", "GPLv2+", "GPLv2+ with
exceptions", "GPLv3", "GPLv3 with exceptions", "GPLv3+", "GPLv3+ with
exceptions", "IBM", "IJG", "ImageMagick", "iMatix", "Imlib2", "Intel ACPI",
"Interbase", "ISC", "Jabber", "JasPer", "LGPLv2", "LGPLv2 with exceptions",
"LGPLv2+", "LGPLv3", "LGPLv3+", "libtiff", "LPL", "LPPL", "mecab-ipadic",
"MIT", "MPLv1.0", "MPLv1.0+", "MPLv1.1", "MPLv1.1+", "NCSA", "NGPL", "NOSL",
"Netscape", "Nokia", "OpenLDAP", "OpenPBS", "OReilly", "OSL 1.0", "OSL 1.0+",
"OSL 1.1", "OSL 1.1+", "OSL 2.0", "OSL 2.0+", "OSL 3.0", "OSL 3.0+",
"OpenSSL", "Phorum", "PHP", "Public Domain", "Python", "QPL", "RPSL", "Ruby",
"Sleepycat", "SISSL", "SLIB", "SPL", "TCL", "UCD", "Vim", "VNLSL", "VSL",
"W3C", "WTFPL", "wxWindows", "xinetd", "Zend", "ZPLv1.0", "ZPLv1.0+",
"ZPLv2.0", "ZPLv2.0+", "ZPLv2.1", "ZPLv2.1+", "zlib", "CDL", "FBSDDL", "GFDL",
"IEEE", "OFSFDL", "Open Publication", "CC-BY", "CC-BY-SA", "DSL", "Free Art",
"Arphic", "Baekmuk", "Bitstream Vera", "mplus", "OFL", "STIX", "Utopia",
"XANO", "Redistributable, no modification permitted", "Freely redistributable
without restriction".

mingetty.src: W: no-url-tag
The URL tag is missing.


Correct license tag is GPLv2+.  Since URL tag is missing, and Red Hat is
upstream, this should probably get space on hosted.

rpmlint on RPMS have the same issues, plus:

The last changelog entry doesn't contain a version. Please insert the
version that is coherent with the version of the package and rebuild it.

Just change 
* Tue Aug 21 2007 Florian La Roche <laroche@redhat.com>

to 

* Tue Aug 21 2007 Florian La Roche <laroche@redhat.com> - 1.07-6

But then of course you'll need to update it again for the rebuild after the
merge review. :)
Comment 2 Gwyn Ciesla 2007-12-26 16:08:36 EST
Created attachment 290425 [details]
Patch for all current issues, save the URL tag.
Comment 3 Florian La Roche 2008-01-11 16:13:33 EST
Current rpm package has the above patch applied plus also an url is now added.

Thanks a lot for the review,

Florian La Roche
Comment 4 Patrice Dumas 2008-01-11 17:10:30 EST
You should not close the review before the reviewer has set
the fedora review flag to +. 
Comment 5 Gwyn Ciesla 2008-01-14 06:11:14 EST
Indeed.  I've just now finished the rest of the review, and there's another
change to be made.  The Source tag must include a URL, a la: 
Source0: http://downloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz

Fix that and I'll approve.
Comment 6 Florian La Roche 2008-01-25 10:31:12 EST
This is fixed now.

Thanks for the review,

Florian La Roche
Comment 7 Gwyn Ciesla 2008-01-25 10:34:46 EST
Perfect. Thank you, and you're very welcome.

APPROVED.

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