-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
Testsuite fixes #45
base: main
Are you sure you want to change the base?
Testsuite fixes #45
Conversation
de47256
to
69c6e83
Compare
we'll worry about the problems this solves later, as I don't think working on the compile tests is a good use of time. |
The other tests are fine, but let's not worry about the 001compile.t tests, those aren't meaningful with the current way code is loaded. a compile test on bugzilla.pl itself (and only it) could be valid. |
69c6e83
to
775d422
Compare
@dylanwh OK, this PR is again ready. I dropped the I see that ab5ce39 included some of the fixes in this PR (whether cherry-picked or recreated), so I dropped them from here too. This leaves |
"Possible precedence issue with control flow operator"
23e3bd3
to
e069bea
Compare
d0d6e53
to
12b5136
Compare
These fix all failures in
t/
for me, except fort/901-secure-mail-loop.t
.