-
-
Notifications
You must be signed in to change notification settings - Fork 12.9k
systemc: Switch to autotools configure #225258
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
Conversation
the cmake switch was due to the fact that autotools build did not work before, accellera-official/systemc#40 |
any issue tracking it? |
pkg-config and so on is marked as TODO in cmake file: Other distros like Debian and Fedora are all using autotools. |
make sense 👍 |
systemc's cmake build lacks many features, namely pkg-config files and document & example files. Fix by switching to autotools configure based build flow, also applying patch from Debian to fix document build. Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
ee0f0f0
to
37a7ed7
Compare
@FlyGoat, thanks for your first contribution to homebrew-core! 🎉 💯 |
🤖 An automated task has requested bottles to be published to this PR. Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch. |
systemc's cmake build lacks many features, namely
pkg-config files and document & example files.
Fix by switching to autotools configure based build flow, also applying patch from Debian to fix document build.
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?