Skip to content

Instantly share code, notes, and snippets.

@DomT4
Last active August 29, 2015 14:08
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save DomT4/6a73cd8c6f8678d79479 to your computer and use it in GitHub Desktop.
Save DomT4/6a73cd8c6f8678d79479 to your computer and use it in GitHub Desktop.
Alpine-Chappa-Yosemite.
Sender: chappa@gmx.com
Dear Dominyk,
Thank you for your messages and your patch. I was not aware of this issue. The problem occurs because two teams of developers agree on a common name for a function that has two different uses. On the one hand, the developers of MACH use a function called "panic", and so did the alpine developers.
I imagine that the panic function for MACH was used to indicate problems with the micro kernel, while the panic function in Alpine was used to report bugs or stop the program altogether. The panic function is not a library function in Alpine, it is a function that a developer using its libraries must write (so Alpine and Web Alpine have their own versions of it) so in order to reduce confusion in the future, what I will do is to rename the panic function to "alpine_panic" so that no program will have the confusion as to what "panic" means in terms of library calls, should anyone ever need to link with a library that calls panic something else.
This is the solution that will be implemented in the upcoming release of Alpine.
Thank you Dominyk, I really appreciate your report and fix!
--
Eduardo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment