Skip to content

simplifying CI: ubuntu install #750

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.

Sign up for GitHub

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

Open
lemire wants to merge 11 commits into
base: main
Choose a base branch
from
Open

simplifying CI: ubuntu install #750

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
88b0e06
simplifying CI: ubuntu install
lemire Oct 6, 2024
89f4a45
Update .github/workflows/ubuntu_install.yml
lemire Oct 6, 2024
c3b8b25
Update .github/workflows/ubuntu_install.yml
lemire Oct 6, 2024
6b7a3ae
Update ubuntu_install.yml
lemire Oct 7, 2024
0e44b34
Merge branch 'main' into simplify_ubuntu_install
Nov 18, 2024
2ad2b3e
upgrading to 24.
Nov 18, 2024
fbe840f
let us go with defaults
Nov 18, 2024
f60614c
bumping everything to ubuntu 24
lemire Nov 18, 2024
69c012b
damn it. gcc 11 is the default?
lemire Nov 18, 2024
b3fa9aa
adding installation with clang
lemire Nov 18, 2024
488385b
Merge branch 'main' into simplify_ubuntu_install
lemire Nov 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update .github/workflows/ubuntu_install.yml 
Co-authored-by: Yagiz Nizipli <[email protected]>
  • Loading branch information
@lemire @anonrig
lemire and anonrig authored Oct 6, 2024
commit c3b8b251ad67a0272778f255c26228af9c60a5d0
2 changes: 1 addition & 1 deletion .github/workflows/ubuntu_install.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ jobs:
- name: Install
run: cmake --install build
- name: Prepare test package
run: cmake -DCMAKE_INSTALL_PREFIX:PATH=../../destination -S tests/installation
run: cmake -DCMAKE_INSTALL_PREFIX:PATH=../../destination -S tests/installation |
Copy link
Member

@anonrig anonrig Nov 18, 2024

Choose a reason for hiding this comment

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

We need to either pass CC=gcc-12 CXX=g++-12 or do sudo ln -sf /usr/bin/gcc-12 /usr/bin/gcc kind of symlink to fix the issue. Ubuntu 22 doesn't include gcc 12 or something newer

Copy link
Member Author

@lemire lemire Nov 18, 2024

Choose a reason for hiding this comment

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

Let us upgrade to ubuntu 24.

Messing around with paths is fragile.

-B buildbabyada
- name: Build test package
run: cmake --build buildbabyada
Expand Down