Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 157588 - media-libs/akode-2.0.1 flac 1.1.3 compatibility patch
Summary: media-libs/akode-2.0.1 flac 1.1.3 compatibility patch
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 152007
  Show dependency tree
 
Reported: 2006-12-09 02:50 UTC by Alexis Ballier
Modified: 2007-03-28 10:30 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch from Josh Coalson (akode-2.0.1+flac-1.1.3.patch,17.46 KB, patch)
2006-12-09 02:51 UTC, Alexis Ballier
Details | Diff
Second Patch from Josh Coalson (akode-2.0.1+flac-1.1.3.patch2,934 bytes, patch)
2006-12-09 02:53 UTC, Alexis Ballier
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexis Ballier gentoo-dev 2006-12-09 02:50:06 UTC
Mail from Josh Coalson :
"
> (cc'ing other package maintainers)
> hi Allan, attached is a patch to akode-2.0.1 to make the code
> work with both the upcoming FLAC 1.1.3 release and previous
> versions.  a beta of FLAC 1.1.3 is available here:
>
http://prdownloads.sourceforge.net/flac/flac-1.1.3-beta2.tar.gz?download
> 
> changelog:
>
http://flac.cvs.sourceforge.net/*checkout*/flac/flac/doc/html/changelog.html
> 
> the main change is the merging of libOggFLAC into libFLAC.
> the LEGACY_FLAC stuff can be removed once FLAC-1.1.3 has propagated
> everywhere.  I know it's ugly, the main purpose of the patch is for
> package maintainers to be able to move to FLAC-1.1.3 with as little
> pain as possible.  for trunk code you can remove the LEGACY_FLAC
> stuff completely; after that you'll notice that native FLAC and
> Ogg FLAC can be handled with the same code except for the call to
> init().
> 
> let me know if you have any questions or problems...
> 
> Josh
> 
"


Maybe this would help if you want to carefully review the patch : 
http://flac.sourceforge.net/api/group__porting.html


The guideline I've followed to patch sound/video apps is : 
review and apply the patch
test with both flac 1.1.2 & flac 1.1.3
remove the ~ dep on flac 1.1.2


By doing so we'll be able to unmask flac 1.1.3 as soon as the deptree will be
correct.
Comment 1 Alexis Ballier gentoo-dev 2006-12-09 02:51:08 UTC
Created attachment 103672 [details, diff]
Patch from Josh Coalson

You'll have such warnings : 

>
> You do not seem to have libOggFLAC installed.
> Without it aKode and aRts will not be
> able to play Ogg-files with embedded FLAC streams
> You need atleast libOggFLAC 1.1.1 or newer.
>

That's normal since it has been merged to libFLAC
Comment 2 Alexis Ballier gentoo-dev 2006-12-09 02:53:15 UTC
Created attachment 103673 [details, diff]
Second Patch from Josh Coalson

Quoting Josh: 
"
here is one more tiny patch to apply on top of the previous one,
which will prevent a possible problem with Ogg FLAC decoding at
end of stream.

Josh
"
Comment 3 Alexis Ballier gentoo-dev 2007-03-28 10:30:09 UTC
revbumped in ~arch for this.

That's good I did that myself as those patches were not enough for oggflac.