-
Notifications
You must be signed in to change notification settings - Fork 18
Proof of Concept: Migrate to Cmake #45
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Minsoo Choo <[email protected]>
Signed-off-by: Minsoo Choo <[email protected]>
Signed-off-by: Minsoo Choo <[email protected]>
Signed-off-by: Minsoo Choo <[email protected]>
Signed-off-by: Minsoo Choo <[email protected]>
Signed-off-by: Minsoo Choo <[email protected]>
Bump version to 0.7.0 since this breaks header API Signed-off-by: Minsoo Choo <[email protected]>
Signed-off-by: Minsoo Choo <[email protected]>
|
@mchoo7: I appreciate the effort ! |
There is no "official" decision made yet. There were several past and ongoing efforts to codify LLM policies though. See D50650, D54366, and D54817. TL;DR: The first proposal D50650 was about completely banning LLM usage like NetBSD and Gentoo, but there were many voices against it (even from some core members and senior committers). The only worry about LLM usage was license issue, but it seems trivial at this point (too many LLM-generated codes are already in-production and there hasn't been any noticeable litigation over it). We won't even have "Coauthored-by: " line in commit message to make sure that the only "author" is the contributor and not LLMs, and the coauthored-by label can blur the responsibility (LLMs can't have authorship). The new proposal was submitted by a core member, so I think this is the project's final decision on LLM usage.
I used Claude code because I forgot Cmake syntax and going over documentation will take a long time while this is just a suggestion. I don't use "automatically accept all changes" (aka Shift+Tab) and I could find some mistakes while reviewing code it generated. |
This is not for review, I'm just suggesting possible implement of cmake version of lutok. Once there is a consensus, we can start review and improve.
I created a cmake prototype with help of AI to see how it works. Currently build and doxygen are tested, whereas the testing part needs cmake-version of atf. IMHO, cmake is way superior to autoconf and transition to cmake should be made at some point. Note that I bumped lutok version to 0.7.0 because
<state.ipp>was merged into<state.hpp>(*.ippisn't used often)This project is now cmake only, and other projects using lutok needs to transition from autoconf to cmake.
Some other ideas to modernize lutok (also for atf and kyua):