Storage node installation - typo

[Wayne Workman]

I guess this was left over from debugging? not sure. I’m running r6357 on Fedora 23, this was a storage node update.

[Wayne Workman]

@Sebastian-Roth I deleted my fog repo and then pulled it down fresh. The THERE! disappeared…

Guess I messed with some code previously and forgot about it. idk, I mess with a lot of stuff lol.

[Sebastian Roth]

Interesting stuff… In the whole source I cannot find “THERE!” - well not in the scripts anyway:

fogproject$ find . -type f -exec grep "THERE" {} /dev/null \;
./license.txt:  THERE IS NO WARRANTY FOR THE PROGRAM, TO THE EXTENT PERMITTED BY
./packages/web/management/languages/en_US.UTF-8/gpl-3.0.txt:  THERE IS NO WARRANTY FOR THE PROGRAM, TO THE EXTENT PERMITTED BY
./packages/web/management/other/gpl-3.0.txt:  THERE IS NO WARRANTY FOR THE PROGRAM, TO THE EXTENT PERMITTED BY

To me it seams like your code is somehow modified. Can you please check fogproject/lib/common/functions.sh on line 604f! Usually the output is going to the logs and should not be printed on the screen as we see in your picture… Maybe delete the repo and check out again?

[Wayne Workman]

The error about the remi repo seems better, but the THERE! is still there.

[Tom Elliott]

@Sebastian-Roth CentOS/RHEL get remi,remi-php56, DNF variants get remi only.

[Sebastian Roth]

@Tom-Elliott said:

I’ve Adjusted the dnf call checker to only pull in repo’s for remi, rather than remi,remi-php56.

So what happens on CentOS/RHEL then? Is only PHP 5.4 installed?

[Tom Elliott]

I’ve Adjusted the dnf call checker to only pull in repo’s for remi, rather than remi,remi-php56.

[Tom Elliott]

@Wayne-Workman There is NO mention of “THERE!” on the updating packages as needed, or anywhere in the code base.

[Sebastian Roth]

Hell! Those external repos are not easy to handle!! Look at this:

http://rpms.famillecollet.com/enterprise/7/ -> remi (PHP 5.4), remi-php55 (only PHP 5.5), remi-php56 (only PHP 5.6), remi-php70 (only PHP 7.0)

BUT

http://rpms.famillecollet.com/fedora/23/ -> remi (PHP 5.6), remi-php70 (only PHP 7.0)

Have fun scripting this to get everything right for CentOS/Fedora. At least it seams consistent for CentOS 5,6,7 / Fedora 21,22,23