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

Bug#786470: debian-policy: [copyright-format] Add an optional “License-Grant” field



Thank you for the new patch, Ben.

Comments:

- please move the License-Grant field to the end of the list of fields
  to avoid any renumbering

- please strip the rewordings that you've made to other parts of the
  document, so this patch addresses this bug alone.

  If you want to make those changes, please file separate 'informative'
  bugs against debian-policy; these do not need to be seconded and are
  applied at the discretion of the policy editors.

- I think you've used too many spaces to indent the License-Grant text
  in the examples.  Shouldn't it be just one space?

- since the field is optional, I don't think we should modify every
  example to include License-Grant.  Perhaps consider reducing the
  number of times you add it?

- note that this bug doesn't apply to policy's HEAD anymore because
  we've switched to rST; it would be nice if you could rebase but I will
  happily do this on your behalf as I was partly responsible for the
  switch

-- 
Sean Whitton

Attachment: signature.asc
Description: PGP signature


Reply to: