Skip to content

Fixed error installation on non standard postgres db port#131

Merged
arogge merged 1 commit intobareos:masterfrom
JayDi85:master
Jul 4, 2019
Merged

Fixed error installation on non standard postgres db port#131
arogge merged 1 commit intobareos:masterfrom
JayDi85:master

Conversation

@JayDi85
Copy link
Contributor

@JayDi85 JayDi85 commented Nov 1, 2018

Installation script are missing from PGPORT environment setup. That's why it's can't install bareos on systems with non standard postgres port. User gets error about connection problem and can't continue to install. That's PR will fix that error.

Installation script was missing from `PGPORT` environment setup. That's
why it's can't install bareos on systems with non standard postgres
port. User gets error about connection problem and can't install. That's
PR will fix that error.
@arogge
Copy link
Member

arogge commented Mar 15, 2019

Hi Oleg,
thank you very much for your pull-request.
The change looks good. Did you rebuild the installer to actually try it out?

Best Regards,
Andreas

@JayDi85
Copy link
Contributor Author

JayDi85 commented Mar 15, 2019

No, I don't rebuild it. But I tested workaround to manually set PGPORT environment variable before install -- it's works correct.

@arogge arogge merged commit c98518e into bareos:master Jul 4, 2019
florian-at-bareos added a commit to florian-at-bareos/bareos that referenced this pull request Jan 29, 2025
Fixes bareos#131: Add automated check for configuration issues in systemtest
configurations

One can disable this behaviour for each systemtest individually by
setting the IGNORE_DAEMON_CONFIG_WARNINGS (CACHE INTERNAL) to true at
the beginning of the systemtests CMakeLists.txt file.
BareosBot pushed a commit to florian-at-bareos/bareos that referenced this pull request Feb 12, 2025
Fixes bareos#131: Add automated check for configuration issues in systemtest
configurations

One can disable this behaviour for each systemtest individually by
setting the IGNORE_DAEMON_CONFIG_WARNINGS (CACHE INTERNAL) to true at
the beginning of the systemtests CMakeLists.txt file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants