Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nagios formula Caveats is outdated #120521

Closed
1 of 3 tasks
glensc opened this issue Jan 13, 2023 · 4 comments · Fixed by #125987
Closed
1 of 3 tasks

Nagios formula Caveats is outdated #120521

glensc opened this issue Jan 13, 2023 · 4 comments · Fixed by #125987
Labels
bug Reproducible Homebrew/homebrew-core bug outdated PR was locked due to age

Comments

@glensc
Copy link
Contributor

glensc commented Jan 13, 2023

brew gist-logs <formula> link OR brew config AND brew doctor output

brew config
✖  brew config
HOMEBREW_VERSION: 3.6.18-50-gebd0d61
ORIGIN: https://github.com/Homebrew/brew
HEAD: ebd0d61dfac6a75b9bab5b5db593709fcc80bf50
Last commit: 2 hours ago
Branch: master
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: bb8047a824bf9c69d160a13f8c636cbc9ec22fb5
Core tap last commit: 46 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_DISPLAY: /private/tmp/com.apple.launchd.QGSNyVtJDe/org.xquartz:0
HOMEBREW_GITHUB_API_TOKEN: set
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.10 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit arm_firestorm_icestorm
Clang: 14.0.0 build 1400
Git: 2.39.0 => /opt/homebrew/bin/git
Curl: 7.85.0 => /usr/bin/curl
macOS: 13.1-arm64
CLT: 14.2.0.0.1.1668646533
Xcode: N/A
Rosetta 2: false

Verification

  • My "brew doctor output" says Your system is ready to brew. and am still able to reproduce my issue.
  • I ran brew update and am still able to reproduce my issue.
  • I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?

Not really a problem, but no issue category to fill in outdated info.

What happened (include all command output)?

installed nagios formula and it speaks of php5 :)

What did you expect to happen?

The current php formula is php@8.2, so it should at least give that version info

Step-by-step reproduction instructions (by running brew commands)

$ brew info nagios
...
==> Caveats
First we need to create a command dir using superhuman powers:

  mkdir -p /opt/homebrew/var/lib/nagios/rw
  sudo chgrp _www /opt/homebrew/var/lib/nagios/rw
  sudo chmod 2775 /opt/homebrew/var/lib/nagios/rw

Then install the Nagios web frontend into Apple's built-in Apache:

  1) Turn on Personal Web Sharing.

  2) Load the cgi and php modules by patching /etc/apache2/httpd.conf:

    -#LoadModule php5_module        libexec/apache2/libphp5.so
    +LoadModule php5_module        libexec/apache2/libphp5.so

    -#LoadModule cgi_module libexec/apache2/mod_cgi.so
    +LoadModule cgi_module libexec/apache2/mod_cgi.so

  3) Symlink the sample config and create your web account:

    sudo ln -sf /opt/homebrew/opt/nagios/share/nagios.conf /etc/apache2/other/
    htpasswd -cs /opt/homebrew/etc/nagios/htpasswd.users nagiosadmin
    sudo apachectl restart

Log in with your web account (and don't forget to RTFM :-)

  open http://localhost/nagios


To restart nagios after an upgrade:
  sudo brew services restart nagios
Or, if you don't want/need a background service you can just run:
  /opt/homebrew/opt/nagios/bin/nagios /opt/homebrew/etc/nagios/nagios.cfg
$ brew info php@8.2
...
==> Caveats
To enable PHP in Apache add the following to httpd.conf and restart Apache:
    LoadModule php_module /opt/homebrew/opt/php/lib/httpd/modules/libphp.so

    <FilesMatch \.php$>
        SetHandler application/x-httpd-php
    </FilesMatch>

Finally, check DirectoryIndex includes index.php
    DirectoryIndex index.php index.html

The php.ini and php-fpm.ini file can be found in:
    /opt/homebrew/etc/php/8.2/

To restart php after an upgrade:
  brew services restart php
Or, if you don't want/need a background service you can just run:
  /opt/homebrew/opt/php/sbin/php-fpm --nodaemonize

should the Nagios caveat still speak about "Apple's built-in Apache" or go with brewed Apache+php?

@glensc glensc added the bug Reproducible Homebrew/homebrew-core bug label Jan 13, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale No recent activity label Feb 9, 2023
@glensc
Copy link
Contributor Author

glensc commented Feb 9, 2023

Yep. still there

➜ brew info nagios|grep php5
    -#LoadModule php5_module        libexec/apache2/libphp5.so
    +LoadModule php5_module        libexec/apache2/libphp5.so

@github-actions github-actions bot removed the stale No recent activity label Feb 9, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale No recent activity label Mar 2, 2023
@glensc
Copy link
Contributor Author

glensc commented Mar 5, 2023

Perhaps the easiest is just to drop the caveats if nobody wants to maintain them.

@github-actions github-actions bot removed the stale No recent activity label Mar 5, 2023
ZhongRuoyu added a commit to ZhongRuoyu/homebrew-core that referenced this issue Mar 17, 2023
The caveats were last updated in 2015. They are no longer applicable.

Resolves Homebrew#120521.
carlocab pushed a commit that referenced this issue Mar 17, 2023
The caveats were last updated in 2015. They are no longer applicable.

Resolves #120521.
@github-actions github-actions bot added the outdated PR was locked due to age label Apr 17, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Reproducible Homebrew/homebrew-core bug outdated PR was locked due to age
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant