Bug 216593
| Summary: | Update sysutils/graylog to 2.2.1 | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Thomas Bartelmess <thomas> | ||||||||||
| Component: | Individual Port(s) | Assignee: | Palle Girgensohn <girgen> | ||||||||||
| Status: | Closed FIXED | ||||||||||||
| Severity: | Affects Some People | CC: | girgen, peter, thomas, vvelox | ||||||||||
| Priority: | --- | Flags: | thomas:
maintainer-feedback+
thomas: maintainer-feedback+ |
||||||||||
| Version: | Latest | ||||||||||||
| Hardware: | Any | ||||||||||||
| OS: | Any | ||||||||||||
| See Also: | https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217534 | ||||||||||||
| Attachments: |
|
||||||||||||
Portlint output: WARN: Makefile: possible use of absolute pathname "/var/db/graylog". 0 fatal errors and 1 warning found. Looks like this needs updated again since 2.2.0 just game out. Also in regard to the 2.1.* branch, there is also 2.1.3 now. h Created attachment 180078 [details]
Patch to update to 2.2.0
Hi, I guess www/graylog-web-interface should be removed as well, since it is all bundled in one package nowadays? Palle (In reply to Palle Girgensohn from comment #5) Yes the web interface is now bundled with the graylog server Created attachment 180435 [details]
Suggested patch, adding files missing in the 2.2.0 and updating to 2.2.1
Hi!
In the second patch, for 2.2.0, you accidentally misswed adding the new files in the files/ directory.
I bumped version to 2.2.1 and add the files. Also did a
svn mv files/graylog.conf.example files/server.conf.sample.in
since the server.conf was quite similar to the graylog.conf.
Check out the attached patch, combining your 2.2.0 patch with the missing files/ from the 2.1.x patch, and also updated to 2.2.1. Haven't tried it with poudriere yet, will do soon.
Is it complete?
OK to commit?
Palle
Also, a quick question. What is the point of having config in /usr/local/etc/graylog/server/server.conf wouldn't /usr/local/etc/graylog/server.conf be sufficient? Is it the default from graylog? (In reply to Palle Girgensohn from comment #8) I agree that the path is a bit verbose, I've copied what graylog does on Linux: http://docs.graylog.org/en/2.2/pages/installation/manual_setup.html (In reply to Thomas Bartelmess from comment #9) OK. In that case, let's stick with that. :) Created attachment 180476 [details]
Patch graylog to 2.2.1
Hello again Thomas,
As I tested the port with poudriere, I stumbled on a thing or two. I ended up making a bucnh of small suggestions to the port, just to make it somewhat more lightweigth.
I suggest installing graylog in /usr/local/share/graylog, it appeals me more than installing directly under /usr/local and is more according to what hier(7) specifies. and DATADIR is already set in the ports infrastructure, so it does slim the Makefile a bit.
Check out the patch. Ia it alright for you if I commit it?
Palle
(In reply to Palle Girgensohn from comment #11) Hi Palle, the changes look good to me. Thanks, - Thomas A commit references this bug: Author: girgen Date: Fri Mar 3 23:58:02 UTC 2017 New revision: 435375 URL: https://svnweb.freebsd.org/changeset/ports/435375 Log: Update graylog to latest version, 2.2.1 PR: 216593 Changes: head/sysutils/graylog/Makefile head/sysutils/graylog/distinfo head/sysutils/graylog/files/graylog.conf.example head/sysutils/graylog/files/graylog.in head/sysutils/graylog/files/pkg-message.in head/sysutils/graylog/files/server.conf.sample.in head/sysutils/graylog/pkg-plist head/www/graylog-web-interface/ Committed. Thanks! (In reply to Palle Girgensohn from comment #14) Hi Palle, thanks for commiting this. However today Graylog 2.2.2 was commited. The port still works fine. I've created a follow up bug (https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217534) to update the port. Nothing changed except for the version number and the checksums. Thanks, - Thomas A commit references this bug: Author: girgen Date: Sat Mar 4 15:28:45 UTC 2017 New revision: 435412 URL: https://svnweb.freebsd.org/changeset/ports/435412 Log: Update graylog to 2.2.2 PR: 216593 Changes: head/sysutils/graylog/Makefile head/sysutils/graylog/distinfo head/sysutils/graylog/pkg-plist |
Created attachment 179417 [details] Patch Updated to graylog 2.1.2