search

sclorg / container-common-scripts

Apache License 2.0
21 stars 45 forks source link

do not source the OC library if already sourced #309

Closed zmiklank closed 1 year ago

phracek commented 1 year ago

@zmiklank Make sense to me to not include the source file twice. Can you please do it also for the other files?

@hhorak @pkubatrh WDYT bash experts? Does it make sense also from your point of view?

zmiklank commented 1 year ago

Yes, if we proceed with this proposal I will add the check also to other libs.

zmiklank commented 1 year ago

[test-all]

hhorak commented 1 year ago

lgtm

zmiklank commented 1 year ago

[test-all]

zmiklank commented 1 year ago

Fixes: https://github.com/sclorg/postgresql-container/issues/478