Skip to content

Conversation

masatake
Copy link
Member

Signed-off-by: Masatake YAMATO [email protected]

@codecov
Copy link

codecov bot commented Jan 15, 2021

Codecov Report

Merging #2803 (b3661ed) into master (1460942) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2803   +/-   ##
=======================================
  Coverage   86.96%   86.96%           
=======================================
  Files         193      193           
  Lines       44000    44000           
=======================================
  Hits        38263    38263           
  Misses       5737     5737           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1460942...b3661ed. Read the comment docs.

@masatake masatake force-pushed the github-actions-fix-package-installation branch from 531a854 to f4d1a7e Compare January 15, 2021 15:15
@masatake masatake force-pushed the github-actions-fix-package-installation branch from f4d1a7e to b3661ed Compare January 15, 2021 15:49
@masatake masatake marked this pull request as ready for review January 15, 2021 16:09
@masatake
Copy link
Member Author

I don't understand why this is needed. However, failing to setup a test environment on MacOS is so critical for our development.
So I will merge this in soon. If you know well about MacOS, I would like you to review .github/workflows/testing-mac.yml .

@masatake masatake merged commit c026132 into universal-ctags:master Jan 15, 2021
Comment on lines +30 to +33
# libxml2 requires [email protected].
# However, the request is not satisfied with just installing libxml2.
brew install libxml2 || brew link --overwrite [email protected]
brew install autoconf automake pkg-config jansson libyaml bash gdb docutils
Copy link
Member

@leleliu008 leleliu008 Jan 16, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GitHub Actions VM macOS10.15 macOS11.0 have pre-installed [email protected] via HomeBrew, the error happens when [email protected] can be upgraded, so we should upgrade [email protected] first.

brew upgrade [email protected] || brew link --overwrite [email protected]

brew install libxml2 jansson libyaml autoconf automake pkg-config bash gdb docutils

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. You know so many things! I will make a pull request reflecting this suggestion.

masatake added a commit to masatake/ctags that referenced this pull request Jan 16, 2021
…teps

Quoted from @leleliu008's comment in universal-ctags#2803:

    GitHub Actions VM macOS10.15 macOS11.0 have pre-installed [email protected]
    via HomeBrew, the error happens when [email protected] can be upgraded, so we
    should upgrade [email protected] first.

Signed-off-by: Masatake YAMATO <[email protected]>
masatake added a commit to masatake/ctags that referenced this pull request Jan 16, 2021
…teps

Quoted from @leleliu008's comment in universal-ctags#2803:

    GitHub Actions VM macOS10.15 macOS11.0 have pre-installed [email protected]
    via HomeBrew, the error happens when [email protected] can be upgraded, so we
    should upgrade [email protected] first.

Signed-off-by: Masatake YAMATO <[email protected]>
masatake added a commit to masatake/ctags that referenced this pull request Jan 16, 2021
…teps

Quoted from @leleliu008's comment in universal-ctags#2803:

    GitHub Actions VM macOS10.15 macOS11.0 have pre-installed [email protected]
    via HomeBrew, the error happens when [email protected] can be upgraded, so we
    should upgrade [email protected] first.
masatake added a commit that referenced this pull request Jan 16, 2021
…teps

Quoted from @leleliu008's comment in #2803:

    GitHub Actions VM macOS10.15 macOS11.0 have pre-installed [email protected]
    via HomeBrew, the error happens when [email protected] can be upgraded, so we
    should upgrade [email protected] first.
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