Bug 225770 - Merge Review: freetype
Merge Review: freetype
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Kevin Kofler
Fedora Package Reviews List
: Reopened
Depends On:
  Show dependency treegraph
Reported: 2007-01-31 13:40 EST by Nobody's working on this, feel free to take it
Modified: 2008-12-09 16:54 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-12-09 16:53:13 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
kevin: fedora‑review+

Attachments (Terms of Use)

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 13:40:09 EST
Fedora Merge Review: freetype

Initial Owner: besfahbo@redhat.com
Comment 1 Julian Sikorski 2007-02-03 11:21:48 EST
IIRC freetype has some rpath issues. I don't know how easily they can be rid of.
Comment 2 Roozbeh Pournader 2007-02-06 08:01:22 EST
Bad news! <http://fedoraproject.org/wiki/Packaging/Guidelines#Legal> says that
the license must be in one of the three lists. Since FTL is nowhere on those
lists, unless FSF reviews it or OSI approves it, we may need to consider pango
and freetype GPL.
Comment 3 Behdad Esfahbod 2007-02-06 12:17:18 EST
So we mark it as GPL.  Do you think anything will happen ever?!
Comment 4 Roozbeh Pournader 2007-02-07 06:58:51 EST
(In reply to comment #3)
> So we mark it as GPL.  Do you think anything will happen ever?!

I don't get what is the question, but marking it as GPL will remove the blocker
on review.
Comment 5 Julian Sikorski 2007-06-01 14:31:22 EDT
RPATH problem still exists in 2.3.4-3.
Comment 6 Adam Jackson 2007-06-29 16:43:56 EDT
rpath buglet fixed in 2.3.4-4.
Comment 7 Kevin Kofler 2007-09-18 04:30:44 EDT
There's still some badness left:
* The Source: tag doesn't contain the full URL. Please change this to:
(likewise for Source1 and Source2)
* The License tag should be "License: FTL or GPLv2+".
* Please ship the license files as %doc, e.g.:
%doc ChangeLog README docs/LICENSE.TXT docs/FTL.TXT docs/GPL.TXT
Comment 8 Behdad Esfahbod 2008-12-09 16:05:07 EST
The license was already fixed.

* Tue Dec 09 2008 Behdad Esfahbod <besfahbo@redhat.com> 2.3.7-3
- Add full source URL to Source lines.
- Add docs to main and devel package.
- rpmlint is happy now.
- Resolves: #225770
Comment 9 Dennis Gilmore 2008-12-09 16:24:35 EST
package is not approved, and cant be closed.
Comment 10 Kevin Kofler 2008-12-09 16:53:13 EST
The package is OK now. All the issues were addressed. I also compared it with the freetype-freeworld package which passed review at Livna.

Merge review APPROVED.

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