search

cnpem / epics-in-docker

Container images with EPICS and modules
5 stars 2 forks source link

base: disable bash tracing from install scripts. #45

Closed henriquesimoes closed 10 months ago

henriquesimoes commented 11 months ago

This level of detail pollutes the output during the build, while often not being used. In addition, it can be easily turned back on in a local setup for debugging purposes.

It should not influence any output seen by epics-in-docker users, so no entry is specified in the changelog.

ericonr commented 11 months ago

I don't know if I agree with this. Most of the time I'm not looking at the build output anyway, and when there are issues, I don't need to run anything again, the trace output is there already. Has it bothered you in some situation?

henriquesimoes commented 10 months ago

That's something I've added during the development of the installation scripts back when it was a sketch for containerizing the RFFE IOC.

I think I misunderstood from one of our personal discussion that it wouldn't be needed, similarly to logic of omitting the verbose output from tar.

Has it bothered you in some situation?

Not really. We may keep them. ;)

ericonr commented 10 months ago

I think I misunderstood from one of our personal discussion that it wouldn't be needed, similarly to logic of omitting the verbose output from tar.

I might have changed my mind on it in the meanwhile, to be quite honest :thinking: