Bug 448901

Summary:	Review Request: nled - Simple curses-based text editor
Product:	[Fedora] Fedora	Reporter:	Chris Tyler <ctyler.fedora>
Component:	Package Review	Assignee:	Andrew Overholt <overholt>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	low
Version:	rawhide	CC:	fedora-package-review, notting
Target Milestone:	---	Flags:	overholt: fedora-review+ kevin: fedora-cvs+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	2.52-3.fc8	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2008-06-03 07:31:59 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Chris Tyler 2008-05-29 12:55:51 UTC

Spec URL: http://chris.tylers.info/rpm/nled.spec
SRPM URL: http://chris.tylers.info/rpm/nled-2.52-1.fc9.src.rpm

Review request for curses-based text editor nled.
- First package submitted to Fedora
- Sponsor needed

Comment 1 Chris Tyler 2008-05-29 18:44:46 UTC

Missed the nled.txt file (documentation), updated. New URLS:

Spec URL: http://chris.tylers.info/rpm/nled.spec
SRPM URL: http://chris.tylers.info/rpm/nled-2.52-2.fc9.src.rpm

Comment 2 Andrew Overholt 2008-05-30 19:49:39 UTC

Hi Chris,

Just a few items to clean up:

- md5sums don't match:
  b3257a99cb113d57eb621a3144d9dfe9
http://cdot.senecac.on.ca/software/nled/nled_2_52_src.tgz
  8862849778bc3fd8c71bbf550e9f9aaf rpmbuild/SOURCES/nled_2_52_src.tgz
- consider using install -p to preserve timestamps
- should we add %{?_smp_mflags} to the make line to do parallel builds?  it
worked for me

Thanks to Deepak Bhole for helping me review this.

Comment 3 Chris Tyler 2008-05-30 20:21:33 UTC

Thanks Andrew, fixed all (not sure what happened with the md5sums!)--

Spec URL: http://chris.tylers.info/rpm/nled.spec
SRPM URL: http://chris.tylers.info/rpm/nled-2.52-3.fc9.src.rpm

Comment 4 Andrew Overholt 2008-05-30 20:27:55 UTC

Awesome, thanks, Chris.  APPROVED.  Don't forget to do the CVS Admin procedure now.

Comment 5 Chris Tyler 2008-05-31 03:46:40 UTC

New Package CVS Request
=======================
Package Name: nled
Short Description: An easy to use, curses-based text editor
Owners: ctyler
Branches: F-8 F-9
InitialCC: overholt
Cvsextras Commits: yes

Comment 6 Kevin Fenzi 2008-05-31 23:20:47 UTC

cvs done.

Comment 7 Fedora Update System 2008-06-01 04:11:22 UTC

nled-2.52-3.fc8 has been submitted as an update for Fedora 8

Comment 8 Fedora Update System 2008-06-01 17:33:50 UTC

nled-2.52-3.fc9 has been submitted as an update for Fedora 9

Comment 9 Fedora Update System 2008-06-03 07:28:07 UTC

nled-2.52-3.fc9 has been pushed to the Fedora 9 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update nled'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2008-4814

Comment 10 Fedora Update System 2008-06-03 07:31:57 UTC

nled-2.52-3.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2008-08-07 23:59:42 UTC

nled-2.52-3.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.