Created attachment 230815 [details] patch Patch to update net-mgmt/librenms to 21.12.0 version.
I am now aiming for 22.1.0
Created attachment 231407 [details] Proposed patch for update to 22.1.0
Comment on attachment 231407 [details] Proposed patch for update to 22.1.0 How did you find out that the EXTRACT_DEPENDS was required? Why run composer?
(In reply to Dan Langille from comment #1) > I am now aiming for 22.1.0 By that, I mean I've been working on that version at $WORK. There are a number of directory permission issues where librenms wants to write to WWWDIR. Also, PHP8 and MySQL80....
(In reply to Dan Langille from comment #3) According to docs[1] running composer is needed to populate vendor/ directory. May be this can be done at another stage, not extract. Or may be I misunderstood your question. [1] https://docs.librenms.org/Installation/Install-LibreNMS/
(In reply to Oleksandr Kryvulia from comment #5) The install docs assume many things, including that you are not installing from a package. I create the vendor directory manually as part of each update so that you don't have to. For example [besser dan ~] % pkg info -l librenms | grep /vendor/ | head /usr/local/www/librenms/vendor/amenadiel/jpgraph/.codeclimate.yml /usr/local/www/librenms/vendor/amenadiel/jpgraph/.scrutinizer.yml /usr/local/www/librenms/vendor/amenadiel/jpgraph/Examples/antispamex01.php /usr/local/www/librenms/vendor/amenadiel/jpgraph/Examples/blueblack400x300grad.png /usr/local/www/librenms/vendor/amenadiel/jpgraph/Examples/builtinplotmarksex1.php /usr/local/www/librenms/vendor/amenadiel/jpgraph/Examples/checkgd.php /usr/local/www/librenms/vendor/amenadiel/jpgraph/Examples/checkgd2.php /usr/local/www/librenms/vendor/amenadiel/jpgraph/Examples/checkttf.php /usr/local/www/librenms/vendor/amenadiel/jpgraph/Examples/classroom.jpg /usr/local/www/librenms/vendor/amenadiel/jpgraph/Examples/colormaps.php [besser dan ~] % My recipe is here: https://gist.github.com/dlangille/e91b77ca4a32dfcbcdd7515ffb263e74
(In reply to Dan Langille from comment #3) Also, how did you find out that the EXTRACT_DEPENDS was required?
(In reply to Dan Langille from comment #7) In my case I run composer after librenms archive is extracted and empty vendor directory is present in ${WRKSRC}. So installed composer is required before this stage.
There is no need to do that.
(In reply to Dan Langille from comment #9) May be you are right. Running ./scripts/composer_wrapper.php without system composer works too.
I have 22.5.0 nearly ready.
(In reply to Dan Langille from comment #11) That should be 22.2.1
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=467e569d02e854b4e3abd762b796da449b57ff72 commit 467e569d02e854b4e3abd762b796da449b57ff72 Author: Dan Langille <dvl@FreeBSD.org> AuthorDate: 2022-02-18 15:19:22 +0000 Commit: Dan Langille <dvl@FreeBSD.org> CommitDate: 2022-02-18 15:21:02 +0000 net-mgmt/librenms: Update to 22.2.1 re: https://github.com/librenms/librenms/releases/tag/22.2.1 This port update skips over several releases. See: https://github.com/librenms/librenms/releases See UPDATING for details on storage directory which needs to move to /var/db/librenms LiberNMS wants to write to the %%WWWDIR%%/storage directory. To comply with man 7 hier, this entry is now a symlink to /var/db/librenms/storage Please adjust your filesystem accordingly. The contrib directory has been removed from the upstream tarball. As such, the port now installs the log rotation file to /${PREFIX}/etc/newsyslog.conf.d/librenms.conf.sample PR: 260085 261031 UPDATING | 8 ++ net-mgmt/librenms/Makefile | 45 +++++---- net-mgmt/librenms/distinfo | 10 +- net-mgmt/librenms/files/patch-config.php.default | 12 +-- net-mgmt/librenms/files/patch-poller-wrapper.py | 17 ++-- net-mgmt/librenms/files/pkg-message.in | 116 +++++++++++++---------- 6 files changed, 122 insertions(+), 86 deletions(-)
Oleksandr: if I didn't get everything you wanted into the port, please let me know. There was a bunch of changes associated with this update and I fear I may have overlooked something. Thank you.