Re: [PATCH v7 06/13] eal: define the EAL parameters in argparse format

[David Marchand <david.marchand@redhat.com>]

On Tue, 30 Sept 2025 at 17:13, Bruce Richardson
<bruce.richardson@intel.com> wrote:
> > It seems this comment got lost.
> >
> > Options don't change that often, but I would prefer we point at the
> > right release doc, rather than origin/main.
> > What do you think of:
> >
> > @@ -164,11 +164,17 @@ eal_usage(const struct rte_argparse *obj)
> >  },
> >  #define STR_ALIAS STR_ARG
> >
> > +#if RTE_VER_RELEASE == 99
> > +#define DOC_SUFFIX "-" RTE_STR(RTE_VER_MAJOR)
> > +#else
> > +#define DOC_SUFFIX
> > +#endif
> > +
> >  struct rte_argparse eal_argparse  = {
> >         .prog_name = "",
> >         .usage = "<DPDK EAL options> -- <App options>",
> >         .epilog = "For more information on EAL options, see the DPDK
> > documentation at: \n"
> > -                       "\thttps://doc.dpdk.org/guides/"
> > RTE_EXEC_ENV_NAME "_gsg/",
> > +                       "\thttps://doc.dpdk.org/guides" DOC_SUFFIX "/"
> > RTE_EXEC_ENV_NAME "_gsg/",
> >
> Yes, that can work, though RTE_VER_MAJOR doesn't exist that I can see. Will
> rework it a bit.

Ah sorry, it was in my previous comment:
https://inbox.dpdk.org/dev/CAJFAV8y5_O3T6hLbZov0LqhXyAu2s_n0P3C3Y+WSGVUjd8gbtg@mail.gmail.com/


-- 
David Marchand