Friday, 21 September 2018

Re: [Feedback needed] New tool for proposed migration help

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEEkD8wCvN9L2OcLX06AGlaQEyevacFAluknfcACgkQAGlaQEye
vacp3A//UcVoWJL7IA+ZKy1JhbWVDuvkyqFUO0BsG8ZK83QFDgLqNYEiot6BwoQz
7ahQLdjCUGzUQMkU6FHKoQV0ha29a+xO48/KA4AIAOSakUrQd9kJBuECGB0J0yve
WzsHouhsLvL+62Ws3cXLh3eHM6888AZej6os7euliOMTn1OwKI6iQ7cYZgrycfeP
8OYLSzVl9jA4OybI+9RFrmppGald5PR5sIpVMQl75Fe7itLUilMh/vxxO0YZA6ae
kLPQOBlIJbqubb0X4h6v9lDPgWNVSLlt0CLsoHGhxNJsvQxbfV7XJ/SIk0NBpO+9
E0sgKH3E581qAV+hiDDbGRhXNo7V+/7Yk556ehc9XDSNe3yjp2rGgTn1s/ZusQOb
v0ppUaAwl00poCm8Y6kjQF/bIjVMf+KhKVE0Ip5uc2fZx8PGNWP4++MWqvn/ZkxC
NJKV9Ltj27vRaI+jCBXiF0KAW8wk9XfCXYxftt7RWwc7qycbjLbgQgM8Fz0ZbNkM
u1sTVKcJ8rH7RQiG7YY3Nf71Su6PadwKomgYoOEQSw/lP3qNOFlyTfsvI4yu28Hn
7YwZYezOWQ0R/xn9wgKIXU5iwG7y1Qr287KR9Z+yRxWYFSUrmgy9c1/WsG8ijzIP
cDd6G5JhtrCISaopA893s5FLET2Hy2n0eMqj87mxnbZdneM455U=
=aTJ7
-----END PGP SIGNATURE-----
On 09/20/18 16:30, Dan Streetman wrote:
[...]
> however, looking at the commit, it appears it's totally isolated and
> uses none of the common code from ubuntutools/
>
> that's unfortunate and some of what you've done duplicates what's
> already under ubuntutools/, and some has better alternatives to how
> you implemented it.

I disagree on some aspects that it has "better alternatives". Some of
this is doing the exact same thing, just going about it a slightly
different way.

To be clear, this code started in a completely separate tree, and this
is the first commit to ubuntu-dev-tools of its codebase. I wasn't using
ubuntutools because it was "not available", not really practical to use
and keep the tree (and a snap) to use as few dependencies as possible. N

pull-lp-source use was added afterwards, and the 'mir' subcommand is
special in that I wouldn't expect it to be used by anyone but the MIR
review team -- it doesn't do much else than display a bug, find the
right bug based on a fuzzy search of the package (binary or source) to
review; and then drop you to a shell to do code review. It's certainly
reusable, but things were done in a way to scratch my own itches for the
purpose of MIR review.

Now, all this was pushed to ubuntu-dev-tools as a way to get more eyes
and more use -- so when you see things you disagree with, it's
absolutely fine for you to go ahead and fix the code. :)

Kindly,

--
Mathieu Trudel-Lapierre <cyphermox@ubuntu.com>
Freenode: cyphermox, Jabber: mathieu.tl@gmail.com
4096R/65B58DA1 818A D123 0992 275B 23C2 CF89 C67B B4D6 65B5 8DA1