Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • P pkgar
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • redox-os
  • pkgar
  • Merge requests
  • !3

Replace clap with pico_args

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open David Karrick requested to merge Rustywolf/pkgar:pico into master Feb 25, 2020
  • Overview 1
  • Commits 4
  • Changes 2

This pull request replace the clap command line argument parser with pico_args.

This has the following benefits:

  • Reduces binary size by 332 KiB (when compiled with RUSTFLAGS='-C codegen-units=1 -C panic=abort -C link-args=-s' cargo build --release)
  • Reduced time to compile by 3 seconds

The UI is mostly the same. The help information is more colorful now. Pico_args parses arguements linearly so error messages will display the first argument not found (whereas clap would tell you all of them).

Edited Feb 25, 2020 by David Karrick
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: pico