Ralph,
I think it would be much better to allow and merge multiple "-E" options.
Cheers, Andreas
> On Sep 9, 2025, at 09:43, Ralph Siemsen <ralph.siemsen@xxxxxxxxxx> wrote:
>
> Make it an error to pass multiple -E options. As per the man page,
> multiple extended options must be specified as a comma-separated list,
> instead of multiple -E options.
>
> This helps avoid surprises, as the existing behaviour is to process
> only the last -E option, and to silently ignore any earlier ones.
>
> Signed-off-by: Ralph Siemsen <ralph.siemsen@xxxxxxxxxx>
> ---
> misc/mke2fs.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/misc/mke2fs.c b/misc/mke2fs.c
> index 3a8ff5b1..59c7be17 100644
> --- a/misc/mke2fs.c
> +++ b/misc/mke2fs.c
> @@ -1796,6 +1796,12 @@ profile_error:
> _("'-R' is deprecated, use '-E' instead"));
> /* fallthrough */
> case 'E':
> + if (extended_opts) {
> + com_err(program_name, 0, "%s",
> + _("Multiple '-E' options are not supported, "
> + "use one comma-separated value instead"));
> + exit(1);
> + }
> extended_opts = optarg;
> break;
> case 'e':
>
> --
> 2.45.2.121.gc2b3f2b3cd
>
>
