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.

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

Jump to bottom

build: use clang for public build #2730

Merged
merged 14 commits into from
Nov 3, 2021
Merged

build: use clang for public build #2730

merged 14 commits into from
Nov 3, 2021

Commits on Nov 1, 2021

  1. cmake/oss: fix typo on config for v8 

    Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    ac99c28 View commit details
    Browse the repository at this point in the history

  2. cmake: ignore warnings when compiling seastar 

    For public build, ignore errors coming from warnings for seastar
compilation

Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    90d0a6f View commit details
    Browse the repository at this point in the history

  3. build: move to use clang 

    Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    1f81767 View commit details
    Browse the repository at this point in the history

  4. build: add dependencies to build with clang 

    Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    52b71e2 View commit details
    Browse the repository at this point in the history

  5. build: add toolchain image for public build 

    Adds a Dockerfile of an image that has all compilation dependencies.
This installs clang-12, libstdc++11.2.1 on fedora:34

Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    d286f3c View commit details
    Browse the repository at this point in the history

  6. build: fix CCACHE_DIR check for public build 

    Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    1f4c448 View commit details
    Browse the repository at this point in the history

  7. build: add DEPOT_TOOLS_DIR path for public build 

    Add a DEPOT_TOOLS_DIR variable that contains the path to where it is
placed and have it be `/opt/depot_tools` by default, to make it
consistent with the way other dependencies are installed for the public
build.

Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    8eaa887 View commit details
    Browse the repository at this point in the history

  8. build: change message install-dependencies shows 

    For the public build, change message shown when installing dependencies.

Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    3cd4b49 View commit details
    Browse the repository at this point in the history

  9. readme: update 'build manually' instructions 

    Public build assumes clang-12. Also adds a reference to instructions for
using the pre-baked docker image with the frozen toolchain

Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    96d0a73 View commit details
    Browse the repository at this point in the history

  10. schemaregistry: bump up version of avro 

    Point to newest version on vectorizedio fork.

Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    22acdcf View commit details
    Browse the repository at this point in the history

  11. gha: use clang in redpanda workflow 

    Make use of the container image introduced recently, which uses clang to
build redpanda.

Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    e5a23ab View commit details
    Browse the repository at this point in the history

  12. gha: change ccache key for redpanda workflow 

    Use git ref as part of the key and remove timestamp. The current limit
from Github is 5GB for all cached files across a repository. With this
change, there is only one ccache file uploaded per branch, instead of
one for each distinct build. This results in having one for the main
branch, as well as one per PR.

Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    8d4a02a View commit details
    Browse the repository at this point in the history

  13. gha: set fs.aio-max-nr on host machine 

    Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    210e311 View commit details
    Browse the repository at this point in the history

  14. gha: move ccache command invocation to container 

    The host doesn't have ccache installed anymore

Signed-off-by: Ivo Jimenez <[email protected]>
    @ivotron
    ivotron committed Nov 1, 2021
    Configuration menu
    Copy the full SHA
    4534358 View commit details
    Browse the repository at this point in the history