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

Migrate to GitHub Actions #434

Merged
merged 8 commits into from
Jan 2, 2021
Merged

Migrate to GitHub Actions #434

merged 8 commits into from
Jan 2, 2021

Conversation

KitsuneRal
Copy link
Member

@KitsuneRal KitsuneRal commented Dec 29, 2020

Closes #431.

@KitsuneRal KitsuneRal added the building/packaging Issues with CMake files or packaging label Dec 29, 2020
@PureTryOut
Copy link
Contributor

Maybe make clear in the commit message that this is about Travis/Github actions?

@KitsuneRal KitsuneRal changed the title Temporarily workaround linking error Migrate to GitHub Actions Dec 30, 2020
@KitsuneRal KitsuneRal self-assigned this Dec 30, 2020
@KitsuneRal
Copy link
Member Author

Maybe make clear in the commit message that this is about Travis/Github actions?

I just squashed that commit, as it didn't have anything useful and was entirely overwritten by subsequent ones. There was a stupid mistake of trying to compile C++ code with gcc instead of g++, and while the code compiles, the ultimate result is a linking error because gcc doesn't link the standard C++ library, only the standard C library.

There's no "job number" anymore but a textual description of the job
is even better.
Along adding update-api support DESTDIR and CMAKE_ARGS were moved to
the environment setup step in order to use `${{ runner.workspace }}`
for installation and not pollute `${{ github.workspace }}` where
libQuotient sources reside.
@KitsuneRal KitsuneRal force-pushed the kitsune-gha-ci branch 3 times, most recently from 03d1963 to 8fb231f Compare January 1, 2021 20:48
@KitsuneRal KitsuneRal merged commit 4831b61 into master Jan 2, 2021
@KitsuneRal KitsuneRal deleted the kitsune-gha-ci branch January 2, 2021 09:20
KitsuneRal added a commit that referenced this pull request Jan 2, 2021
Mostly taken from #434, except removed E2EE matrix variants
since E2EE is not ready on 0.6.x and is discouraged from using.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building/packaging Issues with CMake files or packaging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch from Travis CI to GitHub Actions
2 participants