On Mon, 2019-07-29 at 09:33 +0800, Zang Ruochen wrote:
> -The original file mdmonitor.service is as follows:
> ...
> > [Service]
> > Environment=  MDADM_MONITOR_ARGS=--scan
> > EnvironmentFile=-/run/sysconfig/mdadm
> > ExecStartPre=-/usr/lib/mdadm/mdadm_env.sh
> > ExecStart=/sbin/mdadm --monitor -y $MDADM_MONITOR_ARGS
> ...
> -It has a syntax error and it doesn't work properly, 
>  so add new file mdmonitor.service to overwrite it.

This is better as it tells us what the problem is but it still isn't
quite what I asked for.

Can we patch the file that mdadm is installing so the difference is
clear to anyone reading the recipe? The patch will also have comments
as above so that people don't have to find this commit to understand
why we're changing it and what the problem is. If/as/when upstream
fixes it we'll also be able to drop the patch, rather than not knowing
if/when its fixed.

Thanks,

Richard

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to