Bug 127223

Summary: [patch] audio/last.fm - unbreak build on amd64
Product: Ports & Packages Reporter: Andrius Morkunas <hinokind>
Component: Individual Port(s)Assignee: Martin Wilke <miwi>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Description Flags
file.diff none

Description Andrius Morkunas 2008-09-08 23:00:09 UTC
Recent audio/last.fm update to version has made the port unbuildable on amd64.

build fails while compiling mp3transcode.cpp:
mp3transcode.cpp:284: error: no matching function for call to 'qMin(int&, ssize_t)'

For some (invalid) reason someone decided to cast int to ssize_t. Because on amd64 (and probably some other platforms) ssize_t is not the same as int, compiler lets you enjoy failed builds.

Fix: In the same source file function qMin(int, int) is called four times so its safe to assume one more qMin(int, int) instead of qMin(int, ssize_t(int)) wont do anything bad. Other than that I found no problems with the source code.

I'm not sure about the USE_GCC=4.2+ in makefile but it was there before, so I put it back.

Patch attached with submission follows:
How-To-Repeat: Try building audio/last.fm on amd64.
Comment 1 Martin Wilke freebsd_committer 2008-09-08 23:38:29 UTC
Responsible Changed
From-To: freebsd-ports-bugs->miwi

I'll take it.
Comment 2 Martin Wilke freebsd_committer 2008-09-12 09:49:07 UTC
State Changed
From-To: open->closed

Committed. Thanks!
Comment 3 dfilter service freebsd_committer 2008-09-12 09:49:12 UTC
miwi        2008-09-12 08:48:58 UTC

  FreeBSD ports repository

  Modified files:
    audio/last.fm        Makefile 
  Added files:
    audio/last.fm/files  patch-src_mp3transcode_mp3transcode.cpp 
  - Fix build on amd64
  PR:             127223
  Submitted by:   "Andrius Mork&#363;nas" <hinokind@gmail.com>
  Approved by:    portmgr (erwin)
  Revision  Changes    Path
  1.19      +1 -1      ports/audio/last.fm/Makefile
  1.1       +11 -0     ports/audio/last.fm/files/patch-src_mp3transcode_mp3transcode.cpp (new)
cvs-all@freebsd.org mailing list
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"