Am Mon, 27 Nov 2017 16:57:58 -0700 schrieb Jim Davis <jim.epost@xxxxxxxxx>: > On Mon, Nov 27, 2017 at 9:13 AM, Henning Schild > <henning.schild@xxxxxxxxxxx> wrote: > > The debian packages coming out of "make *deb-pkg" lack some critical > > information in the control-files e.g. the "Depends:" field. If one > > tries to install a fresh system with such a "linux-image" > > debootstrap or multistrap might try to install the kernel before > > its deps and the package hooks will fail. > > Has that shown up in practice? The builddeb script goes back some > years now... Yes it has, i came across this issue with a multistrap-based installer. The kernel got picked as one of the first packages. Without explicit deps i can only assume that deboostrap should be affected as well. Maybe it processes the package-tree in another order and happens to work by chance. My guess is that most people use the builddeb script to install additional kernels on a running system, so the missing deps never showed. That said, the patch would allow for much more customization than just adding deps. But others might want to customize the other fields as well. > > + eval 'create_package "$kernel_headers_packagename" \ > > + "$kernel_headers_dir"' "$KDEB_OPTS_IMAGE_HEADERS" > > eval in a shell script with arbitrary input can lead to shenanigans > like > > make bindeb-pkg KDEB_OPTS_IMAGE_HEADERS="; echo All your base" > > and other potentially nastier things. Probably not an issue for a > typical kernel developer sitting in front of his or her laptop, but if > I ran a big automated unattended build farm I might prefer a non-eval > alternative. True. As you said, that might not be an issue for most people. Let us discuss the general approach first. If we agree on applying such a patch and others also raise this issue we might find an implementation without the eval. That is a first shot that can handle the kinds of strings one might want to pass to dpkg-gencontrol. (spaces, pipes and multiple options). Henning -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
