-
Notifications
You must be signed in to change notification settings - Fork 196
minor: tighten up logging #217
Comments
These log lines are coming from the script from travis CI we use to install go, see here. |
@aarushik93 So are we supposed to be piping those commands into a bash environment? I'm wondering if instead of removing them in that Travis CI file there's a way to not print them when piped? |
thinking out loud: we could also fork that travis script, change it, and use our own copy, non? |
@fool that's absolutely an option, but I think if there's any way we can avoid having to maintain a fork that's ideal. If everyone else (ie Travis) is simply echoing those commands to their output then maybe we need our own version, it seems odd that those shell commands are showing up in output though. I'd be curious if there was a way we were invoking it that were different than expected use. |
As far as I can see we're not actually piping these commands. |
Also, maybe someone can provide me some background on the use of the script from travis as opposed to just installing go ourselves in the container? Maybe that's a possibility too - I believe that would also remove other unnecessary logs? |
There are a few loglines in the output of every build we could safely skip IMNSHO. Not saying don't run the commands, just outputting them doesn't serve a purpose I can understand (vs say showing what $PATH is or something ;))
The text was updated successfully, but these errors were encountered: