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 and build system #14

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

Conversation

cooljeanius
Copy link

@cooljeanius cooljeanius commented Nov 2, 2023

Hi, I've been messing around in my fork of scotty for awhile and figured I'd try submitting a PR back upstream. 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.

(Edit: if you do decide that you want these changes, I would recommend squashing before merging, as there are a lot of individual 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 28 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] <support@github.com>
…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] <support@github.com>
…ub/codeql-action-3

Bump github/codeql-action from 2 to 3
@bovine bovine requested a review from resuna January 22, 2024 17:38
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.

None yet

2 participants