| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
| |
If this eventually poses a problem for unsupported branches we can fix
them directly.
Sponsored by: Innovate UK
Reviewed by: emaste
Differential Revision: https://reviews.freebsd.org/D53838
|
| |
|
|
|
|
|
| |
Reviewed by: imp
Obtained from: CheriBSD
Sponsored by: AFRL, DARPA
Pull Request: https://github.com/freebsd/freebsd-src/pull/1973
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Add code to fetch and decode CODEOWNERS and automatically @ people in
the review.
Create a new file, .github/path-rules.txt that has a set of paths to
match and specific warnings about that part of the tree. We'll use this
to try to wave off pull requests in certain areas of the tree, as well
as remind people when contrib is involved, etc.
Sponsored by: Netflix
|
| |
|
|
|
|
|
|
| |
Rather than superscript <sup>, for a nicer looking list.
Reviewed by: vexeduxr
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D53000
|
| |
|
|
|
|
|
|
| |
This is the latest supported version in Ubuntu 22.04
Reviewed by: vexeduxr, imp, emaste
Sponsored by: Arm Ltd
Pull Request: https://github.com/freebsd/freebsd-src/pull/1862
|
| |
|
|
|
| |
Reviewed by: imp
Differential Revision: https://reviews.freebsd.org/D51749
|
| |
|
|
|
|
|
|
| |
Make it easy for contributors to find the referenced information.
Reviewed by: ziaee
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D51346
|
| |
|
|
|
|
|
|
|
| |
Be explicit that the user should update their branch and re-push to
update the pull request.
Reviewed by: imp
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D51332
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Workflows triggered by the 'pull_request' event can't have write
permissions. With write permissions a malicious pull request can alter
or create a workflow that either leaks the GITHUB_TOKEN with the write
permissions or do malicious things in the workflow itself.
The 'pull_request_target' event on the other hand allows workflows to
run with write permissions but runs on the merge base of the PR, this
way a pull request that alters such a workflow will not have it's code
run until it's merged.
Signed-off-by: Ahmad Khalifa <ahmadkhalifa570@gmail.com>
Reviewed by: imp, emaste
Pull Request: https://github.com/freebsd/freebsd-src/pull/1581
|
| |
|
|
|
|
| |
It shouldn't have been pushed
This reverts commit d16397c1737f5f977e893daebde0e7009d6a9950.
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a new 'checklist' workflow that checks the commit messages on pull
requests. Currently, the workflow creates a comment on the pull request
if any of these conditions are hit:
- Missing Signed-off-by
- Malformed Signed-off-by
- Bad email (i.e *noreply*)
Reviewed by: emaste, imp
Pull request: https://github.com/freebsd/freebsd-src/pull/1570
Signed-off-by: Ahmad Khalifa <ahmadkhalifa570@gmail.com>
|
| |
|
|
|
|
| |
Signed-off-by: Ahmad Khalifa <ahmadkhalifa570@gmail.com>
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/1538
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Only fetch the commits we need instead of fetching the entire history.
Unfortunately there doesn't seem to be a way to add 1 to the number of
commits without an extra step, so do it in a new step and pass the
information onto $GITHUB_ENV so it can be used later.
Signed-off-by: Ahmad Khalifa <ahmadkhalifa570@gmail.com>
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/1538
|
| |
|
|
|
|
| |
Reviewed by: emaste, jhb
Sponsored by: Arm Ltd
Pull Request: https://github.com/freebsd/freebsd-src/pull/1399
|
| |
|
|
|
|
|
|
|
| |
Ubuntu 24.04 is the most recent LTS release. Use it and the previous
22.04 LTS for cross build testing.
Reported by: emaste, jhb
Sponsored by: Arm Ltd
Pull Request: https://github.com/freebsd/freebsd-src/pull/1399
|
| |
|
|
| |
Pull Request: https://github.com/freebsd/freebsd-src/pull/1212
|
| |
|
|
|
| |
GitHub is emitting a warning that v3 is deprecated due to using
Node.js 16.
|
| |
|
|
|
|
|
| |
Sponsored by: Arm Ltd
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/878
|
| |
|
|
|
|
|
| |
Sponsored by: Arm Ltd
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/878
|
| |
|
|
|
|
| |
Let's see if we can get the style issues flagged inline.
Sponsored by: Netflix
|
| |
|
|
|
| |
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/1064
|
| |
|
|
|
|
|
|
|
|
|
|
| |
We don't need to run this on branch pushes, just pull requests. It's
designed to be a gross filter for incoming commits, not something
perfect we need to keep green. It also doesn't work quite right for
branch pushes anyway and needs adjustment.
Also remove some debugging information. We don't need it anymore.
Noticed by: jrtc27
Sponsored by: Netflix
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Connect the checkstyle9.pl script to a github action. This will provide
feedback to people submitting changes when the style is grossly wrong. And
can provide other automated feedback for the commit message in the future.
It already catches the github noreply author.
It pulls the full repo to do this. Optimizations welcome. After messing
with that for a few hours, I decided to punt and commit the slow, working
version and let someone else optimize from here.
Sponsored by: Netflix
|
| |
|
|
|
|
|
|
| |
Building the tools is quick so we should provide coverage of this to
ensure it keeps working, especially on non-FreeBSD.
Reviewed by: emaste, arichardson
Differential Revision: https://reviews.freebsd.org/D39073
|
| | |
|
| |
|
|
|
|
|
|
|
| |
GitHub is emitting a warning that v2 is deprecated due to using
Node.js 12.
Reported by: GitHub
Reviewed by: arichardson, emaste
Differential Revision: https://reviews.freebsd.org/D37769
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Homebrew has added LLVM 14 and made that the default version, but GitHub
continues to install LLVM 13 for now, so it ends up only accessible via
the versioned name and not the unversioned one. We also add an explicit
installation of llvm@13 so that, if GitHub updates the image to using
LLVM 14, the action continues to work, albeit slightly more slowly. This
also ensures the compiler label remains correct rather than outdated, as
has occurred in the past, and that we don't get new versions of LLVM
before we're ready for them, which is especially relevant for stable
branches. This all mirrors how the Ubuntu jobs are configured.
|
| |
|
|
|
| |
Reviewed by: uqs
Pull Request: https://github.com/freebsd/freebsd-src/pull/599
|
| |
|
|
|
|
|
| |
This required adding an explicit os list to the matrix.
Reviewed by: uqs
Pull Request: https://github.com/freebsd/freebsd-src/pull/599
|
| |
|
|
|
|
|
|
|
|
|
| |
- Drop clang 9 build, and switch the remaining ubuntu build to
the more modern clang 12.
- Update the label for the macos-latest builds which are now using
clang 13 rather than clang 12.
Reviewed by: uqs
Pull Request: https://github.com/freebsd/freebsd-src/pull/599
|
| |
|
|
|
|
|
| |
This chases a homebrew change in the default clang version.
Reviewed by: arichardson
Differential Revision: https://reviews.freebsd.org/D31231
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GitHub removed Clang 9 from the 20.04 image[1], breaking this build.
Thus, manually add the specific versioned packages we need for the
Ubuntu jobs to ensure they're installed. Note that we don't do the same
for macOS, as Homebrew does not allow multiple llvm@N to co-exist,
giving an error if you attempt to install a second one. In practice we
don't actually use the compiler field here for anything other than the
build name, it's only the cross-bindir that matters, so when it
eventually moves to 12 the name will get confusing but the job will
still work.
MFC after: immediately
[1] https://github.com/actions/virtual-environments/commit/15a610677be406d250c1f6732b03c8b87e693a0a
|
| |
|
|
|
|
|
|
|
| |
All cross-building patches have been merged to stable/13 so it should
also build fine on macOS+Linux.
Reviewed By: uqs
MFC after: immediately
Differential Revision: https://reviews.freebsd.org/D29831
|
| |
|
|
|
|
|
|
| |
Error: llvm 11.0.0 is already installed
Also make the linking failure non-fatal:
Error: The `brew link` step did not complete successfully
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also fix the run by setting up the environment in non-deprecated way.
Always run with --debug to understand better what sort of stuff is happening in
the background. Also split out the bmake bootstrap stage (takes about 31s on
ubuntu, but 1m14 on macOS?)
Drops the dependency on coreutils (realpath, nproc) and thus (?) fixes macOS to
be just as fast (4 logical cores vs 2 physical cores before, go figure.)
Reviewed by: arichardson
Notes:
svn path=/head/; revision=368056
|
|
|
This builds the kernel-toolchain target and an amd64 GENERIC kernel on
Ubuntu 18.04, 20.04 and the latest macOS to ensure that new changes
don't regress building on non-FreeBSD hosts.
Reviewed By: emaste, lwhsu
Differential Revision: https://reviews.freebsd.org/D26512
Notes:
svn path=/head/; revision=366074
|