-
Notifications
You must be signed in to change notification settings - Fork 17
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
cooljeanius
wants to merge
91
commits into
flightaware:master
Choose a base branch
from
cooljeanius:pr_14_take_2
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains 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
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] <[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
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.
"