Code review comment for lp:~evfool/software-properties/lp1170114

On Wed, Jun 19, 2013 at 07:44:28AM -0000, Robert Roth wrote:
> Robert Roth has proposed merging lp:~evfool/software-properties/lp1170114 into lp:software-properties.
>
> Requested reviews:
> Ubuntu Core Development Team (ubuntu-core-dev)
> Related bugs:
> Bug #1170114 in software-properties (Ubuntu): "Add --disable-component option"
> https://bugs.launchpad.net/ubuntu/+source/software-properties/+bug/1170114
>
> For more details, see:
> https://code.launchpad.net/~evfool/software-properties/lp1170114/+merge/170185
>
> As we have add-apt-repository as the command line tool for adding/removing/enabling/disabling PPAs and distro components, I would suggest removing the command-line switch for enabling a component from software-properties-gtk to avoid duplication of code and functionality within the same project.
> --
> https://code.launchpad.net/~evfool/software-properties/lp1170114/+merge/170185
> You are subscribed to branch lp:software-properties.

Thanks, I think this is good now. +1