Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build error: OpenPluginW #28

Open
asimonov opened this issue Nov 10, 2018 · 1 comment
Open

build error: OpenPluginW #28

asimonov opened this issue Nov 10, 2018 · 1 comment

Comments

@asimonov
Copy link

building on Mac (High Sierra with brew)

cloned this repo inside a clone of far2l (current master).

mkdir build
cd build
cmake ..

first issue: parallel make (-j4) gives random errors. feels like order of build is important

second issue (using sequential build):

[ 96%] Building CXX object far-gvfs/CMakeFiles/gvfspanel.dir/src/PluginMain.cpp.o
/Users/alexeysimonov/dev/far2l/far-gvfs/src/PluginMain.cpp:37:28: error: conflicting types for 'OpenPluginW'
SHAREDSYMBOL HANDLE WINAPI OpenPluginW(int openFrom, intptr_t item)
                           ^
/Users/alexeysimonov/dev/far2l/far-gvfs/../far2l/plugin.hpp:2313:24: note: previous declaration is here
        HANDLE WINAPI _export OpenPluginW(int OpenFrom,INT_PTR Item);
                              ^
1 error generated.
make[2]: *** [far-gvfs/CMakeFiles/gvfspanel.dir/src/PluginMain.cpp.o] Error 1
make[1]: *** [far-gvfs/CMakeFiles/gvfspanel.dir/all] Error 2
make: *** [all] Error 2
@cycleg
Copy link
Owner

cycleg commented Nov 18, 2018

Первое и самое главное: не использую Mac, поэтому проверить применительно к их ОС ничего не могу.

Постоянно сам пользуюсь многопоточной сборкой и ни разу не сталкивался с проблемами, хотя в обсуждениях far2l тема таких ошибок регулярно всплывает. Отсутствие зависимостей в коде gvfspanel от порядка сборки могу гарантировать.

По поводу второй ошибки предлагаю собрать, исправив декларацию OpenPluginW в PluginMain.cpp на:

SHAREDSYMBOL HANDLE WINAPI OpenPluginW(int openFrom, INT_PTR item)

А в файлах Plugin.cpp и Plugin.h поменять декларации метода Plugin::openPlugin() на

HANDLE openPlugin(int openFrom, INT_PTR item)

cycleg added a commit that referenced this issue Dec 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants