<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 15, 2021, 13:29 Johannes Schauer Marin Rodrigues <<a href="mailto:josch@debian.org">josch@debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Felipe,<br>
<br>
Quoting Felipe Sateler (2021-12-15 19:56:29)<br>
> THanks for your patience. Your MRs are very much welcome.<br>
<br>
thank you for your review!<br>
<br>
> On Mon, Nov 15, 2021 at 10:09 AM Johannes Schauer Marin Rodrigues <<a href="mailto:josch@debian.org" target="_blank" rel="noreferrer">josch@debian.org</a>> wrote:<br>
> > <a href="https://salsa.debian.org/debian/init-system-helpers/-/merge_requests/17" rel="noreferrer noreferrer" target="_blank">https://salsa.debian.org/debian/init-system-helpers/-/merge_requests/17</a><br>
> ><br>
> > If you like those changes, I can rebase the DPKG_ROOT patch on top of it<br>
> > and run the entire test suite with DPKG_ROOT enabled instead of fakechroot.<br>
> I think we should run both (so that we test both pathways).<br>
<br>
Yes. That's what's implemented in MR 12 already. See override_dh_auto_test in<br>
debian/rules:<br>
<br>
<a href="https://salsa.debian.org/debian/init-system-helpers/-/merge_requests/12/diffs#8756c63497c8dc39f7773438edf53b220c773f67_41_41" rel="noreferrer noreferrer" target="_blank">https://salsa.debian.org/debian/init-system-helpers/-/merge_requests/12/diffs#8756c63497c8dc39f7773438edf53b220c773f67_41_41</a><br>
<br>
Maybe you can merge MR 18 first? Then it becomes easier to make sure that any<br>
additional changes break nothing because then any change I do to a MR will<br>
trigger the salsa CI:<br>
<br>
<a href="https://salsa.debian.org/debian/init-system-helpers/-/merge_requests/18" rel="noreferrer noreferrer" target="_blank">https://salsa.debian.org/debian/init-system-helpers/-/merge_requests/18</a></blockquote></div></div><div dir="auto"><br></div><div dir="auto">But this should require mr 17 first, no?</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
Thanks!<br>
<br>
cheers, josch</blockquote></div></div></div>