Fix docker-compile -o flag passed to docker instead of container

DEVELOPMENT.md

@@ -150,7 +150,7 @@ tinybpf provides a Docker image for compiling eBPF programs. It bundles libbpf h
 make compile
 
 # Advanced: run docker directly (on Linux, or inside Lima shell on macOS)
-docker run --rm -v $(pwd):/src ghcr.io/gregclermont/tinybpf-compile -o build/ src/*.bpf.c
+docker run --rm -v $(pwd):/src ghcr.io/gregclermont/tinybpf-compile src/*.bpf.c -o build/
 docker run --rm -v $(pwd):/src -e EXTRA_CFLAGS="-DDEBUG" ghcr.io/gregclermont/tinybpf-compile program.bpf.c
 ```
 

GUIDE.md

@@ -14,7 +14,7 @@ tinybpf docker-compile program.bpf.c
 tinybpf docker-compile src/*.bpf.c
 
 # Output to specific directory
-tinybpf docker-compile -o build/ src/*.bpf.c
+tinybpf docker-compile src/*.bpf.c -o build/
 ```
 
 The CLI uses a Docker image with libbpf headers and `vmlinux.h` (kernel 6.18) for CO-RE support. Output `.bpf.o` files are written alongside sources (or to the specified output directory).

docker/entrypoint.sh

@@ -56,23 +56,33 @@ if [ $# -eq 0 ]; then
     echo ""
     echo "Examples:"
     echo "  docker run --rm -v \$(pwd):/src ghcr.io/gregclermont/tinybpf-compile program.bpf.c"
-    echo "  docker run --rm -v \$(pwd):/src ghcr.io/gregclermont/tinybpf-compile -o build/ src/*.bpf.c"
+    echo "  docker run --rm -v \$(pwd):/src ghcr.io/gregclermont/tinybpf-compile src/*.bpf.c -o build/"
     echo ""
     echo "  # Use custom vmlinux.h for older kernel:"
     echo "  docker run --rm -v \$(pwd):/src -v /path/to/vmlinux.h:/vmlinux.h -e VMLINUX=/vmlinux.h \\"
     echo "    ghcr.io/gregclermont/tinybpf-compile program.bpf.c"
     exit 0
 fi
 
-# Parse options
+# Parse arguments (options can appear anywhere)
 OUTPUT_DIR=""
-while getopts "o:" opt; do
-    case $opt in
-        o) OUTPUT_DIR="$OPTARG" ;;
-        *) exit 1 ;;
+SOURCES=()
+while [ $# -gt 0 ]; do
+    case "$1" in
+        -o)
+            if [ -z "$2" ]; then
+                echo "Error: -o requires an argument" >&2
+                exit 1
+            fi
+            OUTPUT_DIR="$2"
+            shift 2
+            ;;
+        *)
+            SOURCES+=("$1")
+            shift
+            ;;
     esac
 done
-shift $((OPTIND - 1))
 
 # Create output directory if specified
 if [ -n "$OUTPUT_DIR" ]; then
@@ -81,7 +91,7 @@ fi
 
 # Compile each source file
 FAILED=0
-for src in "$@"; do
+for src in "${SOURCES[@]}"; do
     if [ ! -f "$src" ]; then
         echo "Error: File not found: $src" >&2
         FAILED=1

llms.txt

@@ -145,7 +145,7 @@ BtfKind.INT, STRUCT, UNION, ARRAY, PTR, TYPEDEF, FLOAT, ...  # BTF type kinds
 
 ```bash
 tinybpf docker-compile program.bpf.c
-tinybpf docker-compile -o build/ src/*.bpf.c
+tinybpf docker-compile src/*.bpf.c -o build/
 ```
 
 Uses a Docker image with libbpf headers and vmlinux.h (kernel 6.18, x86_64/aarch64) for CO-RE support. Output `.bpf.o` files are written alongside sources or to specified output directory.

src/tinybpf/_cli.py

@@ -52,15 +52,14 @@ def cmd_docker_compile(args: argparse.Namespace) -> int:
         "--rm",
         "-v",
         f"{cwd}:/src",
+        image_tag,
+        *args.sources,
     ]
 
     # Pass through output directory if specified
     if args.output:
         docker_cmd.extend(["-o", args.output])
 
-    docker_cmd.append(image_tag)
-    docker_cmd.extend(args.sources)
-
     if args.verbose:
         print(f"Running: {' '.join(docker_cmd)}", file=sys.stderr)