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

Fix debug logs showing up regardless of verbosity on macOS 12.7.1 / 13.6.3 / 14.2 #50

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

kit-ty-kate
Copy link
Contributor

Fixes #48

This seems to be a bug in macOS 12.7.1 / 13.6.3 / 14.2, this has been reported to Apple.
This was tested successfully on Linux and macOS, i haven’t tried BSDs or Windows yet.

@kit-ty-kate
Copy link
Contributor Author

Actually this seems to also happen on FreeBSD 13.1, i somehow never realized... This is embarassing.

@hannesm do you know if the debug logs were always there? They look like this:

$ opam install ...
[...]
Constructing initial basis...
Number of 0-1 knapsack inequalities = 2382
Constructing conflict graph...
Conflict graph has 774 + 104 = 878 vertices
[...]

@hannesm
Copy link

hannesm commented Dec 19, 2023

Dear @kit-ty-kate, yes I've seen this output since a long time. I don't remember whether it was always there or when it appeared. I don't find it too embarassing tbh, my brain is already used to not see it ;) [I wasn't aware this is only a thing on macOS and FreeBSD]

@kit-ty-kate
Copy link
Contributor Author

Successfully tested on Linux, macOS, FreeBSD, OpenBSD, NetBSD, Windows/Cygwin, Windows/MinGW, Windows/MSVC (the last 3 using ocaml/opam#5769).

  • no debug log is shown by default
  • debug log is shown with --debug-level=2 or above

@kit-ty-kate kit-ty-kate merged commit 04cecd0 into ocaml-opam:master Dec 20, 2023
@kit-ty-kate kit-ty-kate deleted the fix-macos-verbosity branch December 20, 2023 17:26
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.

Random verbose output
2 participants