You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A few weeks ago, I was playing with https://github.com/honeycombio/beeline-go which makes use of your project in some capacity. Today, I was very confused when a ./configure script was hanging. I was even more confused because initial inspection showed the name of the program that it was hanging on to be echo.
Of course:
$ which echo
echo: shell built-in command
But after some head scratching, I realized:
$ /usr/bin/which echo
/my/go/path/bin/echo
It's pretty bad form to name your binary the same as such a common shell builtin. It leads to some pretty stunningly confusing behavior, even for somebody who never directly uses your project.
Please consider renaming it to echod or echo-server or something of that nature to avoid this time bomb exploding in some other unfortunate dev's face in the future.
Cheers,
Brandon
The text was updated successfully, but these errors were encountered:
@brandonbloom Echo is a framework/library and it doesn't release any kind of binary with that name. Whoever is creating a binary should not create it with that name.
Hello,
A few weeks ago, I was playing with https://github.com/honeycombio/beeline-go which makes use of your project in some capacity. Today, I was very confused when a
./configure
script was hanging. I was even more confused because initial inspection showed the name of the program that it was hanging on to beecho
.Of course:
But after some head scratching, I realized:
It's pretty bad form to name your binary the same as such a common shell builtin. It leads to some pretty stunningly confusing behavior, even for somebody who never directly uses your project.
Please consider renaming it to
echod
orecho-server
or something of that nature to avoid this time bomb exploding in some other unfortunate dev's face in the future.Cheers,
Brandon
The text was updated successfully, but these errors were encountered: