Bug 205332

Summary: multimedia/ogmrip UN-BREAK (makes fetchable)
Product: Ports & Packages Reporter: Chris Hutchinson <portmaster>
Component: Individual Port(s)Assignee: Kurt Jaeger <pi>
Status: Closed FIXED    
Severity: Affects Some People CC: pi, portmaster
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
svn diff for multimedia/ogmrip
none
QA log for multimedia/ogmrip none

Description Chris Hutchinson 2015-12-15 04:18:11 UTC
Created attachment 164257 [details]
svn diff for multimedia/ogmrip

There was a request on the ports@ mailing list
from Julian, to fix this. Si I did. :)

Apparently upstream relocated, and re-rolled the distfile.
The svn diff, attached, accounts for that, and UNbreaks this port.

Changes limited to:

Makefile: corrects distfile location, bumps PORTREVISION

distinfo

That's it!

Thanks!

--Chris
Comment 1 Chris Hutchinson 2015-12-15 04:18:53 UTC
Created attachment 164258 [details]
QA log for multimedia/ogmrip
Comment 2 Kurt Jaeger freebsd_committer 2015-12-15 04:45:44 UTC
There's also a change between the distfiles, even if the version stays the same:

If you diff the two source trees, you will find that the new version
no longer has provides lavf-related switches to mplayer (because newer
mplayers have different options for lavf).
Comment 3 Kurt Jaeger freebsd_committer 2015-12-15 05:06:41 UTC
Testbuilds are fine.
Comment 4 commit-hook freebsd_committer 2015-12-15 05:08:49 UTC
A commit references this bug:

Author: pi
Date: Tue Dec 15 05:07:52 UTC 2015
New revision: 403761
URL: https://svnweb.freebsd.org/changeset/ports/403761

Log:
  multimedia/ogmrip: unbreak

  - distinfo updated
  - lavf-option to mplayer call was removed

  PR:		205332
  Submitted by:	Chris Hutchinson <portmaster@bsdforge.com>

Changes:
  head/multimedia/ogmrip/Makefile
  head/multimedia/ogmrip/distinfo
Comment 5 Kurt Jaeger freebsd_committer 2015-12-15 05:10:14 UTC
Committed, thanks.
Comment 6 Chris Hutchinson 2015-12-15 07:34:56 UTC
(In reply to Kurt Jaeger from comment #2)
> There's also a change between the distfiles, even if the version stays the
> same:
> 
> If you diff the two source trees, you will find that the new version
> no longer has provides lavf-related switches to mplayer (because newer
> mplayers have different options for lavf).

I wasn't able to test that far, because the version of MEncoder
on my dev box was too old. Sorry for the bother.

Thanks, Kurt!

--Chris