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

Re-include c89 job in CI. #91

Open
duesee opened this issue Jul 6, 2022 · 0 comments
Open

Re-include c89 job in CI. #91

duesee opened this issue Jul 6, 2022 · 0 comments
Labels

Comments

@duesee
Copy link
Contributor

duesee commented Jul 6, 2022

As discussed with @franziskuskiefer, I'll remove the c89 job from CI for now. This job can be re-included when the issues blocking a successful build are resolved.

# XXX: This fails because c89 is utterly broken. We might want to do this at
  #      some point.
  c89:
    runs-on: ubuntu-18.04
    env:
      CC: gcc-4.8
      CXX: g++-4.8
    steps:
      - name: Output name
        id: vars
        run: |
          echo ::set-output name=short_sha::${GITHUB_SHA: -8}
      - name: Setup
        run: |
          sudo apt-get update
          sudo apt-get install ninja-build gcc-4.8 g++-4.8
      - uses: actions/checkout@v2
      - name: Debug Build
        run: ./mach build -v ${{ matrix.options }} -e c89 || true
      # - name: Release Build
      #   run: ./mach build --release -v ${{ matrix.options }} -e c89
      # - name: Install
      #   run: |
      #     mkdir pkg-release
      #     ./mach install -p $PWD/pkg-release -c Release
      #     mkdir pkg-debug
      #     ./mach install -p $PWD/pkg-debug
      # - name: Upload Artifacts
      #   uses: actions/upload-artifact@v2
      #   with:
      #     name: linux-x64-gcc48-c89-${{ steps.vars.outputs.short_sha }}${{ matrix.options }}
      #     path: |
      #       pkg-debug
      #       pkg-release
      #       build/Debug
      #       build/Release
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

No branches or pull requests

1 participant