rename Options to BuildEbpfOptions for clap

pull/454/head
Dmitry Savintsev 3 years ago
parent 30914911e7
commit 3d541e3c82

@ -37,7 +37,7 @@ impl std::fmt::Display for Architecture {
} }
#[derive(Debug, Parser)] #[derive(Debug, Parser)]
pub struct Options { pub struct BuildEbpfOptions {
/// Set the endianness of the BPF target /// Set the endianness of the BPF target
#[clap(default_value = "bpfel-unknown-none", long)] #[clap(default_value = "bpfel-unknown-none", long)]
pub target: Architecture, pub target: Architecture,
@ -49,12 +49,12 @@ pub struct Options {
pub libbpf_dir: PathBuf, pub libbpf_dir: PathBuf,
} }
pub fn build_ebpf(opts: Options) -> anyhow::Result<()> { pub fn build_ebpf(opts: BuildEbpfOptions) -> anyhow::Result<()> {
build_rust_ebpf(&opts)?; build_rust_ebpf(&opts)?;
build_c_ebpf(&opts) build_c_ebpf(&opts)
} }
fn build_rust_ebpf(opts: &Options) -> anyhow::Result<()> { fn build_rust_ebpf(opts: &BuildEbpfOptions) -> anyhow::Result<()> {
let mut dir = PathBuf::from(WORKSPACE_ROOT.to_string()); let mut dir = PathBuf::from(WORKSPACE_ROOT.to_string());
dir.push("test/integration-ebpf"); dir.push("test/integration-ebpf");
@ -92,7 +92,7 @@ fn get_libbpf_headers<P: AsRef<Path>>(libbpf_dir: P, include_path: P) -> anyhow:
Ok(()) Ok(())
} }
fn build_c_ebpf(opts: &Options) -> anyhow::Result<()> { fn build_c_ebpf(opts: &BuildEbpfOptions) -> anyhow::Result<()> {
let mut src = PathBuf::from(WORKSPACE_ROOT.to_string()); let mut src = PathBuf::from(WORKSPACE_ROOT.to_string());
src.push("test/integration-ebpf/src/bpf"); src.push("test/integration-ebpf/src/bpf");

@ -10,7 +10,7 @@ pub struct Options {
pub musl: bool, pub musl: bool,
#[clap(flatten)] #[clap(flatten)]
pub ebpf_options: build_ebpf::Options, pub ebpf_options: build_ebpf::BuildEbpfOptions,
} }
pub fn build_test(opts: Options) -> anyhow::Result<()> { pub fn build_test(opts: Options) -> anyhow::Result<()> {

@ -9,7 +9,7 @@ use std::process::exit;
use clap::Parser; use clap::Parser;
#[derive(Parser)] #[derive(Parser)]
pub struct Options { pub struct XtaskOptions {
#[clap(subcommand)] #[clap(subcommand)]
command: Command, command: Command,
} }
@ -19,12 +19,12 @@ enum Command {
Codegen(codegen::Options), Codegen(codegen::Options),
Docs, Docs,
BuildIntegrationTest(build_test::Options), BuildIntegrationTest(build_test::Options),
BuildIntegrationTestEbpf(build_ebpf::Options), BuildIntegrationTestEbpf(build_ebpf::BuildEbpfOptions),
IntegrationTest(run::Options), IntegrationTest(run::Options),
} }
fn main() { fn main() {
let opts = Options::parse(); let opts = XtaskOptions::parse();
use Command::*; use Command::*;
let ret = match opts.command { let ret = match opts.command {

@ -3,7 +3,7 @@ use std::{os::unix::process::CommandExt, path::PathBuf, process::Command};
use anyhow::Context as _; use anyhow::Context as _;
use clap::Parser; use clap::Parser;
use crate::build_ebpf::{build_ebpf, Architecture, Options as BuildOptions}; use crate::build_ebpf::{build_ebpf, Architecture, BuildEbpfOptions as BuildOptions};
#[derive(Debug, Parser)] #[derive(Debug, Parser)]
pub struct Options { pub struct Options {

Loading…
Cancel
Save