Bug 957465 - Review Request: minised - A smaller, cheaper, faster SED implementation
Summary: Review Request: minised - A smaller, cheaper, faster SED implementation
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Björn 'besser82' Esser
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-04-28 09:32 UTC by Christopher Meng
Modified: 2014-05-09 22:03 UTC (History)
4 users (show)

Fixed In Version: minised-1.14-4.el6
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-05-21 20:33:27 UTC
Type: ---
Embargoed:
besser82: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)
output of fedora-review and manual inspection (6.53 KB, text/plain)
2013-05-16 16:06 UTC, Björn 'besser82' Esser
no flags Details
corrected spec-file (1.26 KB, text/x-rpm-spec)
2013-05-16 16:11 UTC, Björn 'besser82' Esser
no flags Details

Description Christopher Meng 2013-04-28 09:32:09 UTC
Spec URL: http://cicku.me/minised.spec
SRPM URL: http://cicku.me/minised-1.14-1.fc20.src.rpm

Description: This is a smaller, cheaper, faster SED implementation. Minix uses it. GNU used
to use it, until they built their own sed around an extended (some would say
over-extended) regexp package. For embedded use we searched for a tiny sed
implementation especially for use with the dietlibc and found Eric S. Raymond's
sed implementation quite handy. Though it suffered several bugs and was not
under active maintenance anymore. After sending a bunch of fixes we agreed to
continue maintaining this lovely, historic sed implementation.

Fedora Account System Username: cicku

Comment 1 Björn 'besser82' Esser 2013-05-16 16:06:46 UTC
Created attachment 748904 [details]
output of fedora-review and manual inspection

Comment 2 Björn 'besser82' Esser 2013-05-16 16:11:05 UTC
Created attachment 748905 [details]
corrected spec-file

Comment 3 Björn 'besser82' Esser 2013-05-16 16:16:11 UTC
Hi Christopher!

So this was my first review for a Fedora-pkg.

On specand srpm rpmlint output was:

minised.src: W: spelling-error %description -l en_US Minix -> Mi nix, Mi-nix, Min ix
minised.src: W: spelling-error %description -l en_US sed -> tied, ed, seed
minised.src: W: spelling-error %description -l en_US dietlibc -> dietetic
minised.src:9: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 9)
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

Spelling errors are false positives, but mixed-use-of-\w-and-\t was correct.

The Makefile from Sources0 provides a check-target. Why don't you use it in the spec-file? See: http://fedoraproject.org/wiki/How_to_create_an_RPM_package#.25check_section

Please rebuild srpm with attached fixed spec-file.

Comment 4 Christopher Meng 2013-05-17 02:15:04 UTC
(In reply to comment #3)

Fixed.


NEW SPEC URL: http://cicku.me/minised.spec
NEW SRPM URL: http://cicku.me/minised-1.14-2.fc20.src.rpm

Comment 5 Björn 'besser82' Esser 2013-05-17 07:15:16 UTC
Allright, seems fine, now.

APPROVED!

Comment 6 Christopher Meng 2013-05-17 08:47:36 UTC
New Package SCM Request
=======================
Package Name: minised
Short Description: A smaller, cheaper, faster SED implementation
Owners: cicku
Branches: f18 f19

Comment 7 Gwyn Ciesla 2013-05-17 13:04:29 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2013-05-18 01:36:53 UTC
minised-1.14-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/minised-1.14-2.fc18

Comment 9 Fedora Update System 2013-05-18 01:50:12 UTC
minised-1.14-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/minised-1.14-2.fc19

Comment 10 Fedora Update System 2013-05-18 17:57:54 UTC
minised-1.14-2.fc19 has been pushed to the Fedora 19 testing repository.

Comment 11 Fedora Update System 2013-05-28 00:58:27 UTC
minised-1.14-2.fc18 has been pushed to the Fedora 18 stable repository.

Comment 12 Fedora Update System 2013-05-28 02:22:21 UTC
minised-1.14-2.fc19 has been pushed to the Fedora 19 stable repository.

Comment 13 Christopher Meng 2014-04-20 11:09:46 UTC
Package Change Request
======================
Package Name: minised
New Branches: el6 epel7
Owners: cicku

Comment 14 Kevin Fenzi 2014-04-20 23:28:45 UTC
Git done (by process-git-requests).

Comment 15 Fedora Update System 2014-04-23 01:37:54 UTC
minised-1.14-4.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/minised-1.14-4.el6

Comment 16 Fedora Update System 2014-05-09 22:03:29 UTC
minised-1.14-4.el6 has been pushed to the Fedora EPEL 6 stable repository.


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