Bug 264221 - audio/fluidsynth update to 2.2.7
Summary: audio/fluidsynth update to 2.2.7
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-multimedia (Nobody)
URL: https://github.com/FluidSynth/fluidsy...
Keywords:
Depends on:
Blocks:
 
Reported: 2022-05-25 05:00 UTC by Alastair Hogge
Modified: 2022-06-05 11:40 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (multimedia)

Attachments
fluidsynth-2.2.7.patch (1.98 KB, patch)
2022-05-25 05:00 UTC, Alastair Hogge 		no flags Details | Diff
fluidsynth-2.2.7.patch (1.92 KB, patch)
2022-05-25 05:07 UTC, Alastair Hogge 		no flags Details | Diff
fluidsynth-2.2.7.patch (2.25 KB, patch)
2022-05-27 04:01 UTC, Alastair Hogge 		no flags Details | Diff
fluidsynth-2.2.7.patch (2.00 KB, patch)
2022-06-01 21:30 UTC, Alastair Hogge 		no flags Details | Diff
Show Obsolete (3) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alastair Hogge 2022-05-25 05:00:57 UTC 
Created attachment 234188 [details]
fluidsynth-2.2.7.patch
Comment 1 Alastair Hogge 2022-05-25 05:07:50 UTC 
Created attachment 234189 [details]
fluidsynth-2.2.7.patch

Correct post-install-DOCS-on and pkg-plist
Comment 2 Alastair Hogge 2022-05-27 04:01:51 UTC 
Created attachment 234249 [details]
fluidsynth-2.2.7.patch

Update WWW comment in pkg-descr to use https:// over http://
Comment 3 Max Brazhnikov freebsd_committer freebsd_triage 2022-06-01 12:41:18 UTC 
Why do you install man page in newly added post-install target? Besides, man pages should be installed regardless DOCS option.
Comment 4 Alastair Hogge 2022-06-01 21:28:14 UTC 
(In reply to Max Brazhnikov from comment #3)
Ahh no, I have mistakenly believed that man pages should be cover by DOCS.
Comment 5 Alastair Hogge 2022-06-01 21:30:34 UTC 
Created attachment 234380 [details]
fluidsynth-2.2.7.patch

Remove DOCS coverage for man pages. Manual pages are install regardless (https://docs.freebsd.org/en/books/porters-handbook/book/), only additional documentation is covered by DOCS.
Comment 6 commit-hook freebsd_committer freebsd_triage 2022-06-04 22:20:23 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=7cb8d122bdf6fd47720b5edb61e46e2ddcd89b9c

commit 7cb8d122bdf6fd47720b5edb61e46e2ddcd89b9c
Author:     Max Brazhnikov <makc@FreeBSD.org>
AuthorDate: 2022-06-04 22:19:11 +0000
Commit:     Max Brazhnikov <makc@FreeBSD.org>
CommitDate: 2022-06-04 22:19:11 +0000

    audio/fluidsynth: update to 2.2.7

    PR:             264221
    Submitted by:   Alastair Hogge

 audio/fluidsynth/Makefile  | 2 +-
 audio/fluidsynth/distinfo  | 6 +++---
 audio/fluidsynth/pkg-descr | 2 +-
 audio/fluidsynth/pkg-plist | 2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)
Comment 7 Max Brazhnikov freebsd_committer freebsd_triage 2022-06-05 09:56:59 UTC 
Committed, thanks!
Comment 8 Alastair Hogge 2022-06-05 11:40:38 UTC 
(In reply to Max Brazhnikov from comment #7)
Thanking you very muchly, Max.