On Thu, Nov 13, 2014 at 06:30:45PM -0500, Jason Pleau wrote: > Thanks for your feedback on this. I know how time consuming it can be > to review someone else's code when they're new to a project, so please > know I do appreciate it ! Likewise! :) > The logical thing would be to create a separate table to allow the > association between a suite and its aliases. I think it would ease the > rest of the code since we'd let SQLAlchemy handle creating and saving > the 'aliases' array. I agree that would be the best design. I just didn't want to impose it on you, but if you're motivated for changing the SQLAlchemy accordingly, by all means go for it! > Agreed about having the suites and their aliases auto-updated. > > I remember we talked about it a bit on #debian-qa with pabs, there was > the possibility to use an API once it's finished: > https://api.ftp-master.debian.org/suites I think there are two separate concerns here. One is automating the maintenance of Debsources configuration information, even in presence of bugs that forces to hardcode release names (e.g., #714726). This concern could be addressed using the API you mention. A different concern is interpreting information that we already have in the mirrors anyhow (like suite "aliases") and injecting it in Debsources database. This, I believe, is the concern that is at stake in this bug report. There is no need of the ftp-master.d.o API to address it. > If the API is not close to release, I can look into parsing Release > files, are you aware of a parsing tool in python? I was looking at > python-apt, but it seems to rely on a sources.list file, I don't think > that's not what we need. Debsources already depends on python-debian, which has support for parsing all sort of Deb822-like files, including Release files. The class you want for that is debian.deb822.Release. Cheers. -- Stefano Zacchiroli . . . . . . . zack@upsilon.cc . . . . o . . . o . o Maître de conférences . . . . . http://upsilon.cc/zack . . . o . . . o o Former Debian Project Leader . . @zack on identi.ca . . o o o . . . o . « the first rule of tautology club is the first rule of tautology club »
Attachment:
signature.asc
Description: Digital signature