wxHexEditor icon indicating copy to clipboard operation
wxHexEditor copied to clipboard

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

Open boom1 opened this issue 3 years ago • 5 comments

$ 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.

boom1 avatar Oct 15 '20 17:10 boom1

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

Eddi-z avatar Oct 16 '20 14:10 Eddi-z

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.

wtywtykk avatar Oct 24 '20 17:10 wtywtykk

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

beluxx avatar Nov 04 '20 01:11 beluxx

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

trick2011 avatar Aug 15 '23 11:08 trick2011

-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.

EUA avatar Oct 19 '23 23:10 EUA