nix icon indicating copy to clipboard operation
nix copied to clipboard

stat fix unit tests for openbsd.

Open devnexen opened this issue 4 years ago • 10 comments
trafficstars

devnexen avatar Oct 16 '21 17:10 devnexen

What was the problems? Did these tests fail to compile, or fail at runtime?

asomers avatar Oct 16 '21 21:10 asomers

runtime failure (openbsd 7).

devnexen avatar Oct 16 '21 21:10 devnexen

Well, the commit message is inaccurate then, because it doesn't "fix" the tests, it disables them. Are these functions not supported on OpenBSD? If so, we should remove the bindings for them. Or if it's just an issue with certain OpenBSD versions or something with the runtime config, then it would be appropriate to ignore the tests. That way they'll still be compiled, but won't be run by default.

asomers avatar Oct 16 '21 21:10 asomers

The commit message is still pretty mysterious. What's the real problem?

asomers avatar Oct 16 '21 21:10 asomers

getting EPERM errnos for test_fstatat/mknod/mknodat, assertions failures test_stat_and_fstat/test_stat_and_lstat for example

devnexen avatar Oct 16 '21 22:10 devnexen

Have you determined the root cause? Is it a bug in OpenBSD? If you simply ignore the failures without investigating, that might mask a bug in Nix. We wouldn't want that.

asomers avatar Oct 16 '21 22:10 asomers

it seems version related if I run an older version of openbsd the tests pass.

devnexen avatar Oct 16 '21 22:10 devnexen

So is that a bug in OpenBSD then? It sounds worthy of investigation. Disabling tests just because they don't pass is a risky proposition.

asomers avatar Oct 17 '21 00:10 asomers

@devnexen did you ever figure out the root cause of your failures?

asomers avatar Dec 15 '21 03:12 asomers

well I lost the VM in the meantime, will look after when I can.

devnexen avatar Dec 15 '21 04:12 devnexen

I'm closing this for lack of activity. But if you ever do root-cause the issue, you can reopen it.

asomers avatar Nov 29 '22 06:11 asomers