On 7/8/2024 7:20 PM, Khem Raj wrote:
Hi Jeremy

So it seems only needed during build it seems ?

No the value is used in the /usr/bin/net-snmp-create-v3-user script that ends up in the devel package. The value is checked during the build, but used on the system.

The value is placed here:
if @PSCMD@ | egrep ' snmpd *$' > /dev/null 2>&1 ; then
    echo "Apparently at least one snmpd demon is already running."
    echo "You must stop them in order to use this command."
    exit 1
fi


Where PSCMD is concatinated by PSPROG, already set in the cached values and
ac_cv_ps_flags which the patch adds. The configure script attempts to ascertain
what the correct flags are via:

if test "`($PSPROG -e 2>&1) | $EGREP ' (ps) *$' | awk '{print $NF}'`" = "ps" ; then
    ac_cv_ps_flags="-e"
elif test "`($PSPROG -el 2>&1) | $EGREP ' (ps) *$' | awk '{print $NF}'`" = "ps" ; then
    ac_cv_ps_flags="-el"
elif test "`($PSPROG acx 2>&1) | $EGREP ' (ps) *$' | awk '{print $NF}'`" = "ps" ; then
    ac_cv_ps_flags="acx"
elif test "`($PSPROG -acx 2>&1) | $EGREP ' (ps) *$' | awk '{print $NF}'`" = "ps" ; then
    ac_cv_ps_flags="-acx"
elif test "`($PSPROG -o pid,tt,state,time,ucomm 2>&1) | $EGREP ' ps *$' | awk '{print $NF}'`" = "ps" ; then
    ac_cv_ps_flags="-o pid,tt,state,time,ucomm"
elif test "`($PSPROG ax 2>&1) | $EGREP ' (ps) *$' | awk '{print $NF}'`" = "ps" ; then
    ac_cv_ps_flags="ax"
elif test "x$PARTIALTARGETOS" = "xcygwin"; then
    ac_cv_ps_flags="-e"
elif test "x$PARTIALTARGETOS" = "xmingw32" -o "x$PARTIALTARGETOS" = "xmingw32msvc"; then
    ac_cv_ps_flags="-e"
else
    { printf "%s\n" "$as_me:${as_lineno-$LINENO}: WARNING: Unable to determine valid ps flags...  defaulting..." >&5 printf "%s\n" "$as_me: WARNING: Unable to determine valid ps flags...  defaulting..." >&2;}



However the value ends up in the script. We got a wierd build error where a sshared state was built and got -e and the mulitlib version got auf or something similar, which caused the scripts to conflict. For the sake of reproducablity, we should set the value to one value so it is the same for every build.

On Mon, Jul 8, 2024 at 6:47 PM Jeremy Puhlman <jpuhl...@mvista.com> wrote:



    On 7/8/2024 6:00 PM, Khem Raj wrote:
    > On Mon, Jul 8, 2024 at 5:41 PM Jeremy Puhlman via
    > lists.openembedded.org <http://lists.openembedded.org>
    <jpuhlman=mvista....@lists.openembedded.org>
    > wrote:
    >> From: "Jeremy A. Puhlman" <jpuhl...@mvista.com>
    >>
    >> The value the configure script uses is picked up
    >> from the host machine ps, and there are cases when
    >> the flag selection can be different. Which would
    >> break reproducablity between builds on different
    >> machines.
    >>
    >> Signed-off-by: Jeremy A. Puhlman <jpuhl...@mvista.com>
    >> ---
    >>   meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
    <http://net-snmp_5.9.4.bb> | 1 +
    >>   1 file changed, 1 insertion(+)
    >>
    >> diff --git
    a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
    <http://net-snmp_5.9.4.bb>
    b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
    <http://net-snmp_5.9.4.bb>
    >> index 75b3d9e8c..989ec5413 100644
    >> ---
    a/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
    <http://net-snmp_5.9.4.bb>
    >> +++
    b/meta-networking/recipes-protocols/net-snmp/net-snmp_5.9.4.bb
    <http://net-snmp_5.9.4.bb>
    >> @@ -73,6 +73,7 @@ CACHED_CONFIGUREVARS = " \
    >>       lt_cv_shlibpath_overrides_runpath=yes \
    >>       ac_cv_path_UNAMEPROG=${base_bindir}/uname \
    >>       ac_cv_path_PSPROG=${base_bindir}/ps \
    >> +    ac_cv_ps_flags="-e" \
    > While this change seems ok, I wonder if then we also need to ensure
    > that procps provided ps utility is rdepend'ed upon too.
    All the usage there is to see if snmpd is actually running. Any value
    that works for all of them would be reasonable. I thought the
    assumption was that busybox was always on the system and busybox ps
    accepts -e with an output that should work. Probably
    don't need the rdepend, If base busybox is still assumed.

    >>       ac_cv_file__etc_printcap=no \
    >>       NETSNMP_CONFIGURE_OPTIONS= \
    >>   "
    >> --
    >> 2.35.7
    >>
    >>
>> >>

-- Jeremy Puhlman
    jpuhl...@mvista.com


--
Jeremy Puhlman
jpuhl...@mvista.com

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#111276): 
https://lists.openembedded.org/g/openembedded-devel/message/111276
Mute This Topic: https://lists.openembedded.org/mt/107114904/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to