Bug 166054 - Review Request: gtkmathview - Addition of gtkmathview to Extras
Review Request: gtkmathview - Addition of gtkmathview to Extras
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Gérard Milmeister
David Lawrence
http://www.abisource.com/~uwog/gtkmat...
:
Depends On:
Blocks: FE-ACCEPT
  Show dependency treegraph
 
Reported: 2005-08-16 09:02 EDT by Marc Maurer
Modified: 2007-11-30 17:11 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2005-08-30 18:50:04 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Marc Maurer 2005-08-16 09:02:13 EDT
Spec Name or Url:

http://www.abisource.com/~uwog/gtkmathview/gtkmathview.spec

SRPM Name or Url:

http://www.abisource.com/~uwog/gtkmathview/gtkmathview-0.7.3-1.src.rpm

Description:

gtkmathview is a MathML rendering library (the "gtk" in its name is kinda outdated, and is does not strictly depend on gtk anymore; gtk is just one of its backends). This will be a dependency for AbiWord 2.4, which will use it to render equations.
Comment 1 Gérard Milmeister 2005-08-22 14:22:37 EDT
* Missing BuildRequires:
  At least gtk2-devel, t1lib-devel, zlib-devel, maybe a few others
* If development files from other packages are needed to build using
  gtkmathview-devel, they should be indicated in the Requires: tag
  of gtkmathview-devel, e.g. Requires: gtk2-devel
* What about gmetadom?
* There are manpages for mathml2ps and mathmlviewer but no binaries, do not
  include them
* %doc should include CONTRIBUTORS BUGS LICENSE 
* The Description: section for the main package should be more extended IMHO
Comment 2 Marc Maurer 2005-08-23 18:17:48 EDT
Thanks Gérard... I don't know why I missed that all; must have been tired :-/
Your suggestions found their way in a new spec:

  http://www.abisource.com/~uwog/gtkmathview/gtkmathview.spec

SRPM available here:

  http://www.abisource.com/~uwog/gtkmathview/gtkmathview-0.7.3-2.src.rpm
Comment 3 Rex Dieter 2005-08-25 14:03:30 EDT
Drop the explicit Requires:
Requires: glib2 >= 2.2.1
Requires: gtk2 >= 2.2.1
Requires: libxml2 >= 2.6.7
Requires: popt >= 1.7.0
Comment 4 Gérard Milmeister 2005-08-25 14:16:16 EDT
The Description is one long line. Insert some line breaks to make the line
length at most 70.
You don't need "--disable-gmetadom --without-t1lib" if it builds without them.
However the question is: is the functionality reducted because of missing
gmetadom and t1lib. If yes then maybe packages for both these libraries should
be created.
The manpage /usr/share/man/man1/mathmlviewer.1.gz needs also to be removed.
Comment 5 Marc Maurer 2005-08-29 15:34:55 EDT
OK, thesis is done, time to get back on this :-)

@Rex Dieter: why would one want to drop the Requires?

@Gérard: Description is fixed now. The GMetaDom is (iirc) very experimental
right now, is I think it is best to disable it. I'll ask the maintainer for
details. t1lib is only needed to load Type1 fonts, but all important fonts (the
Computer Modern familily) are available as True Type Fonts. I'll invesitgate the
need for Type 1 fonts, we can always enable it if we want t1lib in Extras as
well. I dropped the -disable-gmetadom --without-t1lib options, it builds fine
without them.

Deleted the mathmlviewer.1 manpage too... I thought I built the accompanying
binary, but I was mistaken it with mathmlsvg :-/

New spec: http://www.abisource.com/~uwog/gtkmathview/gtkmathview.spec
New SRPM: http://www.abisource.com/~uwog/gtkmathview/gtkmathview-0.7.3-3.src.rpm

Comment 6 Rex Dieter 2005-08-29 16:21:15 EDT
> @Rex Dieter: why would one want to drop the Requires?

Because they're not needed.  rpm will autodetect these.


Comment 7 Marc Maurer 2005-08-29 17:17:41 EDT
But that assumes that the proper versions are in rawhide. Now i know this is the
case, but I love extra security... But i can drop them if you insist/
Comment 8 Gérard Milmeister 2005-08-29 19:18:26 EDT
IIRC, freetype also handles type 1 fonts, so t1lib isn't necessary.

Explicit Requires: on libraries such as gtk+ is *strongly* discouraged.
It causes more problems than it solves.
If a package builds on a platform, but then requires a specific version of gtk2,
something is wrong and should be fixed.
Comment 9 Marc Maurer 2005-08-29 20:54:19 EDT
Well, additional functions have been added to every stable gtk2 release, so
requiring >= 2.2 is really needed. Is that allowed?
Comment 10 Gérard Milmeister 2005-08-30 05:26:17 EDT
If a package builds on, say, rawhide, the resulting rpm will have the correct
requires for rawhide, namely libgtk-x11-2.0.so.0, etc.
So it will be installed on exactly the same configuration, that it is built on.
If someone installs on a different platform (fedora, or not) that has
libgtk-x11-2.0.so.0, but is not the exact required version, than it is his
on responsability, and is not supported at all.
It may happen, that an update to, say, gtk2 breaks the package dependent on it.
But then the package must be rebuilt as soon as possible.
The version of a package is always the version of latest update.
Comment 11 Michael Schwendt 2005-08-30 05:38:54 EDT
> Is that allowed?

It's overly pedantic and ties your main binary package against a list
of specific package names and versions. Makes it more complicated to
move a needed library into a different package without rebuilding your
one as well. Also, the question ought to be "Does it make sense?",
because Gtk+ >= 2.2 and GLib >= 2.2.1 are available for all the Fedora Core
versions you target. Finally, do you want to guarantee that your binaries
really work with any package named gtk2-2.2.1-something even if you
built it for gtk2-2.8.0-something?

There are special cases when an explicit dependency on package names and
versions makes sense in addition to an automatic dependency on SONAMEs.
Comment 12 Marc Maurer 2005-08-30 09:19:10 EDT
Thanks for the explanations. New spec and SRPM here: 
 
Spec: http://www.abisource.com/~uwog/gtkmathview/gtkmathview.spec 
SRPM: http://www.abisource.com/~uwog/gtkmathview/gtkmathview-0.7.3-4.src.rpm 
 
Comment 13 Rex Dieter 2005-08-30 09:21:17 EDT
Still includes uneeded:
Requires: glib2
Requires: gtk2
Requires: libxml2 
Comment 14 Marc Maurer 2005-08-30 09:59:20 EDT
Sorry about that. Apparently the automation is further along than I expected in the rpm 
build world :) 
 
Spec: http://www.abisource.com/~uwog/gtkmathview/gtkmathview.spec  
SRPM: http://www.abisource.com/~uwog/gtkmathview/gtkmathview-0.7.3-5.src.rpm  
Comment 15 Gérard Milmeister 2005-08-30 10:43:30 EDT
Marc, have you got an account?
If not, you should go to https://admin.fedora.redhat.com/accounts.
I think you also need a sponsor.
Comment 16 Marc Maurer 2005-08-30 11:12:15 EDT
Yes, I'm a fedora extras maintainer already (abiword,link-grammar,aiksaurus, etc)

Does that mean the spec is approved? :)
Comment 17 Gérard Milmeister 2005-08-30 15:38:45 EDT
Everything looks allright to me.
Comment 18 Christian Iseli 2006-10-18 05:25:13 EDT
Normalize summary field for easy parsing

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