Skip to content

Use shell builtin for readlink -f#2232

Open
fiftydinar wants to merge 6 commits intoivan-hc:mainfrom
fiftydinar:patch-6
Open

Use shell builtin for readlink -f#2232
fiftydinar wants to merge 6 commits intoivan-hc:mainfrom
fiftydinar:patch-6

Conversation

@fiftydinar
Copy link
Copy Markdown
Contributor

@fiftydinar fiftydinar commented Apr 7, 2026

Saves a couple of milliseconds when called.

Turns out that it's impossible or not easy to implement classic readlink in POSIX, so I mark this as ready.

Saves a couple of milliseconds when called.
This way, it will ensure that all the code below will use those builtins
@fiftydinar fiftydinar marked this pull request as draft April 7, 2026 09:50
@fiftydinar fiftydinar marked this pull request as ready for review April 7, 2026 18:45
@fiftydinar
Copy link
Copy Markdown
Contributor Author

readlink $1 seems not possible to be in POSIX shell built-in, so I only implemented readlink -f $1

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.

1 participant