Bug 186132

Summary:	Review Request: openoffice.org-dict-cs_CZ
Product:	[Fedora] Fedora	Reporter:	Tomas Mraz <tmraz>
Component:	Package Review	Assignee:	Radek Vokál <rvokal>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Package Reviews List <fedora-package-review>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	caolanm
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-03-30 09:39:01 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:
Bug Blocks:	163779

Description Tomas Mraz 2006-03-21 20:00:46 UTC

Spec Name or Url: 
http://people.redhat.com/tmraz/ooo/openoffice.org-dict-cs_CZ.spec
SRPM Name or Url:
http://people.redhat.com/tmraz/ooo/openoffice.org-dict-cs_CZ-20060303-1.src.rpm
Description: 
This package contains the czech spellchecker and hyphenation dictionaries for
the Openoffice.org application suite.

Please review. rpmlint gives an error about no binary in usr/lib but I think this can be ignored.

Comment 1 Radek Vokál 2006-03-22 09:57:57 UTC

REVIEW
 * builds fine on FC5
 * installs and works with openoffice.org-writer-2.0.2-5.2.2.i386
 * RPM name is ok
 * there's no binary, just data (mock test skipped)
  - missing binary makes sense

Comment 2 Caolan McNamara 2006-03-22 10:15:41 UTC

err, how about I actually put these czech dictionaries directly into our core
openoffice.org rpm cs_CZ langpack rpm ?

I'll be doing an update for FC5 OOo at some stage, so if you want (and reassign
it to me and OOo component) I can achieve this in the OOo package instead of an
extra rpm ?

Comment 3 Radek Vokál 2006-03-22 11:34:22 UTC

As I understand, shipping cs_CZ dictionaries is not possible inside the langpack
due to licencing problems.

Comment 4 Caolan McNamara 2006-03-22 11:41:58 UTC

I see, they're GPL while OOo is LGPL. Fair enough.

Comment 5 Radek Vokál 2006-03-27 12:48:59 UTC

ACCEPT

Comment 6 Tomas Mraz 2006-03-30 09:39:01 UTC

Package built for devel, FC-5, FC-4.