Skip to content

Commit

Permalink
(ci) stop using brew test
Browse files Browse the repository at this point in the history
Apparently `brew test` tries to build dependencies of `brew bundle`.
And it seems like it depends on ronn, which is [dead][1]. The worst
part is that ronn depends on rdiscount 2.2.0.2, which fails to build.
Didn't dig too much into it, but [seems like][2] it's also not in high
priority. Homebrew issue tracker is silent and I don't feel safe to
report anything there, so with this commit going to do simple thing -
avoid using `brew test`. Since I don't have `brew` right now, I am
just going to see if `brew --prefix` does what I expect. Will turn it
into real test later.

[1]: rtomayko/ronn#103
[2]: davidfstr/rdiscount#148
  • Loading branch information
d12frosted committed Nov 19, 2021
1 parent c41625e commit bb62328
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 32 deletions.
7 changes: 1 addition & 6 deletions .github/workflows/emacs-26.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,4 @@ jobs:
run: brew install ./Formula/[email protected] ${{ matrix.build_opts }}

- name: Test installation
if: contains(matrix.build_opts, '--HEAD') == false
run: brew test ./Formula/[email protected]

- name: Test installation (--HEAD)
if: contains(matrix.build_opts, '--HEAD')
run: brew test ./Formula/[email protected] --HEAD
run: $(brew --prefix)/bin/emacs --batch --eval='(print (+ 2 2))'
7 changes: 1 addition & 6 deletions .github/workflows/emacs-27.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,4 @@ jobs:
run: brew install ./Formula/[email protected] ${{ matrix.build_opts }}

- name: Test installation
if: contains(matrix.build_opts, '--HEAD') == false
run: brew test ./Formula/[email protected]

- name: Test installation (--HEAD)
if: contains(matrix.build_opts, '--HEAD')
run: brew test ./Formula/[email protected] --HEAD
run: $(brew --prefix)/bin/emacs --batch --eval='(print (+ 2 2))'
7 changes: 1 addition & 6 deletions .github/workflows/emacs-28.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,9 +46,4 @@ jobs:
run: brew install ./Formula/[email protected] ${{ matrix.build_opts }}

- name: Test installation
if: contains(matrix.build_opts, '--HEAD') == false
run: brew test ./Formula/[email protected]

- name: Test installation (--HEAD)
if: contains(matrix.build_opts, '--HEAD')
run: brew test ./Formula/[email protected] --HEAD
run: $(brew --prefix)/bin/emacs --batch --eval='(print (+ 2 2))'
7 changes: 1 addition & 6 deletions .github/workflows/emacs-29.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,9 +46,4 @@ jobs:
run: brew install ./Formula/[email protected] ${{ matrix.build_opts }}

- name: Test installation
if: contains(matrix.build_opts, '--HEAD') == false
run: brew test ./Formula/[email protected]

- name: Test installation (--HEAD)
if: contains(matrix.build_opts, '--HEAD')
run: brew test ./Formula/[email protected] --HEAD
run: $(brew --prefix)/bin/emacs --batch --eval='(print (+ 2 2))'
7 changes: 1 addition & 6 deletions .github/workflows/emacs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,4 @@ jobs:
run: brew install Aliases/$(readlink Aliases/emacs-plus) ${{ matrix.build_opts }}

- name: Test installation
if: contains(matrix.build_opts, '--HEAD') == false
run: brew test Aliases/$(readlink Aliases/emacs-plus)

- name: Test installation (--HEAD)
if: contains(matrix.build_opts, '--HEAD')
run: brew test Aliases/$(readlink Aliases/emacs-plus) --HEAD
run: $(brew --prefix)/bin/emacs --batch --eval='(print (+ 2 2))'
4 changes: 2 additions & 2 deletions .github/workflows/online.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
run: brew install emacs-plus

- name: Test installation
run: brew test emacs-plus
run: $(brew --prefix)/bin/emacs --batch --eval='(print (+ 2 2))'

install:
runs-on: ${{ matrix.os }}
Expand All @@ -43,4 +43,4 @@ jobs:
run: brew install d12frosted/homebrew-emacs-plus/emacs-plus

- name: Test installation
run: brew test emacs-plus
run: $(brew --prefix)/bin/emacs --batch --eval='(print (+ 2 2))'

0 comments on commit bb62328

Please sign in to comment.