add full cross compilation in travis #963
Merged
Conversation
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
cgxxv
pushed a commit
to cgxxv/logrus
that referenced
this pull request
Mar 25, 2022
* add full cross compilation in travis * reduce the travis build matrix * disable cross build for plan9 and nacl
kevinliu0430
added a commit
to 17media/logrus
that referenced
this pull request
Mar 17, 2025
* Fix typo in docs for New() * Fix error formatting based on best practices from Code Review Comments Signed-off-by: CodeLingo Bot <[email protected]> * Add a CallerPrettyfier callback to the json formatter * Add a CallerPrettyfier callback to the text formatter * Remove debug trace * Add and example for CallerPrettyfier * fix ReportCaller race condition * fix sync.Once usage instead of adding a mutex lock * Add WithContext * Add CHANGELOG for v1.4.0 * Rewrite if-else-if-else to switch statements * Add hook to send logs to custom writer sirupsen#678 * Fix some test conditions * Add Bytes() method to Entry, and use it to avoid double type cast * Add Go 1.12 to Travis CI build matrix * Got rid of IsTerminal call to reduce external dependencies * Removed golang.org/x/crypto refs * Moved moved unix-related parts into terminal * Updated travis.yml * Move terminal package fixes issue where terminal_check_notappengine.go can't access terminal package since terminal package is in an internal package * Test more platforms It would be 5 * 3 = 15 runs * return new entry for Entry.WithContext * Move files to main directory * Make isTerminal un-exported * remove field if val is empty string for func and file field in text formatter * Release 1.4.1 * Fix solaris build * Update x/sys/unix to fix AIX support * remove go 1.10 from ci build matrix * Update README.md * Add a checkTerminal for nacl to support running on play.golang.org * add full cross compilation in travis (sirupsen#963) * add full cross compilation in travis * reduce the travis build matrix * disable cross build for plan9 and nacl * fix build break for plan9 * Release 1.4.2 * tracking commit * tracking commit * add implementation and tests * update comments * update the readme * add a space back * wording shift * bump ci * dynamically space the level text * init the loggers in tests * avoid escapes! h/t @therealplato * len => RuneCount note: this is not currently easily testable without a larger diff that refactors the levels * its => it's * Fixed ineffectual assignment in test Don't assign l if it's not being checked/used afterwards. * Avoid unnecessary conversion No need to convert here. * readme: we have great maintainers now * return early This makes it easier to read / understand and is more idiomatic. * some minimal documentation for Logger.Writer{,Level} This also includes two examples extracted from the readme. * Add terminal_check_js.go file, for compatibility with GopherJS * ReadMe.md file adds The Third Formatter link * Fixed some typos in README.md Fixed a few typos and grammatical issues in the README.md. I hope this is helpful just to make a small improvement. * add Go 1.13 in travis * Disable modules, run on osx * go mod verify; go mod tidy * Enable all of these to see what fails * get some other deps * pull all the install into a single location * This should make gox a little nicer * Exclude go1.13.x from modules off, only build all on go1.13 modules on * Associate this example with what it's an example for * test the error to ensure there isn't an unexpected error in the test * deadcode * fix broken test * Force Quote Closed sirupsen#1005 * fix race conditions on entry closes sirupsen#1046 * run golangci-lint on travis * remove go1.11.x from travis build matrix * add x rights on travis/lint.sh * remove obsolete documentation * improve Logger.WithField documentation * Fix entity data bleed when using WithContext and WithTime Creates a copy of the data map when using WithContext to create a child entity. Without this, the data map of the parent entitiy, which is exposed in the entity struct, is shared between a parent and all children instances. This can create bugs of shared or overwritten data when a parent entity is used to make children in differing contexts, and behaves differently than `WithField` and its diritivites which does make a copy of the data. Additionally implements the same logic for WithTime, for API consistency in behavior. * Make Entry WithContext and WithTime copy tests more clear Clarifies the data used in the EntryWithContextCopiesData test and adds an equivalent test to verify the behavior of WithTime. * Add support for freebsd/arm64 * Fix typo * add caption-json-formatter * Remove annoying punctuation in Readme for better screen reader accessibility. One entry in the Logrus formatters list in the readme contained a lot of extraneous punctuation. When read with a screen reader, nothing but a bunch of question marks and weird symbol names could be heard, making the line impossible to understand. * readme: maintenance-mode * Create stale.yml * Update stale.yml * Only mark issues as stale for now until we go through backlog of PRs Only mark issues as stale for now until we go through backlog of PRs * Get right logrus package name * run CI for go 1.13 and 1.14 * Fix wrong caller * Removed useless files * Title updates Removed the non-breaking spaces in the ReadMe * resolved conflicts * create test to prove issue sirupsen#954 * fix race condition in entry * fix deadlock in previous entry race condition fix * remove errant whitespace * Add loggers that take functions as input * Revert sirupsen#1047 * Adds flag to disable quotes in TextFormatter * Adds additional test cases for DisableQuote * Change CRLF line endings to LF Signed-off-by: ialidzhikov <[email protected]> * update github.com/konsorten/go-windows-terminal-sequences dependency to v1.0.3 * complete documetation on TextFormatter.DisableQuote * update CHANGELOG.md with 1.5.0 and 1.6.0 version contents * Simplify checkIfTerminal for Windows Instead of relying on EnableVirtualTerminalProcessing from github.com/konsorten/go-windows-terminal-sequences which just calls GetConsoleMode, sets ENABLE_VIRTUAL_TERMINAL_PROCESSING and calls SetConsoleMode with the new modified mode, implement it directly inside checkIfTerminal. This also avoids the duplicate call to GetConsoleMode. * Improve tests for logger.*Fn functions * Update doc for new logger Update default formatter for new logger from JsonFormatter to TextFormatter * Add an API to plug a custom buffer free item mangement system * update changelog with v1.7.0 * Replace %v with %w for error https://golang.org/pkg/fmt/#Errorf * bump golang versions in travis ci * bump golangci-lint version * fix linter errors * one more linter error fixed * Add build tag to enable a successful build for zos * desactivate stale bot * migrate cross build target from bash script to mage * Remove dead panic in Entry.Panic [Entry.log itself panics][0] when the log level is set to PanicLevel, (and PanicLevel is always eneabled) so this second panic will never be reached. [0]: https://github.com/sirupsen/logrus/blob/8ae478eb8a850a54ea4915a2b572f377de1f9c7e/entry.go#L253 * migrate lint script to a mage target * add a test target in the magefile * update travis scripts * bump to go 1.16 * remove unneeded part in travis/install.sh * undo golang version bump * fix for entry data field race condition * update changelog * code and comments clean up * update changelog * fix race condition AddHook and traces * travis: run mage with -v to not discard output Before this: $ go run mage.go lint Error: running "/Users/sebastiaan/go/bin/golangci-lint run ./..." failed with exit code 1 exit status 1 $ go run mage.go test Error: running "go test -race -v ./..." failed with exit code 1 exit status 1 After this: $ go run mage.go -v lint Running target: Lint exec: /Users/sebastiaan/go/bin/golangci-lint run ./... entry.go:89:6: `iamNotUsed` is unused (deadcode) func iamNotUsed() { ^ Error: running "/Users/sebastiaan/go/bin/golangci-lint run ./..." failed with exit code 1 exit status 1 $ go run mage.go -v test Running target: Test exec: go test -race -v ./... === RUN TestRegister ... ? github.com/sirupsen/logrus/internal/testutils [no test files] FAIL Error: running "go test -race -v ./..." failed with exit code 1 exit status 1 Signed-off-by: Sebastiaan van Stijn <[email protected]> * move "mage" to a separate module Move the magefile related files to a submodule, so that it does not become a dependency for logrus itself. Signed-off-by: Sebastiaan van Stijn <[email protected]> * go.mod: update golang.org/x/sys to fix openbsd/mips64 on Go 1.16 This should hopefully fix cross-compile on openbsd/mips64 on Go 1.16 Building for GOOS=openbsd GOARCH=mips64 # golang.org/x/sys/unix Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/fcntl.go:26:42: undefined: Flock_t Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/ioctl.go:26:47: undefined: Winsize Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/ioctl.go:37:47: undefined: Termios Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/ioctl.go:55:42: undefined: Winsize Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/ioctl.go:61:42: undefined: Termios Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/syscall_openbsd.go:34:6: missing function body Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/syscall_unix_gc.go:12:6: missing function body Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/syscall_unix_gc.go:13:6: missing function body Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/syscall_unix_gc.go:14:6: missing function body Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/syscall_unix_gc.go:15:6: missing function body Error: ../../../go/pkg/mod/golang.org/x/[email protected]/unix/ioctl.go:61:42: too many errors Signed-off-by: Sebastiaan van Stijn <[email protected]> * CI: use GitHub Actions Use GitHub actions to run golang-ci-lint, cross, and test. The "Test()" target in Mage was a plain "go test -v ./...", and should be portable to other CI systems if needed; running it through the mage file effectively resulted in "compile a go binary to run go". The "Lint()" target in Mage relied on Travis CI setting up Golang-CI lint before it was executed, which required bash. Moving it to GitHub actions also allowed it to be run on Windows. Golang CI can still be run locally either through the mage file (which is kept for now), or running `golangci-lint run ./...` after installing golangci-lint. The "CrossBuild() Mage target is still used to perform cross-compile, as it contains code to generate the GOOS/GOARCH matrix, which makes it easier to run locally. Signed-off-by: Sebastiaan van Stijn <[email protected]> * improve documentation about timestamp format * update changelog * go.mod: github.com/stretchr/testify v1.7.0 Signed-off-by: Sebastiaan van Stijn <[email protected]> * Change godoc badge to pkg.go.dev badge * Add support for the logger private buffer pool. * fix race condition for SetFormatter and properly fix SetReportCaller race as well * Update README.md * ci: add go 1.17 to test matrix * bump golang.org/x/sys depency version * indicates issues as stale automatically * ci: run only on go 1.17 * reduce the list of cross build target * remove duplicated build constraints line * do not run the linter on windows * Improve Log methods documentation * bump version of golang.org/x/sys dependency fixes sirupsen#1332 * bump version of golangci-lint * update gopkg.in/yaml.v3 to v3.0.1 * Use text when shows the logrus output * update dependencies * Fix data race in hooks.test package * Add instructions to use different log levels for local and syslog This commit adds instructions to the syslog readme about how to send different log levels to local logging (`log.SetLevel`) and syslog hook. fixes sirupsen#1369 * This commit fixes a potential denial of service vulnerability in logrus.Writer() that could be triggered by logging text longer than 64kb without newlines. Previously, the bufio.Scanner used by Writer() would hang indefinitely when reading such text without newlines, causing the application to become unresponsive. * Scan text in 64KB chunks This commit fixes a potential denial of service vulnerability in logrus.Writer() that could be triggered by logging text longer than 64KB without newlines. Previously, the bufio.Scanner used by Writer() would hang indefinitely when reading such text without newlines, causing the application to become unresponsive. * Revert "Merge pull request sirupsen#1376 from ozfive/master" This reverts commit 6acd903, reversing changes made to e59b167. * Revert "Revert "Merge pull request sirupsen#1376 from ozfive/master"" This reverts commit 352781d. * fix panic in Writer Commit 766cfec introduced this bug by defining an incorrect split function. First it breaks the old behavior because it never splits at newlines now. Second, it causes a panic because it never tells the scanner to stop. See the bufio.ScanLines function, something like: ``` if atEOF && len(data) == 0 { return 0, nil, nil } ``` is needed to do that. This commit fixes it by restoring the old behavior and calling bufio.ScanLines but also keep the 64KB check in place to avoid buffering for to long. Two tests are added to ensure it is working as expected. Fixes sirupsen#1383 Signed-off-by: Paul Holzinger <[email protected]> --------- Signed-off-by: CodeLingo Bot <[email protected]> Signed-off-by: ialidzhikov <[email protected]> Signed-off-by: Sebastiaan van Stijn <[email protected]> Signed-off-by: Paul Holzinger <[email protected]> Co-authored-by: Albert Nigmatzianov <[email protected]> Co-authored-by: CodeLingo Bot <[email protected]> Co-authored-by: David Bariod <[email protected]> Co-authored-by: georlav <[email protected]> Co-authored-by: Sébastien Lavoie <[email protected]> Co-authored-by: Adam Renberg Tamm <[email protected]> Co-authored-by: Adam Renberg Tamm <[email protected]> Co-authored-by: Kirill Motkov <[email protected]> Co-authored-by: tbunyk <[email protected]> Co-authored-by: Emil Hessman <[email protected]> Co-authored-by: Andrey Tcherepanov <[email protected]> Co-authored-by: Jessica Paczuski <[email protected]> Co-authored-by: Haoran Xu <[email protected]> Co-authored-by: Clément Chigot <[email protected]> Co-authored-by: A. F <[email protected]> Co-authored-by: Nicolas Lepage <[email protected]> Co-authored-by: Lynn Cyrin <[email protected]> Co-authored-by: Lynn Cyrin <[email protected]> Co-authored-by: Christian Muehlhaeuser <[email protected]> Co-authored-by: Simon Eskildsen <[email protected]> Co-authored-by: Edward Muller <[email protected]> Co-authored-by: Jonathan Hall <[email protected]> Co-authored-by: zxc <[email protected]> Co-authored-by: Joel Williams <[email protected]> Co-authored-by: Pantelis Sampaziotis <[email protected]> Co-authored-by: Edward Muller <[email protected]> Co-authored-by: Edward Muller <[email protected]> Co-authored-by: lwsanty <[email protected]> Co-authored-by: David Bariod <[email protected]> Co-authored-by: Taylor Wrobel <[email protected]> Co-authored-by: Dmitri Goutnik <[email protected]> Co-authored-by: Alex Shi <[email protected]> Co-authored-by: nolleh <[email protected]> Co-authored-by: Mikolaj Holysz <[email protected]> Co-authored-by: Mark Phelps <[email protected]> Co-authored-by: Mark Phelps <[email protected]> Co-authored-by: Fabrizio Cirelli <[email protected]> Co-authored-by: Deep Datta <[email protected]> Co-authored-by: David Raleigh <[email protected]> Co-authored-by: Alisdair MacLeod <[email protected]> Co-authored-by: Ariel Simulevski <[email protected]> Co-authored-by: Thomas Lacroix <[email protected]> Co-authored-by: ialidzhikov <[email protected]> Co-authored-by: Tobias Klauser <[email protected]> Co-authored-by: Sohel <[email protected]> Co-authored-by: Ichinose Shogo <[email protected]> Co-authored-by: CreativeCactus <[email protected]> Co-authored-by: David Bariod <[email protected]> Co-authored-by: l-lindsay <[email protected]> Co-authored-by: Alec Benzer <[email protected]> Co-authored-by: Sebastiaan van Stijn <[email protected]> Co-authored-by: Billy Zaelani Malik <[email protected]> Co-authored-by: Qingshan Luo <[email protected]> Co-authored-by: Ruben de Vries <[email protected]> Co-authored-by: heui <[email protected]> Co-authored-by: anajavi <[email protected]> Co-authored-by: Nathan Johnson <[email protected]> Co-authored-by: izhakmo <[email protected]> Co-authored-by: Griffin Abner <[email protected]> Co-authored-by: David Bariod <[email protected]> Co-authored-by: Francois <[email protected]> Co-authored-by: Tommaso Visconti <[email protected]> Co-authored-by: Chris <[email protected]> Co-authored-by: Paul Holzinger <[email protected]>
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.
No description provided.