One of my ebuilds is using virtualx eclass, and I noticed the following
code inside the eclass:

retval=$?

# Now kill Xvfb
kill $(cat /tmp/.X${XDISPLAY}-lock)
else
debug-print "${FUNCNAME}: attaching to running X display"
# Normal make if we can connect to an X display
debug-print "${FUNCNAME}: ${VIRTUALX_COMMAND} "$@""
${VIRTUALX_COMMAND} "$@"
retval=$?
fi

# die if our command failed
[[ $? -ne 0 ]] && die "${FUNCNAME}: the ${VIRTALX_COMMAND} failed."

Shouldn't that last line look more like this (notice $retval instead of $?):

[[ $retval -ne 0 ]] && die "${FUNCNAME}: the ${VIRTALX_COMMAND} failed."

What do you think?

On Sat, 12 Mar 2011 11:37:28 +0100
"Paweł Hajdan, Jr." <phajdan.jr@gentoo.org> wrote:

> Shouldn't that last line look more like this (notice $retval instead of $?):
>
> [[ $retval -ne 0 ]] && die "${FUNCNAME}: the ${VIRTALX_COMMAND} failed."

Sure looks like it to me.


--
fonts, gcc-porting, it makes no sense how it makes no sense
toolchain, wxwidgets but i'll take it free anytime
@ gentoo.org EFFD 380E 047A 4B51 D2BD C64F 8AA8 8346 F9A4 0662

On Sat, 12 Mar 2011 11:37:28 +0100
"Paweł Hajdan, Jr." <phajdan.jr@gentoo.org> wrote:

> Shouldn't that last line look more like this (notice $retval instead
> of $?):
>
> [[ $retval -ne 0 ]] && die "${FUNCNAME}: the ${VIRTALX_COMMAND}
> failed."
>
> What do you think?

I'd say even '${VIRTUALX_COMMAND}'.

--
Best regards,
Michał Górny

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Dne 12.3.2011 11:37, "Paweł Hajdan, Jr." napsal(a):
> One of my ebuilds is using virtualx eclass, and I noticed the following
> code inside the eclass:
>
> retval=$?
>
> # Now kill Xvfb
> kill $(cat /tmp/.X${XDISPLAY}-lock)
> else
> debug-print "${FUNCNAME}: attaching to running X display"
> # Normal make if we can connect to an X display
> debug-print "${FUNCNAME}: ${VIRTUALX_COMMAND} "$@""
> ${VIRTUALX_COMMAND} "$@"
> retval=$?
> fi
>
> # die if our command failed
> [[ $? -ne 0 ]] && die "${FUNCNAME}: the ${VIRTALX_COMMAND} failed."
>
> Shouldn't that last line look more like this (notice $retval instead of $?):
>
> [[ $retval -ne 0 ]] && die "${FUNCNAME}: the ${VIRTALX_COMMAND} failed."
>
> What do you think?
>
Ack to both what you said and what Michal said. Fixed in cvs
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk18f18ACgkQHB6c3gNBRYfoMACfYiOyC7nJlp/AI0gUE37KBVEL
dboAnRkahB1np882D/b7R80vGU9uzXky
=DFzF
-----END PGP SIGNATURE-----