Am Mittwoch, den 05.06.2013, 15:15 -0400 schrieb Daniel Kahn Gillmor: > [i'm not on the devscripts-devel list, please cc me or 610...@bugs.debian.org] > > On Sat 2013-05-04 05:26:55 -0400, Daniel Kahn Gillmor wrote: > > > On Sat 2013-05-04 05:03:36 -0400, Daniel Kahn Gillmor wrote: > > > >> The attached patch implements the above proposal, using (e.g.) > >> opts=pgpsigurlmangle=s/$/.asc/ and debian/upstream-signing-key.pgp. > > > > This time with the patch actually attached :/ > > Any thoughts on this patch?
I took a quick look at the patch and found to things: 1) Please update README in addition to d/control and 2) please check the indentation. The script uses a obscure mixture of tabs and spaces as indentation. You might want to add a test to test/test_uscan. > As a DD, I'm a member of collab-maint -- > should i just go ahead and commit it to the devscripts repo? It works > for me, and i would like to start using it for debian packages that have > an upstream that signs releases. I vote for "go ahead". Welcome in the devscripts devel team! > If i don't hear any objections, i'll probably commit it (plus a > an update to debian/changelog) in a day or two. Nice way to trigger a quick response. :) -- Benjamin Drung Debian & Ubuntu Developer -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
