Lots of updates, mostly focusing on the CI & build system, take 2#15
Open
cooljeanius wants to merge 91 commits intoflightaware:masterfrom
Open
Lots of updates, mostly focusing on the CI & build system, take 2#15cooljeanius wants to merge 91 commits intoflightaware:masterfrom
cooljeanius wants to merge 91 commits intoflightaware:masterfrom
Conversation
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
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
derp
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
- 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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is another attempt at doing PR #14, except from a non-default branch this time. To summarize, changes here include:
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."