Code review comment for ~utkarsh/ubuntu/+source/isc-dhcp:isc-dhcp-lp1894172-interfaces-env

Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thanks, the patch LGTM, so I'm approving this MP.

A comment about git tags: you should not create the upload tag yourself, since you're not going to do the upload. It's the uploader's job to create and push the upload tag (just like we do when we're asking for sponsorship in Debian). Also, even if you could do the upload yourself, it is not a good idea to create the tag before the MP has been reviewed and approved, since you might need to make changes to the branch and force-push things. A good thing here is that the tags you created are under the "utkarsh/" namespace, so they don't conflict with the git-ubuntu tag.

A last comment is that you shouldn't create a "debian/xxyyzz" tag. When using git-ubuntu, we just have to create the "upload/xxyyzz" tag (which is done by running "git ubuntu tag --upload").

About the PPA: we usually upload using the "~ppa1" suffix, which helps to "version" the PPA uploads and makes it easier to deal with multiple uploads of the same package.

Having said all that, I uploaded and pushed the tag:

$ git push pkg upload/4.4.1-2.2ubuntu5
Enumerating objects: 15, done.
Counting objects: 100% (15/15), done.
Delta compression using up to 8 threads
Compressing objects: 100% (10/10), done.
Writing objects: 100% (10/10), 1.29 KiB | 331.00 KiB/s, done.
Total 10 (delta 7), reused 0 (delta 0)
To ssh://git.launchpad.net/ubuntu/+source/isc-dhcp
 * [new tag] upload/4.4.1-2.2ubuntu5 -> upload/4.4.1-2.2ubuntu5

$ dput isc-dhcp_4.4.1-2.2ubuntu5_source.changes
Trying to upload package to ubuntu
Checking signature on .changes
gpg: /home/sergio/work/isc-dhcp/isc-dhcp_4.4.1-2.2ubuntu5_source.changes: Valid signature from 106DA1C8C3CBBF14
Checking signature on .dsc
gpg: /home/sergio/work/isc-dhcp/isc-dhcp_4.4.1-2.2ubuntu5.dsc: Valid signature from 106DA1C8C3CBBF14
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading isc-dhcp_4.4.1-2.2ubuntu5.dsc: done.
  Uploading isc-dhcp_4.4.1-2.2ubuntu5.debian.tar.xz: done.
  Uploading isc-dhcp_4.4.1-2.2ubuntu5_source.buildinfo: done.
  Uploading isc-dhcp_4.4.1-2.2ubuntu5_source.changes: done.
Successfully uploaded packages.

review: Approve

« Back to merge proposal