Bug: gawk essential dependency will likely install brew on fresh macos installation #239
Labels
bug
Broken or unexpected
Milestone
Comments
balupton
changed the title
This was referenced Aug 27, 2024
|
Fixed in dev |
|
Using the bottles returned failures when executing gawk, as such I have reverted the use of bottles. Possible solutions:
|
balupton
added a commit
that referenced
this issue
Sep 16, 2024
this option worked: 7deab4e as such gawk is no longer a needed dep, but will be preferred if available |
balupton
added a commit
that referenced
this issue
Sep 17, 2024
/close #249 #241 #239 changes: - replace common gsed/sed/sd/ripgrep/rg/grep/teip usage with new `echo-regexp` that uses deno, affects `convert-helper`, `config-helper`, `echo-magnet-hash`, `fs-(bytes|kilobytes|megabytes)`, `fs-speed`, `get-devices`, `get-local-to-remote`, `get-opensuse-release`, `get-volumes`, `git-helper`, `is-bash-version-outdated`, `mount-helper`, `network-interface`, `setup-dns`, `setup-util`, `setup-util-1password-cli`, `setup-util-nerd-fonts`, `setup-util-node`, `unziptar`, `what-is-my-(dns|gateway|interface|ip|submit)`, `get-codec`, `is-prefix`, `is-suffix`, `itunes-owners`, `mac-address-helper`, `setup-server` - `bash.bash`: - simplify eval statements and support spaces in paths - add explicit names for `__print*` helpers, and slightly change their functionality to be aligned with expectations - **please change `__print_line ...` to `__print_lines ...` as in the future `__print_line` may not support arguments** - **removed deprecated non `__` prefixed aliases, e.g. `require_globstar` is now only `__require_globstar`** - `ci/dorothy-workflow`: don't test PRs until review requested - `ci/dorothy`: don't eslint as no Node.js code - `ci/vscode`: add an awk syntax highlighting extension, and enable Deno - `command-working`: give details of failures, add `dash` to exceptions - `dorothy-config`/`config-helper`: significant work to better handle multiple instances (result is no longer garbled), output and process is now clearer, includes multiple strategies for multiple instances, uses new `echo-regexp` to replace `sd/rg/teip/grep` combo - `dorothy` minimum deps now only `jq` and `deno`s, - `down`: add `--url=...` support - `echo-gnu-command`: fix it not outputting the command after installation - `echo-if-empty`/`get-devices`/`get-local-to-remote`: fix sporadic success by adding missing `--stdin` - `echo-lines`: now support `--[no-]quote` flags as alias for prior `--[no-]quoted` flag - `echo-math`/`echo-(html|url)-(encode|decode)`: rewrote and added tests, now they work correctly, in the latter case `echo-(html|url)-coder` is now used - `echo-wrap`: now awk compatible, removing dependency on `gawk` - `eval-tester`: output escaped comparison if special characters were used - `fs-(bytes|kilobytes|megabytes)`: use `dust` instead of `du` to achieve macos support, also fix crash in `fs-megabytes` - `fs-rm`: fix trash error on alpine, as trash isn't available on alpine - `get-codec`: removed `--trim` and made it default (prior verbosity available via `--verbose`), now sensible error if `ffprobe` is not available - `get-codec`: rewrote and fixed - `get-filesystem`: no longer fail silently when no args are provided - `get-local-to-remote`: improve styling - `get-opensuse-release`: simpler technique - `get-url-(domain|protocol)` split into `get-url-(domain|protocol).ts` and add tests - `is-match`: deprecated and moved in beta commands, as it is replaced by `echo-regexp -q` which it now uses internally - `mac-address-helper`: fixed broken `new` functionality, and improved formatting of outputs - `set-hostname`/`setup-hosts`/`setup-shell`: fix break when gsed isn't available, now uses the `echo-gnu-command` helper - `setup-util-1password`: install stable instead of beta, as stable is now stable - `setup-util-deno`: add apk, uninstall snap (as snap does not allow script execution), use official arm builds as that is now a thing, don't use official yet stupid `deno_install` script - `setup-util-devel`: add `automake` for macos - `setup-util-gawk`: add more package systems - `setup-util-gsed`: add bottles for macos, and add uninstall build support - `setup-util`: rename `DOWNLOAD_BUILD_EVAL` to `DOWNLOAD_BUILD_INSTALL` and add `DOWNLOAD_BUILD_UNINSTALL`, affected `setup-util-bash`, `setup-util-gawk`, `setup-util-gsed` - `sparse-vault`: fix broken awk script, use `echo-regexp` instead - `sparse-vault`: fix broken macos version fetching - `styles.bash` make code styles more readable on light theme by swapping from dim to intense black, add code error, info, and notice styles - `unziptar`: fix alpine compatibility - `what-is-my-(dns|gateway|inteface|ip|subnet)`: rewrote and fixed - add `.gitattributes` to prevent formatting issues in tests - add `echo-escape-backslashes` and remove its functionality from `echo-escape-regexp-replacement` for compat with JavaScript Regular Expression - add `fs-diff` to compute differences between files - add `get-file` and `setup-util-file` to fix `unziptar` compat on systems without `file` preinstalled - add `setup-util-frawk` - add tests to `echo-math`, `echo-(html|url)-(coder|encode|decode)`, `echo-magnet-hash`, `fs-(bytes|kilobytes|megabytes)`, `get-url-(domain|protocol)` needs more testing: - `get-opensuse-release` - `mount-helper` - `setup-dns` - `setup-server` - `setup-util-gsed` on fresh macos - `unziptar` on macos dmg files squashes: - be4b3ce - ada62e0 - 6860035 - d86e060 - 2b24208 - 1fdae98 - 4506b7f - 6618689 - 7022dba - fe5050b - 0e02dc9 - 7c7265d - bb507bd - 818d5c0 - b91656b - 96d91bc - 6838a37 - 7c7ed59 - a1f96ea - 2e4d9da - e2294c4 - 2a83b20 - da3cf12 - d334b8f - 86de699 - 35593fe - 62d42ca - c723ea2 - e68e2e0 - 7deab4e - ecee164 - 7cc4031 - e035a22 - e9a955a
balupton
added a commit
that referenced
this issue
Sep 17, 2024
/close #249 #241 #239 changes: - replace common gsed/sed/sd/ripgrep/rg/grep/teip usage with new `echo-regexp` that uses deno, affects `convert-helper`, `config-helper`, `echo-magnet-hash`, `fs-(bytes|kilobytes|megabytes)`, `fs-speed`, `get-devices`, `get-local-to-remote`, `get-opensuse-release`, `get-volumes`, `git-helper`, `is-bash-version-outdated`, `mount-helper`, `network-interface`, `setup-dns`, `setup-util`, `setup-util-1password-cli`, `setup-util-nerd-fonts`, `setup-util-node`, `unziptar`, `what-is-my-(dns|gateway|interface|ip|submit)`, `get-codec`, `is-prefix`, `is-suffix`, `itunes-owners`, `mac-address-helper`, `setup-server` - `bash.bash`: - simplify eval statements and support spaces in paths - add explicit names for `__print*` helpers, and slightly change their functionality to be aligned with expectations - **please change `__print_line ...` to `__print_lines ...` as in the future `__print_line` may not support arguments** - **removed deprecated non `__` prefixed aliases, e.g. `require_globstar` is now only `__require_globstar`** - `ci/dorothy-workflow`: don't test PRs until review requested - `ci/dorothy`: don't eslint as no Node.js code - `ci/vscode`: add an awk syntax highlighting extension, and enable Deno - `command-working`: give details of failures, add `dash` to exceptions - `dorothy-config`/`config-helper`: significant work to better handle multiple instances (result is no longer garbled), output and process is now clearer, includes multiple strategies for multiple instances, uses new `echo-regexp` to replace `sd/rg/teip/grep` combo - `dorothy` minimum deps now only `jq` and `deno`s, - `down`: add `--url=...` support - `echo-gnu-command`: fix it not outputting the command after installation - `echo-if-empty`/`get-devices`/`get-local-to-remote`: fix sporadic success by adding missing `--stdin` - `echo-lines`: now support `--[no-]quote` flags as alias for prior `--[no-]quoted` flag - `echo-math`/`echo-(html|url)-(encode|decode)`: rewrote and added tests, now they work correctly, in the latter case `echo-(html|url)-coder` is now used - `echo-wrap`: now awk compatible, removing dependency on `gawk` - `eval-tester`: output escaped comparison if special characters were used - `fs-(bytes|kilobytes|megabytes)`: use `dust` instead of `du` to achieve macos support, also fix crash in `fs-megabytes` - `fs-rm`: fix trash error on alpine, as trash isn't available on alpine - `get-codec`: removed `--trim` and made it default (prior verbosity available via `--verbose`), now sensible error if `ffprobe` is not available - `get-codec`: rewrote and fixed - `get-filesystem`: no longer fail silently when no args are provided - `get-local-to-remote`: improve styling - `get-opensuse-release`: simpler technique - `get-url-(domain|protocol)` split into `get-url-(domain|protocol).ts` and add tests - `is-match`: deprecated and moved in beta commands, as it is replaced by `echo-regexp -q` which it now uses internally - `mac-address-helper`: fixed broken `new` functionality, and improved formatting of outputs - `set-hostname`/`setup-hosts`/`setup-shell`: fix break when gsed isn't available, now uses the `echo-gnu-command` helper - `setup-util-1password`: install stable instead of beta, as stable is now stable - `setup-util-deno`: add apk, uninstall snap (as snap does not allow script execution), use official arm builds as that is now a thing, don't use official yet stupid `deno_install` script - `setup-util-devel`: add `automake` for macos - `setup-util-gawk`: add more package systems - `setup-util-gsed`: add bottles for macos, and add uninstall build support - `setup-util`: rename `DOWNLOAD_BUILD_EVAL` to `DOWNLOAD_BUILD_INSTALL` and add `DOWNLOAD_BUILD_UNINSTALL`, affected `setup-util-bash`, `setup-util-gawk`, `setup-util-gsed` - `sparse-vault`: fix broken awk script, use `echo-regexp` instead - `sparse-vault`: fix broken macos version fetching - `styles.bash` make code styles more readable on light theme by swapping from dim to intense black, add code error, info, and notice styles - `unziptar`: fix alpine compatibility - `what-is-my-(dns|gateway|inteface|ip|subnet)`: rewrote and fixed - add `.gitattributes` to prevent formatting issues in tests - add `echo-escape-backslashes` and remove its functionality from `echo-escape-regexp-replacement` for compat with JavaScript Regular Expression - add `fs-diff` to compute differences between files - add `get-file` and `setup-util-file` to fix `unziptar` compat on systems without `file` preinstalled - add `setup-util-frawk` - add tests to `echo-math`, `echo-(html|url)-(coder|encode|decode)`, `echo-magnet-hash`, `fs-(bytes|kilobytes|megabytes)`, `get-url-(domain|protocol)` needs more testing: - `get-opensuse-release` - `mount-helper` - `setup-dns` - `setup-server` - `setup-util-gsed` on fresh macos - `unziptar` on macos dmg files squashes: - be4b3ce - ada62e0 - 6860035 - d86e060 - 2b24208 - 1fdae98 - 4506b7f - 6618689 - 7022dba - fe5050b - 0e02dc9 - 7c7265d - bb507bd - 818d5c0 - b91656b - 96d91bc - 6838a37 - 7c7ed59 - a1f96ea - 2e4d9da - e2294c4 - 2a83b20 - da3cf12 - d334b8f - 86de699 - 35593fe - 62d42ca - c723ea2 - e68e2e0 - 7deab4e - ecee164 - 7cc4031 - e035a22 - e9a955a
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
gawk is now needed within dorothy, and its usage in echo-wrap requires it as the fresh macos awk version is not suitable. Unfortunately, gawk is now bundled and needs to be installed and currently setup-until-gawk does not reference the homebrew bottles, so that needs to be done such that Dorothy installs on fresh macos don't immediately start installing homebrew.
https://github.com/bevry/dorothy/blob/master/commands/setup-util-gawk
https://github.com/bevry/dorothy/blob/master/commands/setup-util-bash
I also believe because of the new echo-gnu-command we can purge the symlink junk inside setup-util-gawk and setup-util-gsed as that symlink junk would just confuse setup-util and echo-gnu-command.
it seems the symlink junk was for alpine so it's removal will need to be validated with alpine
The text was updated successfully, but these errors were encountered: