Code review comment for lp:~vanvugt/ubuntu/oneiric/compiz-plugins-main/fix-793897

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Ok, I was mislead by the fact that the patch was applied when you committed in the upstream branch, sorry for the confusion :)

Thanks a lot for the patch.
However, the real branch for compiz-plugins-main is lp:~compiz/compcomm-plugins-main/ubuntu. It contains a debian/ only branch. (this is the case for most desktop team branches).

To be sure to checkout the right branch, you can use $ debcheckout <source_package_name>

That should (if a bzr-vcs tag is provided), bring you the correct branch. If nothing is available, it's generally lp:ubuntu/<source_package_name>.

No worry, I've checkouted the right branch and commit it under your nam (also filed some debian/changelog info which is what you should do to document your changes). You can find the result there: http://bazaar.launchpad.net/~compiz/compcomm-plugins-main/ubuntu/revision/186

Thanks a lot for your contribution :)

review: Approve

« Back to merge proposal