Bug 224274 - audio/yoshimi: update to 1.5.5
Summary: audio/yoshimi: update to 1.5.5
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: Tobias Kortkamp
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-12 14:21 UTC by bye
Modified: 2017-12-12 15:20 UTC (History)
1 user (show)

See Also:
linimon: maintainer-feedback+


Attachments
svn diff (2.37 KB, patch)
2017-12-12 14:21 UTC, bye
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description bye 2017-12-12 14:21:57 UTC
Created attachment 188752 [details]
svn diff

This is an svn diff for Yoshimi 1.5.5
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2017-12-12 14:46:45 UTC
Fix Summary.  I would notify maintainer except bugzilla won't let me.
Comment 2 bye 2017-12-12 14:59:27 UTC
(In reply to Mark Linimon from comment #1)

it's a bit complicated at the moment but my current bugzilla email address is owen94012@gmail.com, I am migrating my emails from gmail to private servers.
Comment 3 Mark Linimon freebsd_committer freebsd_triage 2017-12-12 15:03:20 UTC
Submitter is maintainer from another address.
Comment 4 Tobias Kortkamp freebsd_committer freebsd_triage 2017-12-12 15:04:27 UTC
I'm going to take this bug, but I consider it rude to be pinged like
you did on bug #223397 while also sending an email to ports@.  Please do
not do this.
Comment 5 bye 2017-12-12 15:06:24 UTC
(In reply to Tobias Kortkamp from comment #4)
My apologizes Tobias, I wasn't sure what's the proper procedure. Won't happen again.
Comment 6 commit-hook freebsd_committer freebsd_triage 2017-12-12 15:15:43 UTC
A commit references this bug:

Author: tobik
Date: Tue Dec 12 15:15:08 UTC 2017
New revision: 456167
URL: https://svnweb.freebsd.org/changeset/ports/456167

Log:
  audio/yoshimi: Update to 1.5.5

  PR:		224274
  Submitted by:	maintainer

Changes:
  head/audio/yoshimi/Makefile
  head/audio/yoshimi/distinfo
  head/audio/yoshimi/pkg-plist
Comment 7 Tobias Kortkamp freebsd_committer freebsd_triage 2017-12-12 15:15:59 UTC
Committed, thanks!

The patch you attached here didn't delete any patch files.  Not sure
how you generated it.  If you delete files make sure to use `svn rm` instead
of just `rm`, so that svn knows you want to delete it.

But files/patch-src_CMakeLists.txt needs to stay anyway to fix the
build on FreeBSD 10.3 (see the top of the patch as to why) and to
install man pages into the correct location.

If you want to get rid of it in the future consider sending it to
upstream.
Comment 8 bye 2017-12-12 15:20:59 UTC
(In reply to Tobias Kortkamp from comment #7)

Thanks for the tips, I am working with the guys upstream.

I built it with poudriere but I was unsure how to get SVN diff to work in there so I just overwrote the files in a manually created jail then did a svn diff of that.

I'm not sure how to do an svn diff from poudriere after a testport command but that's a different thing.