[InterNetNews/inn] 884d30: Add GitHub Actions configuration

Russ Allbery noreply at github.com
Mon Sep 13 01:09:51 UTC 2021


  Branch: refs/heads/main
  Home:   https://github.com/InterNetNews/inn
  Commit: 884d30e45ee84ec2fd9607e0d7d86acf9dbe9a08
      https://github.com/InterNetNews/inn/commit/884d30e45ee84ec2fd9607e0d7d86acf9dbe9a08
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    A .github/dependabot.yml
    A .github/workflows/build.yaml
    M MANIFEST
    A ci/README.md
    A ci/apt-packages
    A ci/install
    A ci/test

  Log Message:
  -----------
  Add GitHub Actions configuration

Configure a build workflow running on pushes and pull requests
that does the same build logic as support/mksnapshot does, except
for testing --enable-largefiles (which is irrelevant on GitHub
Actions runners).

Tell dependabot to send PRs for updates to any workflow actions
we use (currently only checkout).


  Commit: b67efaaddc8899100daec820bfd9ccc93beea473
      https://github.com/InterNetNews/inn/commit/b67efaaddc8899100daec820bfd9ccc93beea473
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    M backends/actsync.c
    M contrib/auth_pass.c
    M frontends/rnews.c
    M innd/rc.c
    M lib/conffile.c
    M nnrpd/post.c

  Log Message:
  -----------
  Check return status of fgets

The version of Ubuntu used for GitHub Actions requires the return
status of fgets be tested or -Wunused-result fails.  Add explicit
checks to those fgets invocations that weren't already tested.


  Commit: 132fe32a3ae66e51cb3dfdc92c0f4ff95b00977a
      https://github.com/InterNetNews/inn/commit/132fe32a3ae66e51cb3dfdc92c0f4ff95b00977a
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    M backends/innbind.c
    M contrib/reset-cnfs.c
    M innd/rc.c
    M nnrpd/misc.c

  Log Message:
  -----------
  Check status on all write calls

Similar to fgets, the version of Ubuntu on which GitHub Actions
is running wants the return status of write to be checked for
-Wunused-result.


  Commit: aaf1cc352ec6a5aa0f085e15b2c972a64bfa5988
      https://github.com/InterNetNews/inn/commit/aaf1cc352ec6a5aa0f085e15b2c972a64bfa5988
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    M nnrpd/commands.c

  Log Message:
  -----------
  Check status on all read calls

Similar to write, the version of Ubuntu on which GitHub Actions
is running wants the return status of read to be checked for
-Wunused-result.


  Commit: 0f11ff4a6227204157f67d1dccabbb039fa63d1d
      https://github.com/InterNetNews/inn/commit/0f11ff4a6227204157f67d1dccabbb039fa63d1d
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    M nnrpd/newnews.c
    M nnrpd/nnrpd.c

  Log Message:
  -----------
  Check status on all nice calls

Similar to write, the version of Ubuntu on which GitHub Actions
is running wants the return status of nice to be checked for
-Wunused-result.


  Commit: 43833ca01897223668e05eef12b0075c626a3039
      https://github.com/InterNetNews/inn/commit/43833ca01897223668e05eef12b0075c626a3039
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    M innfeed/endpoint.c
    M innfeed/main.c

  Log Message:
  -----------
  Remove innfeed debugWrites support

I have never used this in decades of working on INN, and it was
generating compiler warnings.  It's almost as easy to add in
approrpriate print statements than enable this flag, which required
a source modification anyway.


  Commit: 4ea501408c159e22fd59555318fad9fa1627a884
      https://github.com/InterNetNews/inn/commit/4ea501408c159e22fd59555318fad9fa1627a884
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    M frontends/ovdb_init.c

  Log Message:
  -----------
  Check status on all chdir calls

Similar to write, the version of Ubuntu on which GitHub Actions
is running wants the return status of chdir to be checked for
-Wunused-result.


  Commit: 85adb71d0db0eea44973986679dbaf11c0fd3067
      https://github.com/InterNetNews/inn/commit/85adb71d0db0eea44973986679dbaf11c0fd3067
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    M innfeed/misc.c
    M innfeed/misc.h

  Log Message:
  -----------
  Remove dead innfeed dump_core function

This was apparently intended for use with a die handler, but
there was no mechanism to call it.


  Commit: 62d35d9bc560cd57ef96e8c55f2312e62bda7293
      https://github.com/InterNetNews/inn/commit/62d35d9bc560cd57ef96e8c55f2312e62bda7293
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    M backends/archive.c
    M backends/batcher.c
    M backends/buffchan.c

  Log Message:
  -----------
  Check status on all freopen calls

Similar to write, the version of Ubuntu on which GitHub Actions
is running wants the return status of freopen to be checked for
-Wunused-result.


  Commit: 5bf57d12c197e7d2ae9e59630a2dfaed32a6e638
      https://github.com/InterNetNews/inn/commit/5bf57d12c197e7d2ae9e59630a2dfaed32a6e638
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    M backends/filechan.c

  Log Message:
  -----------
  Check status on all chown calls

Similar to write, the version of Ubuntu on which GitHub Actions
is running wants the return status of chown to be checked for
-Wunused-result.


  Commit: d7d9ae3bc3f6c1ffca0bc0085a642a2bffaa3c00
      https://github.com/InterNetNews/inn/commit/d7d9ae3bc3f6c1ffca0bc0085a642a2bffaa3c00
  Author: Russ Allbery <eagle at eyrie.org>
  Date:   2021-09-12 (Sun, 12 Sep 2021)

  Changed paths:
    A .github/dependabot.yml
    A .github/workflows/build.yaml
    M MANIFEST
    M backends/actsync.c
    M backends/archive.c
    M backends/batcher.c
    M backends/buffchan.c
    M backends/filechan.c
    M backends/innbind.c
    A ci/README.md
    A ci/apt-packages
    A ci/install
    A ci/test
    M contrib/auth_pass.c
    M contrib/reset-cnfs.c
    M frontends/ovdb_init.c
    M frontends/rnews.c
    M innd/rc.c
    M innfeed/endpoint.c
    M innfeed/main.c
    M innfeed/misc.c
    M innfeed/misc.h
    M lib/conffile.c
    M nnrpd/commands.c
    M nnrpd/misc.c
    M nnrpd/newnews.c
    M nnrpd/nnrpd.c
    M nnrpd/post.c

  Log Message:
  -----------
  Merge pull request #166 from rra/ci

Enable GitHub Actions for testing


Compare: https://github.com/InterNetNews/inn/compare/a001a5fb9612...d7d9ae3bc3f6


More information about the inn-committers mailing list