Skip to content

Fix macos build github action#8525

Merged
Frenzie merged 5 commits into
masterfrom
unknown repository
Dec 26, 2021
Merged

Fix macos build github action#8525
Frenzie merged 5 commits into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Dec 7, 2021

Copy link
Copy Markdown

This change is Reviewable

@ghost

ghost commented Dec 7, 2021

Copy link
Copy Markdown
Author

this can be run on macos 12, and the mouse works. https://github.com/sayomelu/koreader/actions/runs/1549478342

fetch-depth: 0

- name: Homebrew install dependencies
# Compared to the README, adds ccache for faster compilation times and removes sh5sum to prevent some conflict with coreutils

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

sh5sum seems to have been removed in #5891

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

sh5sum is removed, see line 21 to 23

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, but you changed the comment without saying why it was justified to do so in the commit message or PR. ;-)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is for me to put in the squashed commit message, in other words.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I just follow the build guide, it does not have sh5sum https://github.com/koreader/koreader/blob/master/doc/Building.md#macos

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

And for future reference, that should be stated in the commit message. ;-) But I already said it was a remark from me to me. ^_^

@Frenzie Frenzie added this to the 2021.12 milestone Dec 7, 2021
@Frenzie

Frenzie commented Dec 7, 2021

Copy link
Copy Markdown
Member

Do you know what might be the case here?

[ 87%] Performing install step for 'zstd'
Configured with: --prefix=/Applications/Xcode_13.1.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode_13.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
creating pkgconfig
Installing includes
install: mkdir /Users/runner/work/koreader/koreader/base/thirdparty/zstd/build/x86_64-apple-darwin20.6.0/zstd-prefix/src/zstd-build/lib: File exists

@Frenzie Frenzie modified the milestones: 2021.12, 2022.01 Dec 18, 2021
@Frenzie

Frenzie commented Dec 25, 2021

Copy link
Copy Markdown
Member

@sayomelu Ping? ^_^

@ghost

ghost commented Dec 26, 2021

Copy link
Copy Markdown
Author

https://github.com/sayomelu/koreader/actions/runs/1623577082 the action works correctly on my repo, but i do not know why it not work on the main repo

@Frenzie

Frenzie commented Dec 26, 2021

Copy link
Copy Markdown
Member

An error like that might suggest some kind of caching-related issue?

Right now I'm seeing something very odd as well. The job has been done for a while, but GHA seems to think it's still going.

Screenshot_2021-12-26_12-27-16

@ghost

ghost commented Dec 26, 2021

Copy link
Copy Markdown
Author

@Frenzie Frenzie merged commit adea668 into koreader:master Dec 26, 2021
@Frenzie

Frenzie commented Dec 26, 2021

Copy link
Copy Markdown
Member

Fingers crossed. 👍

yparitcher added a commit to yparitcher/koreader-base that referenced this pull request Dec 27, 2021
There is a job for linters a matrix for the emulator builds and a matrix for cross compile

Fix macos build based on koreader/koreader#8525
yparitcher added a commit to yparitcher/koreader-base that referenced this pull request Jan 2, 2022
There is a job for linters a matrix for the emulator builds and a matrix for cross compile

Fix macos build based on koreader/koreader#8525
yparitcher added a commit to yparitcher/koreader-base that referenced this pull request Jan 2, 2022
Frenzie pushed a commit to koreader/koreader-base that referenced this pull request Jan 2, 2022
roygbyte pushed a commit to roygbyte/koreader-base that referenced this pull request Mar 3, 2022
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant