Bug 217279

Summary:	Review Request: tideEditor - Editor for Tide Constituent Database (TCD) files
Product:	[Fedora] Fedora	Reporter:	Mamoru TASAKA <mtasaka>
Component:	Package Review	Assignee:	Patrice Dumas <pertusus>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Package Reviews List <fedora-package-review>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	dave, pertusus
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2006-11-27 12:51:33 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:
Bug Depends On:	211626, 217277
Bug Blocks:	163779

Description Mamoru TASAKA 2006-11-26 12:26:36 UTC

Spec URL: http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SPECS/tideEditor.spec
SRPM URL: http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SRPMS/tideEditor-1.3.12-2.fc7.src.rpm
Description: 
tideEditor is an editor for Tide Constituent Database (TCD) files.  It
was written by Jan C. Depner but is now jointly maintained by David
Flater and Jan Depner.

Comment 1 Patrice Dumas 2006-11-26 21:45:57 UTC

* rpmlint is silent
* name right
* free software license, included
* source match upstream
f833a2b7ccd4721c719642845ec2da47  tideEditor-1.3.12.tar.bz2
* %files right

Needswork:
Are the buildrequires expat-devel, lcms-devel, libXfixes-devel
required?

Comment 2 Mamoru TASAKA 2006-11-27 02:52:44 UTC

(In reply to comment #1)
 
> Needswork:
> Are the buildrequires expat-devel, lcms-devel, libXfixes-devel
> required?

Umm.. not needed. Maybe dependency changed? Anyway I removed them.
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SPECS/tideEditor.spec
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SRPMS/tideEditor-1.3.12-3.fc7.src.rpm

Comment 3 Patrice Dumas 2006-11-27 11:00:58 UTC

Forgot to add that this is a gui app, but I think that it is not wrong
not to have a menu entry, since it requires a command line argument.
If that changes one day, and something like a filechoser is added to
tideEditor, then a desktop file will be needed.


APPROVED

Comment 4 Mamoru TASAKA 2006-11-27 12:51:33 UTC

Well,

* Rebuild for FE-devel succeeded.
* SyncNeeded is requested for FE5/6

Now I close this bug as CLOSED NEXTRELEASE. Thank you.