EUA / wxHexEditor

wxHexEditor official GIT repo
http://www.wxhexeditor.org
GNU General Public License v2.0
563 stars 121 forks source link

make under Fedora 32 fails: No rule to make target '-lgomp' #170

Open boom1 opened 4 years ago

boom1 commented 4 years ago

$ make make: *** No rule to make target '-lgomp', needed by 'src/HexEditorGui.o'. Stop.

IMHO it is not good to try to create $(LIBS), especially not "-lgomp". Below patch fixes this by introducing $(LOCAL_LIBS), which are meant to be "made" by make. Then $(LIBS) is just "-lgomp", which is an external dependency of the project. patch.txt

PS: Maybe there is a better name than LOCAL_LIBS.

Eddi-z commented 4 years ago

i try to compile in OpenSUSE, and i run into the same issue.

additionally, after applying your patch, i run into this issue that "itab.h" is missing, which seems to be related to this: https://github.com/vmt/udis86/issues/105

wtywtykk commented 4 years ago

i try to compile in OpenSUSE, and i run into the same issue.

additionally, after applying your patch, i run into this issue that "itab.h" is missing, which seems to be related to this: vmt/udis86#105

I run into this problem on fedora32, I tried:

goto the udis86 directory,and

autoreconf -i ./configure make

then everything is fine.

beluxx commented 4 years ago

grep lgomp

./COMPILE:122:Opps! I forgot to say that, you probably need to remove -lgomp flag from Makefile. ./Makefile:4:LIBS += -lgomp

remove LIBS += -lgomp, it worked

trick2011 commented 1 year ago

What does -lgomp even do? I needed to remove it too from the Makefile to compile it on Ubuntu 22.04

sidenote: why does the readme say "just make" and the COMPILE file has large instructions to actually do it. Which also explains this issue for macosx but not for other OS'es

EUA commented 1 year ago

-lgomp flag makes wxHexEditor enable multi-processing library functions. Why does it required? Well, if you search on a TB SSD, that will increase search speed. That's all.

I don't know proper way to add this libraries into makefile. Might be, it's better to remove all those libraries and use them as an external library do the job.