Feedback so far:

#2.3 - dracut-network Recommends iscsiuio which is in universe. See the
[Dependencies] section. We could either drop iscsiuio to Suggests or do
an additional MIR for it.

-> We should do a MIR for iscsiuio. iscsiuio comes from the open-iscsi
source package where open-iscsi is already in main. But I found no
previous MIR for open-iscsi. So redoing a MIR for open-iscsi is the way
to go.

#2.5 - The complexity involved in running the upstream test-suite as
build-time tests is noted. However, would it be possible to add some
simple tests instead?

-> we could run TEST-80-GETARGS (which is the only unit test) and run
"make syncheck" to run shellcheck. ... and done in 107-1ubuntu2.

#2.6 - Address pending “Later TODOs” #3 and #4  from the previous review
       https://bugs.launchpad.net/ubuntu/+source/dracut/+bug/2031304/comments/1

-> I like to address point #3 by introducing 3cpio. Point #4 is
translations which is the same as point #2.4 isn't it?

#2.7 - It has been mentioned earlier that the cpio support is being
addressed in 3cpio. It might help to understand how 3cpio would be
integrated and shipped. Is 3cpio ready to be Debian packaged and MIR’d
in the current release cycle?

-> We have 3cpio in Ubuntu, but not in Debian yet. The next step for 3cpio are:
* Finish creation support in 3cpio: https://github.com/bdrung/3cpio/issues/2 (I 
have done work locally)
* Land 3cpio in Debian
* Add support for 3cpio to dracut
Maybe 3cpio is ready in time for this cycle, but maybe not.

#2.8 - The dracut-network package has four alternative dependencies
        isc-dhcp-client | systemd | connman | network-manager
Among these isc-dhcp-client and connman are not in main. We could probably have 
systemd as the primary alternative?

-> Good catch. I changed the order in 107-1ubuntu2

#2.9 - Enable autopkgtests on arm64, i386 and riscv64
       https://autopkgtest.ubuntu.com/packages/dracut

-> That is not trivial. I opened bug #2115532 for tracking it.

#2.10 - Address lintian warnings as noted  in [Packaging red flags]

-> handled via bug #2115494

#2.11 - Address incautious use of malloc as noted in [Upstream red
flags], through a fix or an upstream bug report.

-> That malloc has been replaced by commit aac5c914af84 ("feat(dracut-
install): extend new ELF parsing code to replace ldd calls") which will
be part of the 108 which will be released on 2025-08-01 and included in
questing.

** Bug watch added: github.com/bdrung/3cpio/issues #2
   https://github.com/bdrung/3cpio/issues/2

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2031304

Title:
  [MIR] dracut

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/dracut/+bug/2031304/+subscriptions


-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to