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

Reduce linutil binary size by removing unused features from tui/Cargo.toml #922

Merged
merged 4 commits into from
Nov 11, 2024

Conversation

zdiff
Copy link
Contributor

@zdiff zdiff commented Nov 6, 2024

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Hotfix
  • Security patch
  • UI/UX improvement

Description

Reduce the size of the linutil binary by removing unused features from tui/Cargo.toml

Before PR

$ cargo build --release && ls -l target/release/linutil
    Finished `release` profile [optimized] target(s) in 0.41s
-rwx------ 2 ghost ghost 4336488 Nov  6 12:15 target/release/linutil

After PR

$ cargo build --release && ls -l target/release/linutil
    Finished `release` profile [optimized] target(s) in 0.08s
-rwx------ 2 zdiff zdiff 4189032 Nov  6 12:11 target/release/linutil

Testing

Build and ran linutil using cargo build --release && ./target/release/linutil

Impact

Reduction in binary size

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

@zdiff zdiff changed the title Reduce the size of the linutil binary by removing unused features fro… Reduce linutil binary size by removing unused features from tui/Cargo.toml Nov 6, 2024
@adamperkowski adamperkowski added the rust Pull requests that update Rust code label Nov 6, 2024
@ChrisTitusTech ChrisTitusTech merged commit 16dd91c into ChrisTitusTech:main Nov 11, 2024
2 of 3 checks passed
jeevithakannan2 pushed a commit to jeevithakannan2/linutil that referenced this pull request Nov 11, 2024
…m tui/Cargo.toml (ChrisTitusTech#922)

Co-authored-by: zdiff <zdiff@localhost>
@jeevithakannan2 jeevithakannan2 mentioned this pull request Nov 11, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants