Skip to content

Removed obsolete xtensa-build-zephyr.sh - #6016

Merged
lgirdwood merged 1 commit into
thesofproject:mainfrom
aborisovich:remove-obsolete-xtensa-build-zephyr-sh
Jul 19, 2022
Merged

Removed obsolete xtensa-build-zephyr.sh#6016
lgirdwood merged 1 commit into
thesofproject:mainfrom
aborisovich:remove-obsolete-xtensa-build-zephyr-sh

Conversation

@aborisovich

Copy link
Copy Markdown
Contributor

This script had been rewritten to python xtensa-build-zephyr.py script and is no longer used in CI.
Due to changes to SOF processes related to Zephyr integration (PR6004 and PR6005)
this script will no longer be maintained.

Signed-off-by: Andrey Borisovich andrey.borisovich@intel.com

This script had been rewritten to python xtensa-build-zephyr.py
script and is no longer used in CI.
Due to changes to sof processes related to Zephyr integration
this script will no longer be maintained.

Signed-off-by: Andrey Borisovich <andrey.borisovich@intel.com>
@aborisovich
aborisovich requested a review from marc-hb as a code owner July 18, 2022 09:45
@lgirdwood

Copy link
Copy Markdown
Member

@thesofproject/amd @thesofproject/nxp @thesofproject/mediatek do you guys still use this bash script ? if so, we can remove the Intel platforms from it in favor of the python script. The aim is to deprecate/remove the bash script in favor of Python which is easier to maintain and more platform/OS generic.

@kuanhsuncheng

Copy link
Copy Markdown
Contributor

@thesofproject/amd @thesofproject/nxp @thesofproject/mediatek do you guys still use this bash script ? if so, we can remove the Intel platforms from it in favor of the python script. The aim is to deprecate/remove the bash script in favor of Python which is easier to maintain and more platform/OS generic.

HI Liam,

We don't use this script in MediaTek platforms. Thanks.

@dbaluta

dbaluta commented Jul 18, 2022

Copy link
Copy Markdown
Collaborator

@lgirdwood we still use this in our scripts, but I think it's fine to remove it in order to force us to switch to the python one.

Cc: @LaurentiuM1234 @paulstelian97

@marc-hb

marc-hb commented Jul 18, 2022

Copy link
Copy Markdown
Collaborator

@marc-hb

marc-hb commented Jul 18, 2022

Copy link
Copy Markdown
Collaborator

SOFCI TEST

EDIT: the https://sof-ci.01.org/sofpr/PR6016/build910/build build was successful this time.

@bhiregoudar

Copy link
Copy Markdown

@thesofproject/amd @thesofproject/nxp @thesofproject/mediatek do you guys still use this bash script ? if so, we can remove the Intel platforms from it in favor of the python script. The aim is to deprecate/remove the bash script in favor of Python which is easier to maintain and more platform/OS generic.

We are not using this script.

@lgirdwood
lgirdwood merged commit ef028f6 into thesofproject:main Jul 19, 2022
@aborisovich
aborisovich deleted the remove-obsolete-xtensa-build-zephyr-sh branch June 11, 2023 15:35
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.

10 participants