Bug 318527 - Error while compile kde-baseapps (nepomuk-core).
Summary: Error while compile kde-baseapps (nepomuk-core).
Status: RESOLVED FIXED
Alias: None
Product: kde-windows
Classification: Miscellaneous
Component: buildsystem (show other bugs)
Version: 4.10
Platform: Other Other
: NOR normal
Target Milestone: ---
Assignee: KDE-Windows
URL: http://techbase.kde.org/Getting_Start...
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-17 22:15 UTC by mparchet
Modified: 2013-04-25 13:19 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.10.3


Attachments
Error details kde-baseapps (nepomuk-core) (31.07 KB, text/plain)
2013-04-17 22:24 UTC, mparchet
Details
My git commit result (99 bytes, text/plain)
2013-04-21 20:05 UTC, mparchet
Details
git pull result branch kde_4.10 (191 bytes, text/plain)
2013-04-21 20:25 UTC, mparchet
Details
git merge result (77 bytes, text/plain)
2013-04-21 20:33 UTC, mparchet
Details
extractorplugin correction (4.60 KB, patch)
2013-04-23 21:41 UTC, mparchet
Details

Note You need to log in before you can comment on or make changes to this bug.
Description mparchet 2013-04-17 22:15:58 UTC
Hello,

When I tried to compile  kde-baseapps I get an error.

See attachement and url for more details.

Could you help me please

Best regards

mparchet

Reproducible: Always

Steps to Reproduce:
1.cd emerge
2.git checkout kde-4.10
3.emerge qt
4. emerge kdelibs
5. emerge kde-baseapps
Actual Results:  
See attachement for details

Expected Results:  
compile successfully
Comment 1 mparchet 2013-04-17 22:24:52 UTC
Created attachment 78995 [details]
Error details kde-baseapps (nepomuk-core)

The error details

Package kde-baseapps.

portage nepomuk-core-20120728.py.
Comment 2 Hannah von Reth 2013-04-18 07:59:30 UTC
Git commit b72a3c731730e935b9dc93d1f526ff0d88b5bce3 by Patrick von Reth.
Committed on 18/04/2013 at 09:56.
Pushed by vonreth into branch 'master'.

Fixed windows build by fixing the dll export

Backport from
https://projects.kde.org/projects/kdesupport/emerge/repository/revisions/master/changes/portage/kde/nepomuk-core/nepomuk-core-4.10.0-20130217.diff

M  +1    -1    services/fileindexer/indexer/extractorplugin.h

http://commits.kde.org/nepomuk-core/b72a3c731730e935b9dc93d1f526ff0d88b5bce3
Comment 3 Hannah von Reth 2013-04-18 08:00:18 UTC
Git commit b44434dc7a4dda3ebb854e966971804a50635f35 by Patrick von Reth.
Committed on 18/04/2013 at 09:56.
Pushed by vonreth into branch 'KDE/4.10'.

Fixed windows build by fixing the dll export

Backport from
https://projects.kde.org/projects/kdesupport/emerge/repository/revisions/master/changes/portage/kde/nepomuk-core/nepomuk-core-4.10.0-20130217.diff
(cherry picked from commit b72a3c731730e935b9dc93d1f526ff0d88b5bce3)

M  +1    -1    services/fileindexer/indexer/extractorplugin.h

http://commits.kde.org/nepomuk-core/b44434dc7a4dda3ebb854e966971804a50635f35
Comment 4 Hrvoje Senjan 2013-04-19 16:50:57 UTC
Patrick, can you please try another solution, as this same commit was already reverted with 190e7eac10dbeeb8080c7e457036a1a4968ec6fd.
Your solution breaks file indexing.
Thanks
Comment 5 mparchet 2013-04-21 20:05:03 UTC
Created attachment 79073 [details]
My git commit result

According to your proposal I put :

Git commit b44434dc7a4dda3ebb854e966971804a50635f35

Here is my result in attachment.

Best regards

mparchet
Comment 6 mparchet 2013-04-21 20:25:14 UTC
Created attachment 79075 [details]
git pull result branch kde_4.10

Hello,

I also tried git pull to update git.

See my result attachment.

Why kdevent.bat was modified by error ?

Best regards

mparchet
Comment 7 mparchet 2013-04-21 20:33:55 UTC
Created attachment 79076 [details]
git merge result

Hello,

I also tried this command.

git merge.

See attachment the result.

Best regards

Could you help me please ?

Best regards

mparchet
Comment 8 Hannah von Reth 2013-04-22 08:05:40 UTC
(In reply to comment #4)
> Patrick, can you please try another solution, as this same commit was
> already reverted with 190e7eac10dbeeb8080c7e457036a1a4968ec6fd.
> Your solution breaks file indexing.
> Thanks

Are you sure?
In Qt plugins must not use exports and removing the export clearly fixed the windows build.
Do you get a linker error, or does it fail on runtime?
Comment 9 mparchet 2013-04-22 17:04:20 UTC
Hello,

In answer to your email of 2013-04-22 08:05:40 UTC, here is the history :

On 2013-04-17 I opened this ticket 318527. here is the error when I compiled emerge kde-baseapps  : attachement (78995).

Your proposal : git commit b44434dc7a4dda3ebb854e966971804a50635f35 was faileld (attachement 79073).

Then I tried git pulll. See the result attachement (79075)

Thanks by advanced for your help.

Best regards

mparchet
Comment 10 Hrvoje Senjan 2013-04-22 18:54:37 UTC
(In reply to comment #8)
> (In reply to comment #4)
> > Patrick, can you please try another solution, as this same commit was
> > already reverted with 190e7eac10dbeeb8080c7e457036a1a4968ec6fd.
> > Your solution breaks file indexing.
> > Thanks
> 
> Are you sure?
> In Qt plugins must not use exports and removing the export clearly fixed the
> windows build.
> Do you get a linker error, or does it fail on runtime?
Vishesh should explain better. Vishesh?
Comment 11 Luca Beltrame 2013-04-23 09:51:13 UTC
IIRC the indexing fails at runtime, not at compile time.
Comment 12 Vishesh Handa 2013-04-23 10:43:54 UTC
It fails at runtime with the following error - 

16:06:31.473 nepomukindexer(28994)/nepomuk (strigi service): Could not create Extractor:  "nepomukffmpegextractor" 
16:06:31.473 nepomukindexer(28994)/nepomuk (strigi service): "The service 'Nepomuk FFmpeg Extractor' does not provide an interface 'Nepomuk2::ExtractorPlugin' with keyword ''" 

The reason it might be failing is cause the ExtractorPlugin is not part of any library, and each plugin compiles the extractorplugin.h/.cpp file on their own. Still trying to figure out why.
Comment 13 Patrick Spendrin 2013-04-23 11:54:57 UTC
This looks weird, is there something like a plugin interface besides the qt plugin interface? This could actually work by accident on Linux and fail to build on windows.
Comment 14 mparchet 2013-04-23 21:41:16 UTC
Created attachment 79402 [details]
extractorplugin correction

Hello,

I checked extractorplugin.h in version 4.10.0 and 4.10.2 and I 	found the patrick's correction. I diden't find it in the repertory, so I think that the correction has not been confirmed

See attachement. Can you check it and put it into correct git repository ?

Thanks by advanced

Best regards

mparchet
Comment 15 Vishesh Handa 2013-04-25 13:19:15 UTC
This has been fixed for 4.10.3 and master by moving ExtractorPlugin to a shared library.