On Mon, Feb 26, 2024 at 11:11 AM Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> wrote: > > The driver might decide to put the _shipped files to the subdir. Please stop this sentence. This sounds like we are not learning. https://lore.kernel.org/all/CAHk-=wgSEi_ZrHdqr=20xv+d6dr5G895CbOAi8ok+7-CQUN=fQ@xxxxxxxxxxxxxx/ > In such > case the cmd_copy might fail because the destination directory is not > present. Call mkdir -p to make sure that the destination directory is > present. There is no justification for this. If you need a single generated directory (drivers/gpu/drm/msm/registers/, divers/gpu/drm/msm/generated/ or whatever) that should be super simple. Why does scripts/Makefile.lib need the modification? > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > --- > scripts/Makefile.lib | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib > index cd5b181060f1..94373eeac420 100644 > --- a/scripts/Makefile.lib > +++ b/scripts/Makefile.lib > @@ -297,7 +297,7 @@ endef > # the copy would be read-only as well, leading to an error when executing the > # rule next time. Use 'cat' instead in order to generate a writable file. > quiet_cmd_copy = COPY $@ > - cmd_copy = cat $< > $@ > + cmd_copy = mkdir -p $(shell dirname $@) && cat $< > $@ > > $(obj)/%: $(src)/%_shipped > $(call cmd,copy) > > -- > 2.39.2 > -- Best Regards Masahiro Yamada
