Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 649510 - media-fonts/glass-tty-vt220: TrueType VT220 Font
Summary: media-fonts/glass-tty-vt220: TrueType VT220 Font
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Default Assignee for New Packages
URL: http://sensi.org/~svo/glasstty/
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2018-03-04 01:02 UTC by Conrad Kostecki
Modified: 2018-04-11 23:08 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
glass-tty-vt220-001.3.ebuild (glass-tty-vt220-001.3.ebuild,608 bytes, text/plain)
2018-03-04 01:02 UTC, Conrad Kostecki
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Conrad Kostecki gentoo-dev 2018-03-04 01:02:33 UTC
media-fonts/glass-tty-vt220: add new ebuild
Comment 1 Conrad Kostecki gentoo-dev 2018-03-04 01:02:51 UTC
Created attachment 522230 [details]
glass-tty-vt220-001.3.ebuild
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2018-03-06 08:25:24 UTC
Comment on attachment 522230 [details]
glass-tty-vt220-001.3.ebuild

># Copyright 1999-2018 Gentoo Foundation
># Distributed under the terms of the GNU General Public License v2
>
>EAPI="6"
>
>inherit font
>
>DESCRIPTION="Hacking with Style: TrueType VT220 Font"
>HOMEPAGE="http://sensi.org/~svo/glasstty/"
>SRC_URI="http://sensi.org/~svo/glasstty/Glass_TTY_VT220.ttf"

That should probably have a version or you will run into trouble later when the file is changed, as that trips up the Manifest checks.

>KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~mips ~ppc ~ppc64 ~s390 ~sh ~sparc ~x86"

I am quite sure you didn't test on all those platforms. If you did, then that's fine, but please make a note of it.

>LICENSE="freedist"
>SLOT="0"
>IUSE=""

No need to set an empty IUSE.

>RESTRICT="strip"

Should not be needed as there is nothing to strip. Why did you add it?
Comment 3 Conrad Kostecki gentoo-dev 2018-03-06 10:11:42 UTC
Hi Jeroen,
thank you very much for your review.

(In reply to Jeroen Roovers from comment #2)
> Comment on attachment 522230 [details]
> >SRC_URI="http://sensi.org/~svo/glasstty/Glass_TTY_VT220.ttf"
> 
> That should probably have a version or you will run into trouble later when
> the file is changed, as that trips up the Manifest checks.

Is there something I can do? The Website is not mine. They only offer the *.ttf directly. When opening the *.ttf, it shows 001.3 as version.

But I can say, the the last 8? years, the file hasn't changed. Or should I create an archive?
 
> >KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~mips ~ppc ~ppc64 ~s390 ~sh ~sparc ~x86"
> 
> I am quite sure you didn't test on all those platforms. If you did, then
> that's fine, but please make a note of it.

My intention was, that any *.ttf should work on any arch? Anyway, I've changed it to ~x86, ~amd64, ~arm and ~arm64, as I have tested on this arches only.

> >LICENSE="freedist"
> >SLOT="0"
> >IUSE=""
> 
> No need to set an empty IUSE.

Okay, removed now.

> >RESTRICT="strip"
> 
> Should not be needed as there is nothing to strip. Why did you add it?

This was a misinterpretation by myself. I've removed it now.

Thanks!
Conrad
Comment 4 Larry the Git Cow gentoo-dev 2018-04-11 23:08:52 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0bd820db8d562ce297e0779bb9132794b8619a6e

commit 0bd820db8d562ce297e0779bb9132794b8619a6e
Author:     Conrad Kostecki <conrad@kostecki.com>
AuthorDate: 2018-03-26 13:31:52 +0000
Commit:     Patrice Clement <monsieurp@gentoo.org>
CommitDate: 2018-04-11 23:07:46 +0000

    media-fonts/glass-tty-vt220: new package.
    
    Closes: https://bugs.gentoo.org/649510
    Closes: https://github.com/gentoo/gentoo/pull/7347

 media-fonts/glass-tty-vt220/Manifest               |  1 +
 .../glass-tty-vt220/glass-tty-vt220-001.3.ebuild   | 29 ++++++++++++++++++++++
 media-fonts/glass-tty-vt220/metadata.xml           | 12 +++++++++
 3 files changed, 42 insertions(+)