1575821 - pr/tests/attach.c:170:2: error: unterminated conditional directive

Status: RESOLVED FIXED

(NSPR :: NSPR, defect)

Description

[Jan Beich]

$ cc --version
FreeBSD clang version 8.0.1 (tags/RELEASE_801/final 366581) (based on LLVM 8.0.1)
Target: x86_64-unknown-freebsd13.0
Thread model: posix
InstalledDir: /usr/bin

$ ./configure; gmake -sj8; gmake -sj8 -C lib/tests; gmake -sj8 -C pr/tests
attach.c:170:2: error: unterminated conditional directive
#ifdef _PR_PTHREADS
 ^
attach.c:275:2: error: expected '}'
}
 ^
attach.c:109:1: note: to match this '{'
{
^
attach.c:176:8: error: use of undeclared label 'exit_now'
                goto exit_now;
                     ^
nonblock.c:44:57: error: use of undeclared identifier 'UNIT_TIME'
    PRIntervalTime unitTime = PR_MillisecondsToInterval(UNIT_TIME);
                                                        ^
nonblock.c:117:57: error: use of undeclared identifier 'UNIT_TIME'
    PRIntervalTime unitTime = PR_MillisecondsToInterval(UNIT_TIME);
                                                        ^
stack.c:56:22: error: expected end of line in preprocessor expression
#if defined(__WINS__))
                     ^
testfile.c:78:2: error: #endif without #if
#endif
 ^

Comment 1

[BugBot [:suhaib / :marco]]

:jbeich, since this bug is a regression, could you fill (if possible) the regressed_by field?
For more information, please visit auto_nag documentation.

Comment 2

[Jan Beich]

Sorry, the new modal UI is confusing. I can't easily find bug dependencies.

Comment 3

[Ludovic Hirlimann [:Usul]]

Attachment: 1575821.patch

Looks like I cleaned up and #else #endif too much in https://phabricator.services.mozilla.com/D24708#change-9ZG7lYM4ZRsr

Does it work with the attached patch ?

Comment 4

[Jan Beich]

(In reply to Ludovic Hirlimann [:Usul] from comment #3)

Does it work with the attached patch ?

No, there're more such typos. comment 0 was edited to document all errors.

Comment 5

[Ludovic Hirlimann [:Usul]]

Attachment: 1575821_2.patch

I think This fixes everything you are seeing - I'm sorry I don't have a freebsd box for testing, can you let me know if this fixes everything?

Comment 6

[Jan Beich]

(In reply to Ludovic Hirlimann [:Usul] from comment #5)

I think This fixes everything you are seeing

Looks same as v1.

I'm sorry I don't have a freebsd box for testing, can you let me know if this fixes everything?

Can you try to build tests on Linux or macOS? There's nothing FreeBSD-specific about these errors.

Comment 7

[Ludovic Hirlimann [:Usul]]

Attachment: 1575821_2.patch

of course, I should be diffing the dir, not a file :(

Comment 8

[Jan Beich]

Comment on attachment 9087391 [details] [diff] [review]
1575821_2.patch

Still missing a typo mentioned in comment 0:
```c
testfile.c:78:2: error: #endif without #if
#endif
 ^
```

Comment 9

[Jan Beich]

Attachment: v1

Here's what I had in mind (except for #else in attach.c) before you've assigned the bug.

Kai, what value tests provide given they're not routinely built/run on Tier1 platforms?

Comment 10

[Landry Breuil (:gaston)]

Comment on attachment 9087671 [details] [diff] [review]
v1

patch allows me to build & run tests on OpenBSD/amd64 w/ 4.22.

Comment 11

[Kai Engert [:KaiE:]]

(In reply to Jan Beich from comment #9)

Kai, what value tests provide given they're not routinely built/run on Tier1 platforms?

Very little... I had started to get them enabled in bug 1385061 and bug 1385039 but there's just always too much work waiting to be done.

Comment 12

[Kai Engert [:KaiE:]]

Comment on attachment 9087671 [details] [diff] [review]
v1

allows me to build on Linux, too.

Comment 13

[Kai Engert [:KaiE:]]

https://hg.mozilla.org/projects/nspr/rev/71b3c15c753cbdacb24527400fd633da885fab55