← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~benji/lpsetup/bug-1016645-add-command-line-options into lp:lpsetup

 

@Gary: didn't we decided to also get rid of --dependencies-dir? And to just checkout them inside the branch/sandbox?

As a side note, not really related to this MP, I've started thinking that the conservative approach we are using to work on these cards creates more problems than it solves: get and update share with the old commands only the name. Of course there is some code to reuse, but I suspect there is the risk that old code is influencing too much the way we factor the new commands. That's only a feeling I have, and I can only use my experience on initlxc as evidence: I started working on it trying to change lxcinstall before realizing that initlxc is a much simpler command, without all the lxcinstall options and complications.

-- 
https://code.launchpad.net/~benji/lpsetup/bug-1016645-add-command-line-options/+merge/112588
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~benji/lpsetup/bug-1016645-add-command-line-options into lp:lpsetup.


References