Review Request: Make Amarok compile and link with TagLib installed in $HOME

[Ralf Engels]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104693/#review14179
-----------------------------------------------------------



src/CMakeLists.txt
<http://git.reviewboard.kde.org/r/104693/#comment11212>

    There is a block linking the taglib just a few lines previously.
    Around 922
    
    Why do you need that here?


- Ralf Engels


On April 22, 2012, 5:25 p.m., Mathias Stephan Panzenböck wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104693/
> -----------------------------------------------------------
> 
> (Updated April 22, 2012, 5:25 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> I installed TagLib 1.8-GIT in $HOME so I can try the new features (see also my other review request #101598). $HOME/bin is in $PATH and thus "taglib-config" returns all the right settings. Still Amarok did not compile. The reason is that it adds the taglib include directory to the list of include directories, but then still accesses the taglib headers with "#include <taglib/tstring.h>" in some places (instead of using "#include <tstring.h>"). This is clearly wrong, but coincidentally works if taglib is installed in /usr. I fixed that.
> 
> For some reason which I can't explain I had to also add the taglib libs to the amarok binary. I would have thought that adding it to amaroklib would be enough, but then the linker complained.
> 
> Maybe the patch isn't good like it is (the linking part), but something needs to be done about this.
> 
> 
> Diffs
> -----
> 
>   shared/tag_helpers/APETagHelper.h c9f23c7 
>   shared/tag_helpers/ASFTagHelper.h 41b58b2 
>   shared/tag_helpers/ID3v2TagHelper.h 1bbae70 
>   shared/tag_helpers/ID3v2TagHelper.cpp 27e0cf0 
>   shared/tag_helpers/MP4TagHelper.h 9a6beee 
>   shared/tag_helpers/StringHelper.h 3f6b9b8 
>   shared/tag_helpers/VorbisCommentTagHelper.h 5c5b1bf 
>   shared/tag_helpers/VorbisCommentTagHelper.cpp 1fbb437 
>   src/CMakeLists.txt 43bda90 
> 
> Diff: http://git.reviewboard.kde.org/r/104693/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mathias Stephan Panzenböck
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20120526/17de2097/attachment.html>