Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(13185)

Issue 101000043: Update debian/{changelog,control,copyright} to standards compliance. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 11 months ago by roubert (google)
Modified:
9 years, 10 months ago
Reviewers:
daniel2
Base URL:
https://libphonenumber.googlecode.com/svn/trunk
Visibility:
Public.

Description

Update debian/{changelog,control,copyright} to standards compliance. R=daniel@pocock.pro BUG= Committed: https://code.google.com/p/libphonenumber/source/detail?r=670

Patch Set 1 #

Total comments: 3

Patch Set 2 : Renamed packages. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -427 lines) Patch
M debian/changelog View 1 chunk +3 lines, -394 lines 0 comments Download
M debian/control View 1 3 chunks +7 lines, -3 lines 0 comments Download
M debian/copyright View 1 chunk +19 lines, -28 lines 0 comments Download
D debian/libphonenumber-dev.install View 1 1 chunk +0 lines, -3 lines 0 comments Download
D debian/libphonenumber-java.jlibs View 1 1 chunk +0 lines, -1 line 0 comments Download
A + debian/libphonenumber6-dev.install View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + debian/libphonenumber6-java.jlibs View 1 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 7
roubert
This is based on your https://github.com/dpocock/libphonenumber/tree/debian branch, but I've edited it a bit so please ...
9 years, 11 months ago (2014-06-04 10:42:12 UTC) #1
roubert (google)
Ping?
9 years, 10 months ago (2014-06-10 08:32:51 UTC) #2
daniel2
https://codereview.appspot.com/101000043/diff/1/debian/control File debian/control (right): https://codereview.appspot.com/101000043/diff/1/debian/control#newcode3 debian/control:3: Maintainer: Debian Java Maintainers <pkg-java-maintainers@lists.alioth.debian.org> Group maintainership is usually ...
9 years, 10 months ago (2014-06-12 15:22:58 UTC) #3
roubert (google)
On 2014/06/12 15:22:58, daniel2 wrote: > https://www.debian.org/doc/debian-policy/ch-sharedlibs.html#s-sharedlibs-dev > > As explained in s8.4, it is ...
9 years, 10 months ago (2014-06-13 09:59:52 UTC) #4
roubert (google)
https://codereview.appspot.com/101000043/diff/1/debian/control File debian/control (right): https://codereview.appspot.com/101000043/diff/1/debian/control#newcode3 debian/control:3: Maintainer: Debian Java Maintainers <pkg-java-maintainers@lists.alioth.debian.org> On 2014/06/12 15:22:57, daniel2 ...
9 years, 10 months ago (2014-06-13 09:59:58 UTC) #5
roubert (google)
On Fri, Jun 13, 2014 at 3:20 PM, Daniel Pocock <daniel@pocock.pro> wrote: > I agree ...
9 years, 10 months ago (2014-06-13 13:50:49 UTC) #6
roubert (google)
9 years, 10 months ago (2014-06-14 13:09:09 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 manually as r670 (presubmit successful).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b