Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 218210

Summary: Review Request: chemical-mime-data - Support for chemical/* MIME types
Product: [Fedora] Fedora Reporter: Julian Sikorski <belegdol>
Component: Package ReviewAssignee: Mamoru TASAKA <mtasaka>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: mr.ecik
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-12-06 07:54:17 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, 218172    

Description Julian Sikorski 2006-12-03 12:42:41 UTC
Spec URL: http://www.republika.pl/belegdol/rpmstuff/chemical-mime-data.spec
SRPM URL: http://www.republika.pl/belegdol/rpmstuff/chemical-mime-data-0.1.93-1.src.rpm
Description: A collection of data files which tries to give support for various chemical MIME types (chemical/*) on Linux/UNIX desktops. Chemical MIME's have been proposed in 1995, though it seems they have never been registered with IANA.

Package builds fine inside mock. Rpmlint is silent for source rpm, but outputs the following for binary one:
E: chemical-mime-data no-binary
W: chemical-mime-data devel-file-in-non-devel-package /usr/share/pkgconfig/chemical-mime-data.pc
I think that it is harmless, and similar to gnome-mime-data package.

Comment 1 Michał Bentkowski 2006-12-03 13:10:57 UTC
Nice to see another Pole's trying to put his package into Extras :) Good luck!
As for rpmlint output, you can simple shut it up by adding "BuildArch: noarch"
in preamble.
Also, there are some missing dependencies. This package owns files in some
directories owned by other packages. Probably, you have to remove crystalsvg
icons and mimelnk as they're owned by kdelibs package so
they make unneeded kde dependency. %{_datadir}/icons/hicolor creates
hicolor-icon-theme dep; {_datadir}/mime/packages - shared-mime-info.
These are all things need to be fixed I found at first sight :)

Comment 2 Julian Sikorski 2006-12-03 13:23:40 UTC
Well, I have checked gnome-mime-data package and the comment form the spec says:
### WE ARE NOT NOARCH, /usr/lib/pkgconfig is not noarch. 
### don't change this and don't file a bug. ;-)
As for dependencies, I don't think that I understand a problem. We want chemical
MIME support for KDE as well, don't we? So shall I add kdelibs,
hicolor-icon-theme and shared-mime-info to Requires?

Comment 3 Mamoru TASAKA 2006-12-03 13:35:05 UTC
(In reply to comment #2)
> Well, I have checked gnome-mime-data package and the comment form the spec says:
> ### WE ARE NOT NOARCH, /usr/lib/pkgconfig is not noarch. 
> ### don't change this and don't file a bug. ;-)
Yes. .pc file is arch-dependent and if this package requires
this file this package canNOT be noarch.
However....
* Is this file needed?
* Anyway the directory is wrong. Currently pc file is under 
 %{_datadir}/pkgconfig, which should be %{_libdir}/pkgconfig

> As for dependencies, I don't think that I understand a problem. We want chemical
> MIME support for KDE as well, don't we? So shall I add kdelibs,
> hicolor-icon-theme and shared-mime-info to Requires?
Owning the directory:
* which is owned by other packages
* however, all the packages which own the directory is not needed
  for the package
is _NOT_ prohibited, and in that case it is the package which
should also own the directory.



Comment 4 Mamoru TASAKA 2006-12-03 13:37:21 UTC
By the way I have not yet checked this package...

Comment 5 Julian Sikorski 2006-12-03 13:43:05 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > Well, I have checked gnome-mime-data package and the comment form the spec says:
> > ### WE ARE NOT NOARCH, /usr/lib/pkgconfig is not noarch. 
> > ### don't change this and don't file a bug. ;-)
> Yes. .pc file is arch-dependent and if this package requires
> this file this package canNOT be noarch.
> However....
> * Is this file needed?
> * Anyway the directory is wrong. Currently pc file is under 
>  %{_datadir}/pkgconfig, which should be %{_libdir}/pkgconfig
> 

Strange, but that's where it got installed by default. I have some files in
%{_datadir}/pkgconfig here: xml2po.pc, gnome, icon-theme.pc and a few others. I
can probably fix it to install pkgconfig file into %{_libdir}, though.

Comment 6 Michał Bentkowski 2006-12-03 13:52:39 UTC
(In reply to comment #3)
> Owning the directory:
> * which is owned by other packages
> * however, all the packages which own the directory is not needed
>   for the package
> is _NOT_ prohibited, and in that case it is the package which
> should also own the directory.

Thus chemical-mime-data should also own a %{_datadir}/mimelnk dir. If it doesn't
then removing it will leave empty %{_datadir}/mimelnk dir unowned. I believe
that hicolor-icon-theme and shared-mime-info should be required by this package
nevertheless. A lot of packages own hicolor-icon-theme and it isn't a problem.
Also, shared-mime-info looks like a package related with chemical-mime-data
so requiring it may be necessary.


Comment 7 Mamoru TASAKA 2006-12-03 15:06:35 UTC
Well, for directory ownership issue:

/usr/share/icons/crystalsvg/
- (Under my envoronment) only owned by kdelibs.
  I think this package should own this
  (The subdirectories should also)

- /usr/share/icons/hicolor/
  For this directory, adding hicolor-icon-theme to Requires
  is a simplest solution as MichaÅ commented 
  (sorry, however the last character of your first name
  is corrupted in my browser)

- /usr/share/mime
  Requiring shared-mime-info should be the simplest.

- /usr/share/mimelnk/
  Only owned by kdelibs and Ooo related packages.
  This directory should be owned by this package.

Comment 8 Julian Sikorski 2006-12-03 19:36:27 UTC
Spec URL: http://www.republika.pl/belegdol/rpmstuff/chemical-mime-data.spec
SRPM URL:
http://www.republika.pl/belegdol/rpmstuff/chemical-mime-data-0.1.93-2.src.rpm
New release:
- Fixed ownership
- Fixed .pc file location

I'm not sure I've fixed the ownership issues correctly, please advise.


Comment 9 Mamoru TASAKA 2006-12-04 07:12:25 UTC
(In reply to comment #7)
> - /usr/share/mime
>   Requiring shared-mime-info should be the simplest.

Oh, requiring shared-mime-info is _mandatory_ as .pc file
requires it.

(In reply to comment #8)
> I'm not sure I've fixed the ownership issues correctly, please advise.
0.1.93-2 is okay.

Well, then
* %doc INSTALL
- This is not necessary because it is required for installing
  this package by himself.

Other things are okay.
-----------------------------------------------------
  This package (chemical-mime-data) is APPROVED by me.
-----------------------------------------------------

Please go forward according to 

http://fedoraproject.org/wiki/Extras/Contributors

to import this package to Fedora Extras. I will sponsor
you when you have taken steps partway written in the page above
(then I should receive the mail that you need a sponsor)


Comment 10 Julian Sikorski 2006-12-04 07:37:42 UTC
Spec URL: http://www.republika.pl/belegdol/rpmstuff/chemical-mime-data.spec
SRPM URL:
http://www.republika.pl/belegdol/rpmstuff/chemical-mime-data-0.1.93-3.src.rpm
New release:
- Removed INSTALL from %doc

Thank you for sponsorship. I'll create the account now.

Comment 11 Mamoru TASAKA 2006-12-05 01:55:02 UTC
Removing FE-NEEDSPONSOR, now I am sponsoring.

Comment 12 Julian Sikorski 2006-12-05 08:19:20 UTC
Package imported, FC-5 and FC-6 branches requested.

Comment 13 Mamoru TASAKA 2006-12-05 11:35:51 UTC
You can close this bug as CLOSED NEXTRELEASE when you 
succeeds in rebuilding this on FE-devel.