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

Guo (testing only) #147

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Guo (testing only) #147

wants to merge 3 commits into from

Conversation

sjg20
Copy link
Contributor

@sjg20 sjg20 commented Aug 13, 2024

No description provided.

Windows apparently builds pylibfdt if swig is declared as a dependency
in the setup file. Otherwise it must be installed manually and Windows
does not sport a package manager, so this is hard for users.

This change seems to have no ill effects on Linux, at least.

So add a new setup_requires line.

Commit-notes:
This was picked up from:

devicetree-org/pylibfdt#2
END

Series-to: dtc
Series-cc: Gua Guo <[email protected]>, david

Signed-off-by: Gua Guo <[email protected]>
Signed-off-by: Simon Glass <[email protected]>
Allow use of the 'pip' tool within the actions.

Signed-off-by: Simon Glass <[email protected]>
@dgibson
Copy link
Owner

dgibson commented Aug 14, 2024

@sjg20 I'm assuming from the subject line that I shouldn't be looking to merge this yet?

@sjg20
Copy link
Contributor Author

sjg20 commented Aug 14, 2024

Yes I'm just trying to figure out the MSYS2 error...

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.

3 participants