Created attachment 191557 [details]
I have a pending patch for src/ that removes deprecated signal code values (first deprecated in 1999). FreeBSD adopted standard siginfo codes consistent with POSIX starting in 7.0 release. The src change can be found here:
During an exa-run, smlnj failed to build as it is still using the older constants. The attached patch changes it to use the modern constants instead. Both old and new constants have the same value, so this should not be a runtime change.
(In reply to John Baldwin from comment #0)
> I have a pending patch for src/ that removes deprecated signal code
> values (first deprecated in 1999). [...]
> The src change can be found here:
Thank you for noticing and fixing this issue with lang/smlnj.
> FreeBSD adopted standard siginfo codes consistent with POSIX starting
> in 7.0 release. [...]
> The attached patch changes it to use the modern
> constants instead. Both old and new constants have the same value, so
> this should not be a runtime change.
Then I can't see a reason, why this port should block the cleanup of
src any further. (The FPE_*_TRAP macros should be removed upstream as
well. Hopefully, I'll remember to report this after an update of the
port to 110.82. There's a patch for this update , but it's not tested
on i386, hence not yet submitted:-(
> During an exa-run, smlnj failed to build as it is still using the
> older constants. [...]
I assume, that your sigfpe.patch makes the port compile on -CURRENT,
right? I briefly checked it on 11.1-STABLE amd64, and it continues to
compile and work, as expected.
So I suggest that you commit your patch to lang/smlnj and unblock
226579. If there's anything else I should do, please let me know.
Thanks, you are correct that this is only needed for current once my src/ patch is committed. I just wanted your review / approval before committing to your port. If you could upstream the patch that is even better, thanks!
A commit references this bug:
Date: Mon Mar 19 17:33:04 UTC 2018
New revision: 465024
Use current signal code constants for SIGFPE rather than deprecated ones.
Reviewed by: firstname.lastname@example.org (maintainer)
Approved by: brooks