[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: add font review links to the PTS?



On Fri, Jan 29, 2010 at 5:26 PM, Stefano Zacchiroli <zack@debian.org> wrote:

> I confess it is not clear to me the purpose of adding this to the
> PTS. Or better, it is not clear to me in which part of the PTS you want
> that. Do you want a link in the links section like the homepage
> stuff. That would be reasonable. Otherwise, if you want it as a "TODO",
> I'm not sure which kind of TODO message you envisage (maybe I've just
> not ended up on the right pages in the current font review page).

Initially just a link in the links section. Later on when we figure
out how to parse the fontlint, otvalid etc output, possibly something
in the TODO section, probably like what is shown for lintian
complaints from lintian.d.o.

> In both cases, the steps to have integration are pretty much the
> standard ones:
>
> - Provide a complete index in a single file, giving all the information
>  the PTS would need to generate font-review-related information at each
>  pulse.
>
>  Ideally, the index should be source-package oriented, but we can leave
>  also with binary-package oriented indexes. If you want me to name a
>  file format for the index, I vote for YAML.

A binary-package oriented index would be easier, otherwise we'd need
to call dpkg a lot or rewrite the review script in something other
than shell.

> - If you just want links, the index should map packages to the relevant
>  links. That way you will be able to change the URL scheme without
>  needing interaction with us. If you want also TODO items, that
>  information should be added to, we can discuss details once it is
>  clear which kind of TODO you will need.

How about something like this? (not sure if it is valid YAML)

link:
    text: Font review
    - package: a2ps
        url: http://pkg-fonts.alioth.debian.org/review/pkg-a2ps.html
    - package: adonthell-data
        url: http://pkg-fonts.alioth.debian.org/review/adonthell-data.html
todo:
    text: The Debian font review
    url: http://pkg-fonts.alioth.debian.org/review/
    - type: fontlint
      text: fontlint complaint(s)
    - type: fontlint
      text: fontlint complaint(s)
    - package: a2ps
      fontlint: 3
    - package: adonthell-data
      fontlint: 2

Initially the todo items would not be present and we would add them later.

> Well, this seems to go in the TODO/problem directions, but if you
> already have that information I'd prefer doing the integration once and
> for all, what would be the point in diluting it?

The information for the TODO stuff is (mostly) in the review pages
already, the main issue is parsing it into a useful form for the PTS.

PS: this is something that could probably benefit from #509416 being
implemented.

-- 
bye,
pabs

http://wiki.debian.org/PaulWise


Reply to: