Good news is that we’re still in time for the change. Bad news is that there’s no time for a real ‘community vote’ on this.
I also liked the first option better but since Ole and Jon pushed for the change, I went with what they liked. Does anyone object to using the first option? Thanks, Florin > On Oct 25, 2017, at 3:56 PM, Luke, Chris <chris_l...@comcast.com> wrote: > > Hrm, I personally feel the breakdown by file to be more useful than who made > the change. > > Chris > >> -----Original Message----- >> From: vpp-dev-boun...@lists.fd.io [mailto:vpp-dev-boun...@lists.fd.io] On >> Behalf Of Florin Coras >> Sent: Wednesday, October 25, 2017 1:58 PM >> To: Ole Troan <otr...@employees.org> >> Cc: vpp-dev <vpp-dev@lists.fd.io> >> Subject: Re: [vpp-dev] 17.10 draft release notes >> >> Updated release notes to use the format Jon proposed. >> >> Florin >> >>> On Oct 25, 2017, at 9:11 AM, Ole Troan <otr...@employees.org> wrote: >>> >>> Thanks Jon, love that! >>> >>> So something like: >>> git shortlog --format="* [%h] %s" v17.07..v17.10-rc2 -- *.api >>> >>> Cheers, >>> Ole >>> >>> >>>> On 25 Oct 2017, at 07:13, Jon Loeliger <j...@netgate.com> wrote: >>>> >>>> On Wed, Oct 25, 2017 at 7:28 AM, Ole Troan <otr...@employees.org> >> wrote: >>>> Hi Florin, >>>> >>>> Hi Ole, >>>> >>>> What about something like this? >>>> >>>> #!/usr/bin/env python >>>> import os, fnmatch, subprocess >>>> starttag = 'v17.07' >>>> endtag = 'v17.10-rc2' >>>> apifiles = [] >>>> for root, dirnames, filenames in os.walk('.'): >>>> for filename in fnmatch.filter(filenames, '*.api'): >>>> apifiles.append(os.path.join(root, filename)) for f in >>>> apifiles: >>>> commits = subprocess.check_output(['git', 'log', >>>> '--oneline', starttag + '..' + endtag, >>>> f]) >>>> if commits: >>>> print f >>>> print commits >>>> >>>> >>>> Just as an FYI, Git might be able to help simplify some of this >>>> script for you too. For example: >>>> >>>> $ git ls-files *.api >>>> src/examples/sample-plugin/sample/sample.api >>>> src/plugins/acl/acl.api >>>> src/plugins/dpdk/api/dpdk.api >>>> src/plugins/flowprobe/flowprobe.api >>>> src/plugins/gtpu/gtpu.api >>>> <snip> >>>> >>>> Depending on development at HEAD, you may want: >>>> $ git ls-files v17.10-rc2 *.api >>>> or so. >>>> >>>> Then, if you are not too concerned about the individual break-down by >>>> file, you can one-line it: >>>> >>>> $ git log --oneline v17.07..v17.10-rc2 -- *.api >>>> 50570ec Update of free text tag patch for BD >>>> 831fb59 Stats refactor >>>> 2297af0 Add a name to the creation of an IP and MPLS table >>>> c29940c ACL-plugin add "replace" semantics for adding a new MacIP acl >>>> 8a19f12 Allow individual stats API and introduce stats.api - >> want_interface_simple_stats - want_interface_combined_stats - >> want_ip4|6_fib|nbr_stats >>>> d630713 LISP: add neighbor discovery and CP protocol separation APIs >>>> 111a5ce LISP: Add APIs for enable/disable xTR/P-ITR/P-ETR modes >>>> 4802632 Punt socket: Fix coverity error for pathname length mismatch >> between API and sun_path. >>>> 33e002b Fix session connect_* api message handling. >>>> <snip> >>>> >>>> Also, the "shortlog" command is tailored to producing release notes: >>>> >>>> $ git shortlog v17.07..v17.10-rc2 -- *.api >>>> Dave Barach (1): >>>> TCP source address automation >>>> >>>> Dave Wallace (1): >>>> Fix session connect_* api message handling. >>>> >>>> Eyal Bari (3): >>>> API:fix arp/ND event messages - remove context >>>> SPAN:add l2 mirror >>>> SPAN/API:enable L2 dump >>>> >>>> Filip Tehlar (4): >>>> LISP: make TTL for map register messages configurable >>>> LISP: Map-server fallback feature >>>> LISP: Add APIs for enable/disable xTR/P-ITR/P-ETR modes >>>> LISP: add neighbor discovery and CP protocol separation APIs >>>> <snip> >>>> >>>> HTH, >>>> jdl >>> >> >> _______________________________________________ >> vpp-dev mailing list >> vpp-dev@lists.fd.io >> https://lists.fd.io/mailman/listinfo/vpp-dev > _______________________________________________ vpp-dev mailing list vpp-dev@lists.fd.io https://lists.fd.io/mailman/listinfo/vpp-dev
