Review Request 125059: Fix for Amarok's kf5 branch failing to build

Aroonav Mishra aroonav11 at gmail.com
Sat Sep 5 18:29:48 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125059/#review84872
-----------------------------------------------------------

Ship it!


Ship It!

- Aroonav Mishra


On Sept. 5, 2015, 12:47 p.m., Somsubhra Bairi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125059/
> -----------------------------------------------------------
> 
> (Updated Sept. 5, 2015, 12:47 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Repository: amarok
> 
> 
> Description
> -------
> 
> Fix for Amarok's kf5 branch failing to build. Right now it throws the following error:
> 
> CMakeFiles/amarok_storage-mysqlserverstorage.dir/MySqlServerStorageFactory.cpp.o: In function `factory::factory()':
> /home/somsubhra/kde/src/amarok/src/core-impl/storage/sql/mysqlserverstorage/MySqlServerStorageFactory.cpp:23: undefined reference to `vtable for factory'
> CMakeFiles/amarok_storage-mysqlserverstorage.dir/MySqlServerStorageFactory.cpp.o: In function `factory::~factory()':
> /home/somsubhra/kde/src/amarok/src/core-impl/storage/sql/mysqlserverstorage/MySqlServerStorageFactory.cpp:23: undefined reference to `vtable for factory'
> collect2: error: ld returned 1 exit status
> 
> So added the necessary meta object compiler generated file.
> 
> 
> Diffs
> -----
> 
>   src/core-impl/storage/sql/mysqlserverstorage/MySqlServerStorageFactory.cpp dd86d0d 
> 
> Diff: https://git.reviewboard.kde.org/r/125059/diff/
> 
> 
> Testing
> -------
> 
> Amarok builds successfully.
> 
> 
> Thanks,
> 
> Somsubhra Bairi
> 
>

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


More information about the Amarok-devel mailing list