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

tools: fix doctor output on OpenBSD #23578

Merged
merged 1 commit into from
Jan 26, 2025
Merged

Conversation

lcheylus
Copy link
Contributor

@lcheylus lcheylus commented Jan 25, 2025

  • get gcc version via egcc --version
  • don't run ldd to get glibc version

Fix #23576


vdoctor output OK on OpenBSD and Linux

  • OpenBSD/amd64
$ ./v doctor
|V full version      |V 0.4.9 d710d9e.62d618e
|:-------------------|:-------------------
|OS                  |openbsd, 7.6, GENERIC.MP#518
|Processor           |4 cpus, 64bit, little endian
|Memory              |9.3e-10GB/3.98GB
|                    |
|V executable        |/home/fox/dev/vlang.git/v
|V last modified time|2025-01-25 17:44:17
|                    |
|V home dir          |OK, value: /home/fox/dev/vlang.git
|VMODULES            |OK, value: /home/fox/.vmodules
|VTMP                |OK, value: /tmp/v_1000
|Current working dir |OK, value: /home/fox/dev/vlang.git
|                    |
|Git version         |git version 2.48.1
|V git status        |weekly.2025.03-40-ga1c67509
|.git/config present |true
|                    |
|cc version          |OpenBSD clang version 16.0.6
|gcc version         |egcc (GCC) 11.2.0
|clang version       |OpenBSD clang version 16.0.6
|tcc version         |tcc version 0.9.28rc 2024-09-14 HEAD@b8b6a5fd (x86_64 OpenBSD)
|tcc git status      |thirdparty-openbsd-amd64 8205cc59
|emcc version        |N/A
|glibc version       |N/A
  • Linux Debian/testing
$ ./v doctor
|V full version      |V 0.4.9 5b27233.62d618e
|:-------------------|:-------------------
|OS                  |linux, Debian GNU/Linux trixie/sid
|Processor           |8 cpus, 64bit, little endian, Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz
|Memory              |0.55GB/15.5GB
|                    |
|V executable        |/home/fox/dev/vlang.git/v
|V last modified time|2025-01-25 17:49:25
|                    |
|V home dir          |OK, value: /home/fox/dev/vlang.git
|VMODULES            |OK, value: /home/fox/.vmodules
|VTMP                |OK, value: /tmp/v_1000
|Current working dir |OK, value: /home/fox/dev/vlang.git
|                    |
|Git version         |git version 2.45.2
|V git status        |weekly.2025.03-40-ga1c67509
|.git/config present |true
|                    |
|cc version          |cc (Debian 14.2.0-12) 14.2.0
|gcc version         |gcc (Debian 14.2.0-12) 14.2.0
|clang version       |Debian clang version 19.1.6 (1+b1)
|tcc version         |tcc version 0.9.28rc 2024-07-31 HEAD@1cee0908 (x86_64 Linux)
|tcc git status      |thirdparty-linux-amd64 0134e9b9
|emcc version        |N/A
|glibc version       |ldd (Debian GLIBC 2.40-5) 2.40

- get gcc version via 'egcc --version'
- don't run ldd to get glibc version

Fix vlang#23576

Signed-off-by: Laurent Cheylus <foxy@free.fr>
Copy link

Connected to Huly®: V_0.6-22004

Copy link
Member

@spytheman spytheman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work.

@spytheman spytheman merged commit 5bfea9d into vlang:master Jan 26, 2025
26 checks passed
@lcheylus lcheylus deleted the openbsd-doctor branch January 26, 2025 14:30
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.

doctor: irrelevant informations on OpenBSD
2 participants