Skip to content

[Hexagon] Fix use of subprocess.run in _check_call_verbose#11985

Merged
kparzysz-quic merged 1 commit intoapache:mainfrom
kparzysz-quic:check-call-verbose
Jul 1, 2022
Merged

[Hexagon] Fix use of subprocess.run in _check_call_verbose#11985
kparzysz-quic merged 1 commit intoapache:mainfrom
kparzysz-quic:check-call-verbose

Conversation

@kparzysz-quic
Copy link
Contributor

@kparzysz-quic kparzysz-quic commented Jul 1, 2022

It uses parameters that are not present in Python 3.6, plus it catches generic exception, which may not have stdout or stderr members.

cc @mehrdadh

It uses parameters that are not present in Python 3.6, plus it
catches generic exception, which may not have `stdout` or `stderr`
members.
@kparzysz-quic kparzysz-quic requested a review from mehrdadh July 1, 2022 15:23
@kparzysz-quic kparzysz-quic changed the title [Hexagon] Fix use of subprocess.run in _check_call_verbose [Hexagon] Fix use of subprocess.run in _check_call_verbose Jul 1, 2022
Copy link
Member

@mehrdadh mehrdadh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for fixing this!

@kparzysz-quic kparzysz-quic merged commit c97895e into apache:main Jul 1, 2022
@kparzysz-quic kparzysz-quic deleted the check-call-verbose branch July 1, 2022 19:00
blackkker pushed a commit to blackkker/tvm that referenced this pull request Jul 7, 2022
)

It uses parameters that are not present in Python 3.6, plus it
catches generic exception, which may not have `stdout` or `stderr`
members.
masahi pushed a commit to masahi/tvm that referenced this pull request Jul 15, 2022
)

It uses parameters that are not present in Python 3.6, plus it
catches generic exception, which may not have `stdout` or `stderr`
members.
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
)

It uses parameters that are not present in Python 3.6, plus it
catches generic exception, which may not have `stdout` or `stderr`
members.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants