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

Lots of updates, mostly focusing on the CI & build system, take 2 #15

Open
wants to merge 91 commits into
base: master
Choose a base branch
from

Conversation

cooljeanius
Copy link

@cooljeanius cooljeanius commented Nov 1, 2024

This is another attempt at doing PR #14, except from a non-default branch this time. To summarize, changes here include:

  • turning on dependabot updates
  • addition of a codeql GitHub Actions workflow
  • conversion of the Travis CI workflow to GitHub Actions
  • minor gitignore updates
  • minor build system tweaks

If some of these changes aren't wanted, that's ok, you can just close this PR, and I'll continue to maintain the changes in my fork. On the other hand, if you do decide that you want these changes, I would recommend squashing before merging, as there are a lot of individual commits. I tried squashing commits on my own locally, but it said, "Unable to squash. Squashing replays all commits up to the last one required for the squash. A merge commit cannot exist among those commits."

remove concurrency
do everything in tnm
…actions-20230821-130214

Convert cooljeanius/scotty to GitHub Actions
attempting to debug
more attempting to debug
more Linux packages
I'd have to learn how rpcgen(1) works in order to fix this correctly
look in another place for Tcl
no longer need to locate Tcl
ok now let's debug THIS build failure
replace sudo with explicit dist specification
let's get a bit more output here
ignore more logs
set some additional Homebrew-related environment variables
add ca-certificates
add some extra verbosity to brew
try speeding up the build with caching and parallelization
let's make that global, not osx
more tweaking
some additional debugging messages
only do "-Icompat" on linux
let's see about skipping <config.h> on Linux
cooljeanius and others added 29 commits November 1, 2023 16:10
just remove the cpan part entirely
looks like gem needs sudo
remove some rubygems
actually, maybe if we try moving some of the previously-failing installations to the end, maybe they'll work there?
ok, so rubygems still don't work there...
messing around with packages
more messing with packages
tinker tinker tinker
la de da de da
tinker some more
remove Tkx from cpan
remove Tcl::pTK from cpan
a few more cpan package changes
ok, so it looks like those gems just really won't work...
override CFLAGS everywhere when actually building
add flag to find rpc headers
need rpc headers in 1 more place
ignore errors
add some more flags
try increasing codeql coverage
ok so maybe it isn't actually called "net-snmp"?
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
…ons/checkout-4

Bump actions/checkout from 3 to 4
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2 to 3.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@v2...v3)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
…ub/codeql-action-3

Bump github/codeql-action from 2 to 3
- silence some complaints from yamllint (some still remain, though)
- Update scotty.yml (a few more minor tweaks)
- 1 more instance of -Werror=implicit-int
- Update scotty.yml (reformat so it's readable enough for me to figure out where I need to edit to fix that latest failure)
- a fix for -Werror=implicit-int
- Update scotty.yml (try looking 1 more place for tcl on macOS)
- Update scotty.yml (oops, I forgot libtirpc doesn't install on macOS)
- Update scotty.yml (update brew dependencies)
- to be squashed upon merging
- autotools updates for 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant