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

P4CP configuration file generator #322

Merged
merged 10 commits into from
Dec 27, 2023
Merged

P4CP configuration file generator #322

merged 10 commits into from
Dec 27, 2023

Conversation

ffoulkes
Copy link
Contributor

@ffoulkes ffoulkes commented Oct 29, 2023

Command-line utility to generate cmake configuration files for building P4 Control Plane.

This is a convenient alternative to environment variables and helper scripts. which become rather tedious when you find yourself typing the same command-line flags every you run (and even more so when you find yourself switching between the same two or three configurations).

cmake -B build -C es2k-setup.cmake
cmake --build build -j6 --target install

The utility can also save default parameters to a file that it loads on subsequent runs, and it can generate files to define common environment variables.

It does not yet support cross-compilation.

@ffoulkes ffoulkes added the medium effort Moderate effort required label Oct 29, 2023
@ffoulkes
Copy link
Contributor Author

I'm think of dropping the .py suffix and making the command just p4cpconfig. Any opinions?

Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
- "A foolish consistency is the hobgoblin of little minds."
  -- Ralph Waldo Emerson

Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
@ffoulkes ffoulkes merged commit 8dfb3d3 into main Dec 27, 2023
5 checks passed
@ffoulkes ffoulkes deleted the p4cpconfig branch December 27, 2023 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
medium effort Moderate effort required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant