Skip to content

tools: remove specific case for FreeBSD in cmd/tools/vtest_test.v - #24707

Merged
spytheman merged 1 commit into
vlang:masterfrom
lcheylus:freebsd-vtest
Jun 12, 2025
Merged

spytheman merged 1 commit into
vlang:masterfrom
lcheylus:freebsd-vtest

Conversation

@lcheylus

@lcheylus lcheylus commented Jun 12, 2025 •

Copy link
Copy Markdown
Contributor

A specific case was added in cmd/tools/vtest_test.v for FreeBSD to bypass an issue with tcc => PR #21534

This issue (support of .symver for tcc on FreeBSD) is now fixed (see https://repo.or.cz/tinycc.git/commit/ea75d5cf390c095503544718a6c3099c4e3cdabb) and merged in our tcc binary for FreeBSD/amd64.

Tests OK on FreeBSD/amd64 with tcc, clang and gcc

$ ./v -stats -W test cmd/tools/vtest_test.v                                                                                                                                                                                                                
---- Testing... ----------------------------------------------------------------------------------------------------------------------------   
        V  source  code size:      30017 lines,     138470 tokens,     808795 bytes,   291 types,    13 modules,   133 files                                                                                                                                                      
generated  target  code size:      11490 lines,     405356 bytes                                                                                                                                                                                                                  
compilation took: 2867.759 ms, compilation speed: 10467 vlines/s, cgen threads: 3                                                                                                                                                                                                 
running tests in: /home/fox/dev/vlang.git/cmd/tools/vtest_test.v                                                                                                                                                                                                                  
      OK    [1/7]    23.943 ms     4 asserts | main.testsuite_begin()                                                                                                                                                                                                             
      OK    [2/7]  3796.201 ms     1 assert  | main.test_vtest_executable_compiles()                                                                                                                                                                                              
      OK    [3/7]  3184.043 ms     5 asserts | main.test_with_several_test_files()                                                                                                                                                                                                
      OK    [4/7]  3405.343 ms     4 asserts | main.test_with_stats_and_several_test_files()
      OK    [5/7]  3102.359 ms     4 asserts | main.test_partial_failure()
      OK    [6/7]  3517.202 ms     4 asserts | main.test_with_stats_and_partial_failure()
      OK    [7/7]     2.175 ms    NO asserts | main.testsuite_end()
     Summary for running V tests in "vtest_test.v": 22 passed, 22 total. Elapsed time: 17032 ms.

 OK   19991.265 ms cmd/tools/vtest_test.v
--------------------------------------------------------------------------------------------------------------------------------------------
Summary for all V _test.v files: 1 passed, 1 total. Elapsed time: 20006 ms, on 1 job. Comptime: 0 ms. Runtime: 19991 ms.

Signed-off-by: Laurent Cheylus <foxy@free.fr>
@huly-for-github

Copy link
Copy Markdown

Connected to Hulyยฎ: V_0.6-23062

@lcheylus lcheylus changed the title tools: remove specific case for FreeBSD in cmd/tools/vtest_test.v tools: remove specific case for FreeBSD in cmd/tools/vtest_test.v Jun 12, 2025
@spytheman

Copy link
Copy Markdown
Contributor

Excellent.
Thank you ๐Ÿ™‡๐Ÿป .

@spytheman
spytheman merged commit b77887e into vlang:master Jun 12, 2025
@lcheylus
lcheylus deleted the freebsd-vtest branch June 12, 2025 19:28
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.

2 participants