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

meson fixes #154

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

meson fixes #154

wants to merge 2 commits into from

Conversation

neheb
Copy link
Contributor

@neheb neheb commented Jul 18, 2020

Normally flashrom is built with all options enabled. But it starts to go wrong when several options are disabled. This is the case in the OpenWrt package where 4 separate packages are provided with differing functionality. (flashrom, -pci, -usb, -spi).

Currently, when passing meson -Dconfig_rayer_spi=false , compilation
fails as the proper headers and defines are not available.

Added need_raw_access based on whichever files include hwaccess.h

Signed-off-by: Rosen Penev <rosenp@gmail.com>
Fixes meson error:

meson.build:304:3: ERROR: Unknown variable "config_bitbang_spi".

Signed-off-by: Rosen Penev <rosenp@gmail.com>
@neheb
Copy link
Contributor Author

neheb commented Apr 8, 2022

ping @felixsinger

@felixsinger
Copy link
Contributor

Is there any reason why you don't upload them to Gerrit yourself?

@neheb
Copy link
Contributor Author

neheb commented Apr 8, 2022

I don't use gerrit.

@felixsinger
Copy link
Contributor

Yeah, why? What's the reason for this? Like don't you know how to setup your local repository or do you just don't want to create another account somewhere? In the latter case, you can choose between many OAuth providers, including GItHub, and you also can use OpenID if you prefer that. https://review.coreboot.org/login

@neheb
Copy link
Contributor Author

neheb commented Apr 8, 2022

latter case. Maybe the LLVM workflow left a bad taste in my mouth (Phabricator). git send-mail is simple enough.

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.

None yet

2 participants