Bug 302451 - Review Request: baekmuk-bdf-fonts - Korean bitmap fonts (separated from fonts-korean)
Summary: Review Request: baekmuk-bdf-fonts - Korean bitmap fonts (separated from fonts...
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review   
(Show other bugs)
Version: rawhide
Hardware: All Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Keywords:
Depends On: 300651
Blocks: 253155
TreeView+ depends on / blocked
 
Reported: 2007-09-24 01:29 UTC by Jens Petersen
Modified: 2010-01-13 06:38 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-09-24 12:44:18 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
mtasaka: fedora-review+
petersen: fedora-cvs+


Attachments (Terms of Use)

Description Jens Petersen 2007-09-24 01:29:36 UTC
Spec URL: http://petersen.fedorapeople.org/baekmuk-bdf/baekmuk-bdf-fonts.spec
SRPM URL: http://petersen.fedorapeople.org/baekmuk-bdf/baekmuk-bdf-fonts-2.2-1.fc7.src.rpm
Description: Free Korean bitmap fonts

This package is being split out of fonts-korean like baekmuk-ttf-fonts.

Comment 1 Jens Petersen 2007-09-24 01:33:04 UTC
(baekmuk-ttf-fonts review is bug 300651.)

Comment 2 Mamoru TASAKA 2007-09-24 06:31:49 UTC
* Similar with bug 300651, COPYRIGHT.ks can be converted to UTF-8.
* Well, while this package uses relative symlink on
  %_sysconfdir/X11/fontpath.d, bug 300651 uses absolute symlink.
  Would you unify them? (for these symlinks, all other packages
  seem to use absolute symlinks)
* %ghost %verify(not md5 size mtime) files 
  "fonts.cache-1" "encodings.dir" are no longer needed?

Comment 3 Mamoru TASAKA 2007-09-24 07:08:14 UTC
* Also,
  Similar with bug 300651, "Conflicts: fonts-korean <= <somever>" is better
  IMO

... By the way koji seems down

Comment 4 Jens Petersen 2007-09-24 07:29:20 UTC
> * Similar with bug 300651, COPYRIGHT.ks can be converted to UTF-8.

Yes, thanks.

> * Well, while this package uses relative symlink on
>   %_sysconfdir/X11/fontpath.d, bug 300651 uses absolute symlink.
>   Would you unify them? (for these symlinks, all other packages
>   seem to use absolute symlinks)

Actually now I made it relative in baekmuk-ttf-fonts now too, but if 
it is a bad idea I can revert.

> * %ghost %verify(not md5 size mtime) files 
>   "fonts.cache-1" "encodings.dir" are no longer needed?

Yes, I think they can go.

(In reply to comment #3)
>   Similar with bug 300651, "Conflicts: fonts-korean <= <somever>" is better

Hmm, maybe it makes sense, though I think this is no actual conflict
of files.

> ... By the way koji seems down

Seems to be mostly back now, I think. :)


Spec URL:
http://people.redhat.com/petersen/reviews/baekmuk-bdf/baekmuk-bdf-fonts.spec
SRPM URL:
http://people.redhat.com/petersen/reviews/baekmuk-bdf/baekmuk-bdf-fonts-2.2-2.fc7.src.rpm


Comment 5 Mamoru TASAKA 2007-09-24 08:06:58 UTC
(In reply to comment #4)
> > * Well, while this package uses relative symlink on
> >   %_sysconfdir/X11/fontpath.d, bug 300651 uses absolute symlink.
> >   Would you unify them? (for these symlinks, all other packages
> >   seem to use absolute symlinks)
> 
> Actually now I made it relative in baekmuk-ttf-fonts now too, but if 
> it is a bad idea I can revert.
  - I don't know which is better (although I prefer relative).

> 
> > * %ghost %verify(not md5 size mtime) files 
> >   "fonts.cache-1" "encodings.dir" are no longer needed?
> 
> Yes, I think they can go.
  Thanks.

> 
> (In reply to comment #3)
> >   Similar with bug 300651, "Conflicts: fonts-korean <= <somever>" is better
> 
> Hmm, maybe it makes sense, though I think this is no actual conflict
> of files.
  - Yes, there is no actual conflict, however baekmuk-bdf and old
    fonts-korean rpms owns same files and marking conflicts is better
    (IMO)

Comment 6 Jens Petersen 2007-09-24 08:29:12 UTC
>   - Yes, there is no actual conflict, however baekmuk-bdf and old
>     fonts-korean rpms owns same files and marking conflicts is better
>     (IMO)

Ok, thanks, added.

Spec URL:
http://people.redhat.com/petersen/reviews/baekmuk-bdf/baekmuk-bdf-fonts.spec
SRPM URL:
http://people.redhat.com/petersen/reviews/baekmuk-bdf/baekmuk-bdf-fonts-2.2-3.fc7.src.rpm


Comment 7 Mamoru TASAKA 2007-09-24 08:36:34 UTC
Okay.

----------------------------------------------------------
   This package (baekmuk-bdf-fonts) is APPROVED by me
----------------------------------------------------------

Comment 8 Parag AN(पराग) 2007-09-24 08:44:50 UTC
Thanks for quick review.

Comment 9 Jens Petersen 2007-09-24 08:46:50 UTC
Thanks.

New Package CVS Request
=======================
Package Name: baekmuk-bdf-fonts
Short Description: Korean bitmap fonts
Owners: cchance
Branches: devel
InitialCC: petersen
Cvsextras Commits: yes

Comment 10 Jens Petersen 2007-09-24 09:28:58 UTC
cvsadmin done

Comment 11 Jens Petersen 2007-09-24 12:44:18 UTC
Package has been imported and built.

Thank you very much.


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