Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update install_locally to match the directory created in prepare_build_environment #469

Merged
merged 2 commits into from
Jul 4, 2023

Conversation

bblacher
Copy link
Contributor

@bblacher bblacher commented Jun 2, 2023

The directory mentioned in install_locally doesn't match the one mentioned in prepare_build_environment.

This mr fixes this.

…d_environment

The directory mentioned in install_locally doesn't match the one mentioned in prepare_build_environment.

This commit fixes this.
Change other occurrences of ~/sof/linux to ~/work/sof/linux
@bblacher bblacher requested a review from plbossart June 7, 2023 09:06
Copy link
Member

@plbossart plbossart left a comment

Choose a reason for hiding this comment

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

Thanks @bblacher

@lgirdwood
Copy link
Member

lgirdwood commented Jul 4, 2023

@bblacher looks like 1 warning that is showing in CI but its unrealted to this PR.

waiting for workers...
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
writing output... [ 20%] algos/eq/equalizers_tuning .. architectures/firmware/sof-xtos/mem-mgmt
writing output... [ 40%] architectures/firmware/sof-xtos/overview .. developer_guides/add_new_arch
writing output... [ 60%] developer_guides/debugability/coredump-reader/index .. developer_guides/virtualization/virtualization
writing output... [ 80%] developer_guides/xtrun/index .. platforms/intel-legacy/index
writing output... [100%] platforms/intel-legacy/merrifield/index .. tsc/representatives

waiting for workers...
WARNING: dot code 'blockdiag edf_scheduling {\n\n\tnode_width = 250;\n\tnode_height = 120;\n\tdefault_fontsize = 16;\n\n\tComp_1 -> Comp_2\n\tcomment_1 -> Comp_2 [style=dashed]\n\tComp_2 -> Comp_3\n\tcomment_2 -> Comp_3 [style=dashed]\n\tComp_3 -> Comp_4\n\tcomment_3 -> Comp_4 [style=dashed]\n\tComp_4 -> sink\n\tcomment_4 -> sink [style=dashed]\n\n\tComp_1 [label="DP component 1\\n\n\t\t*processing period\\n\n\t\t*compute requirement"]\n\tComp_2 [label="DP component 2\\n\n\t\t*processing period\\n\n\t\t*compute requirement"]\n\tComp_3 [label="DP component 3\\n\n\t\t*processing period\\n\n\t\t*compute requirement"]\n\tComp_4 [label="DP component 4\\n\n\t\t*processing period\\n\n\t\t*compute requirement"]\n\n\tsink [label="real time sink", shape=endpoint, fontsize = 16]\n\n\tcomment_1 [label="DP1 to deliver data let\\n\n\t\tDP2 meet its objective"]\n\tcomment_2 [label="DP2 to deliver data let\\n\n\t\tDP3 meet its objective"]\n\tcomment_3 [label="DP3 to deliver data let\\n\n\t\tDP4 meet its objective"]\n\tcomment_4 [label="DP4 to deliver data\\n\n\t\tto real time-sink"]\n}\n': 'ImageDraw' object has no attribute 'textsize'
generating indices... genindex done
writing additional pages... search done

@lgirdwood lgirdwood merged commit 28025fe into thesofproject:master Jul 4, 2023
@marc-hb
Copy link
Collaborator

marc-hb commented Jul 11, 2023

@bblacher looks like 1 warning that is showing in CI but its unrealted to this PR.

Unrelated indeed. It's triggered by brand new pillow version 10.0.0. Probably some subtle incompatibility. Fix workaround submitted below, please review

Note this affects the sof daily builds too

https://github.com/thesofproject/sof/actions/workflows/daily-tests.yml

Screenshot 2023-07-10 at 17 01 56

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.

5 participants