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

Add Tail implementaiton #153

Closed
wants to merge 25 commits into from
Closed

Add Tail implementaiton #153

wants to merge 25 commits into from

Conversation

mike-ward
Copy link
Contributor

Tail implementation

  • uses 4K buffers for read/write operations. Handles large files easily.
  • -f option tracks by name, not descriptor. GNU uses descriptors but it really complicates things for little gain (imo)
  • uses ps to track PID's. May not work on Windows although most developers probably have it installed because tasklist is a GUI

@JalonSolov
Copy link
Contributor

Looks like you have fmt changes in here as well. It would be better to separate those to a different PR.

@mike-ward
Copy link
Contributor Author

oops, probably fat fingered something. will fix.

@mike-ward mike-ward closed this Jun 1, 2024
@mike-ward mike-ward deleted the tail branch June 2, 2024 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants