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

CLI: Speed up via --compile=min -O0 flags #549

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

rht
Copy link

@rht rht commented Feb 17, 2022

The -S feature has existed since Linux coreutils 8.30.
See https://stackoverflow.com/questions/4303128/how-to-use-multiple-arguments-for-awk-with-a-shebang-i-e/52979955#52979955.
The comment also says that it should work on FreeBSD (not tested).

This speeds up from ~1:16.69 s to ~31 s on my laptop (Linux).

TODO:

  • I haven't tested this on macOS. I can't test this by myself.

The -S feature has existed since Linux coreutils 8.30.
See https://stackoverflow.com/questions/4303128/how-to-use-multiple-arguments-for-awk-with-a-shebang-i-e/52979955#52979955.
The comment also says that it should work on FreeBSD (not tested).
@domluna
Copy link
Owner

domluna commented Feb 18, 2022

i have a macbook air now so I can test this later.

I'm honestly surprised anyone uses this script still it's really, really, outdated; with respect to option selection.

@rht
Copy link
Author

rht commented Feb 19, 2022

I'm honestly surprised anyone uses this script still it's really, really, outdated; with respect to option selection.

I suppose that no one uses the script because it's slow regardless of the speedup?
Do you think it is better to delete the script, and recommend people to use other means, like via vim integration?
Though I see it could be used in CI.

This is the original discussion that led to this PR: https://julialang.zulipchat.com/#narrow/stream/137791-general/topic/cli.20not.20included.

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.

None yet

2 participants