Hi, On Wed, Nov 29, 2017 at 08:25:49PM +0100, Salvatore Bonaccorso wrote: > Hi Guido, > > On Wed, Nov 29, 2017 at 04:49:41PM +0100, Guido Günther wrote: > > Hi, > > On Wed, Nov 29, 2017 at 04:24:54PM +0100, Salvatore Bonaccorso wrote: > > > Hi Guido, > > > > > > On Wed, Nov 29, 2017 at 01:48:02PM +0100, Guido Günther wrote: > > > > Address the bts already and put the CVEs in the subject. > > > > > > > > --- > > > > This can be further improved regards temp id handling, providing a > > > > better subject in case of only a single CVE, etc. but already makes like > > > > simpler. O.k. to apply? > > > > > > Looks fine as long the -m remains the non-default for now and > > > bin/report-vuln just can be used to generate templates as before. > > > > Withou '-m' the output is as it was before. > > Ack!
Pushed now. Thanks. -- Guido > > > > > > > Would it be possible though to change the generated subject to include > > > the source package name (or binary package name, depending on the > > > mode) as provided, and append after a : just the CVEs space separated? > > > > > > That is rather than > > > > > > Subject: CVE-id1, CVE-id2, ... > > > > > > more like > > > > > > Subject: srcpkg: CVE-id1 CVE-id2 ... > > > > Attached patch does just that. > > Looks good to me, so if you don't hear any objection please go ahead > as well with that third change. > > Regards, > Salvatore
