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

Bug#582038: qa.debian.org: [PTS] Please link to UbuntuDiff



On 18/05/10 at 11:29 +0200, Mehdi Dogguy wrote:
> On 18/05/2010 09:52, Lucas Nussbaum wrote:
> > On 18/05/10 at 09:04 +0200, Stefano Zacchiroli wrote:
> >> PS as we already have some ubuntu information in the PTS, I'm copying
> >>    Lucas which IIRC contributed that other information: can you please
> >>    you two check that there is no overlapping in having both links?
> > 
> > I'm not entirely convinced by the approach. What value does it add *on
> > the PTS* compared to what is already available in the Ubuntu box?
> > Wouldn't it be better to work with Ubuntu on a better patches.u.c?
> 
> Well… I'not convinced by the current approach :) For me, it's way simpler
> to read some lines of a changelog containing exactly the information I'm
> looking for, instead of digging into a big patch with a lot of noise.
> 
> Honestly, I consider the current "patches for …" link near to useless. I
> think that maintainers (at least, me) don't want to waste too much time to
> know important changes done in Ubuntu. The current "Ubuntu" box tell me
> (in a readable and quick way) only the Ubuntu version, but doesn't list
> the changes. It provides a patch, but I won't read every debdiff patch for
> every package I want to inspect because it's a PITA. IMO, "UbuntuDiff"
> answers that need. I also have in mind to add¹ some features like: a
> simple patch viewer, list of packages in Ubuntu and not in Debian, etc…


UbuntuDiff does 3 things:
A) aggregate data about a set of packages selected by grep-dctrl
   => orthogonal to what the PTS does
B) provide links to some useful resources about the package
   => already done by the PTS
C) extract the changelog from the diff
   => that's the main point we are interested in

While I agree that (C) is a nice feature, it could go much further,
providing direct links to the patches in debian/patches, extracting
their description, etc. That's similar to what patch-tracker.debian.org
aims at doing. The problem is that we need a good script that parses the
diff and extracts the useful information.
That script could be used on patch-tracker.debian.org and patches.u.c.

Now, if we stuck to the changelog, it would be better to provide it as a
box that appears when the mouse cursor is over the Ubuntu patch (using
some cool javascript). You don't really need a separate service for that.
-- 
| Lucas Nussbaum
| lucas@lucas-nussbaum.net   http://www.lucas-nussbaum.net/ |
| jabber: lucas@nussbaum.fr             GPG: 1024D/023B3F4F |



Reply to: